Skip to content

fix merge on read #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Mar 13, 2025
Merged

fix merge on read #27

merged 1 commit into from
Mar 13, 2025

Conversation

@chenzl25
Copy link
Collaborator

@chenzl25 chenzl25 commented Mar 12, 2025

Fix merge on read:

  • change FileScanTaskDeleteFile to FileScanTask
  • bypass position delete read stream
  • add sequence number and equity delete to FileScanTask
  • Fix position delete and equality delete sequence number comparison
  • add position delete to the global delete map

@chenzl25 chenzl25 requested review from Li0k, ZENOTME, xxchan and xxhZs March 12, 2025 10:54
@xxhZs xxhZs merged commit 7f3a9d7 into dev_rebase_main_20250307 Mar 13, 2025
11 checks passed
@xxchan xxchan mentioned this pull request Mar 13, 2025
Open
@chenzl25 chenzl25 mentioned this pull request Mar 25, 2025
Merged
chenzl25 added a commit that referenced this pull request Mar 25, 2025
@chenzl25
fix: cherry-pick #27 (#41)
c9da916
Li0k pushed a commit that referenced this pull request Sep 29, 2025
@chenzl25 @Li0k
fix: cherry-pick #27 (#41)
5e3b2e3
chenzl25 added a commit that referenced this pull request Nov 5, 2025
@Li0k @chenzl25
fix: cherry-pick #27 (#41) (#81)
a843556 
Co-authored-by: Dylan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xxhZs xxhZs xxhZs approved these changes

@xxchan xxchan Awaiting requested review from xxchan

@Li0k Li0k Awaiting requested review from Li0k

@ZENOTME ZENOTME Awaiting requested review from ZENOTME

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chenzl25 @xxhZs