Skip to content

feat: update commit message to version 12 - #179

Merged
SteNicholas merged 4 commits into
apache:mainfrom
lszskye:update_commit_message_version
Aug 5, 2026
Merged

feat: update commit message to version 12#179
SteNicholas merged 4 commits into
apache:mainfrom
lszskye:update_commit_message_version

Conversation

@lszskye

@lszskye lszskye commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Purpose

Add source_meta to GlobalIndexMeta and update the commit message version from 11 to 12.

Commit message version 11 uses a five-field GlobalIndexMeta schema. Reading it with the new six-field schema would break compatibility, so this change introduces IndexFileMetaV4Deserializer to deserialize version 11 commit messages with the legacy schema. Version 12 uses the current schema containing the nullable _SOURCE_META field.

Tests

  • Added IndexFileMetaSerializerTest.TestToFromRowWithGlobalIndexWithSourceMeta to verify source_meta row serialization and deserialization.
  • Updated CommitMessageTest.TestCompatibleWithVersion11 to verify that existing version 11 commit messages remain readable through the legacy deserializer.
  • Existing tests continue to cover GlobalIndexMeta without source_meta.

API and Format

This changes the commit message serialization format, the current commit message version is increased from 11 to 12.

@lszskye
lszskye force-pushed the update_commit_message_version branch from c1ae920 to 48a070e Compare August 5, 2026 03:56
@lszskye
lszskye force-pushed the update_commit_message_version branch from 138a6d0 to 48a87d0 Compare August 5, 2026 04:38

@lxy-9602 lxy-9602 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@SteNicholas SteNicholas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@SteNicholas
SteNicholas merged commit 653cf18 into apache:main Aug 5, 2026
9 checks passed
SteNicholas pushed a commit that referenced this pull request Aug 5, 2026
* feat: update commit message to version 12

* add compatibility test

* fix pre-commit
SteNicholas pushed a commit that referenced this pull request Aug 5, 2026
* feat: update commit message to version 12

* add compatibility test

* fix pre-commit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants