Skip to content

feat: implement incremental changelog scan#611

Open
WZhuo wants to merge 2 commits intoapache:mainfrom
WZhuo:incremental_changelog_scan
Open

feat: implement incremental changelog scan#611
WZhuo wants to merge 2 commits intoapache:mainfrom
WZhuo:incremental_changelog_scan

Conversation

@WZhuo
Copy link
Copy Markdown
Contributor

@WZhuo WZhuo commented Mar 31, 2026

No description provided.

@WZhuo WZhuo force-pushed the incremental_changelog_scan branch 2 times, most recently from 1c908dd to c647f06 Compare March 31, 2026 11:53
Copy link
Copy Markdown
Member

@wgtmac wgtmac left a comment

Choose a reason for hiding this comment

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

Mostly looks good! I have some inline comments.

Comment thread src/iceberg/table_scan.h Outdated
Comment thread src/iceberg/table_scan.h Outdated
Comment thread src/iceberg/table_scan.h Outdated
Comment thread src/iceberg/table_scan.h
Comment thread src/iceberg/table_scan.h Outdated
Comment thread src/iceberg/table_scan.cc Outdated
Comment thread src/iceberg/table_scan.cc Outdated
Comment thread src/iceberg/table_scan.cc
Comment thread src/iceberg/test/incremental_changelog_scan_test.cc
Comment thread src/iceberg/test/scan_test_base.h
@WZhuo WZhuo force-pushed the incremental_changelog_scan branch 3 times, most recently from ec12ad3 to 20cd399 Compare April 16, 2026 07:54
@wgtmac wgtmac force-pushed the incremental_changelog_scan branch from 20cd399 to 8de07c9 Compare April 16, 2026 16:04
Copy link
Copy Markdown
Member

@wgtmac wgtmac left a comment

Choose a reason for hiding this comment

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

+1

@wgtmac wgtmac changed the title feat: Impl Incremental changelog scan feat: implement incremental changelog scan Apr 16, 2026
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.

2 participants