Skip to content

Conversation

@zhyass
Copy link
Member

@zhyass zhyass commented Aug 28, 2025

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

This commit ensures that the location field in AdditionalStatsMeta is always set (never None) when creating snapshots. Previously, location could be None, causing compatibility issues during schema evolution. By providing a default placeholder (("", 0)), this change facilitates smoother upgrades and backward compatibility for downstream consumers.

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

@github-actions github-actions bot added pr-backport pr-bugfix this PR patches a bug in codebase labels Aug 28, 2025
@zhyass zhyass added ci-cloud Build docker image for cloud test and removed ci-cloud Build docker image for cloud test labels Aug 28, 2025
@zhyass zhyass added ci-cloud Build docker image for cloud test ci-benchmark Benchmark: run all test and removed ci-cloud Build docker image for cloud test ci-benchmark Benchmark: run all test labels Aug 28, 2025
@everpcpc everpcpc removed the ci-cloud Build docker image for cloud test label Aug 28, 2025
@zhyass zhyass added the ci-cloud Build docker image for cloud test label Aug 28, 2025
@github-actions
Copy link
Contributor

Docker Image for PR

  • tag: pr-18629-43109ec-1756372681

note: this image tag is only available for internal use.

@zhyass zhyass marked this pull request as draft August 28, 2025 10:07
@zhyass zhyass changed the title fix: ensure additionalStatsMeta.location is always populated in snapshots fix: [DO NOT MERGE]ensure additionalStatsMeta.location is always populated in snapshots Aug 28, 2025
@BohuTANG BohuTANG closed this Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-cloud Build docker image for cloud test pr-backport pr-bugfix this PR patches a bug in codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants