Skip to content

feat: support fixed-bucket partial-update merge engine#263

Open
QuakeWang wants to merge 2 commits intoapache:mainfrom
QuakeWang:feat/partial-update-groundwork
Open

feat: support fixed-bucket partial-update merge engine#263
QuakeWang wants to merge 2 commits intoapache:mainfrom
QuakeWang:feat/partial-update-groundwork

Conversation

@QuakeWang
Copy link
Copy Markdown
Contributor

@QuakeWang QuakeWang commented Apr 18, 2026

Purpose

Linked issue: part of #256

This PR delivers a minimum usable merge-engine=partial-update implementation for primary-key tables with fixed buckets.
It shifts the earlier groundwork-heavy approach toward end-to-end functionality, while keeping only the validation that is still necessary for correctness.

This PR does not close #256 yet. Support for bucket=-1 partial-update, including cross-partition existing-key behavior, remains follow-up work.

Brief change log

  • support fixed-bucket merge-engine=partial-update end to end for primary-key tables
  • keep fail-fast for unsupported advanced partial-update options
  • remove redundant validation and generic guard rails
  • implement the read/write path changes required by basic latest-non-null partial-update semantics
  • keep bucket=-1 and deletion-vectors.enabled=true explicitly unsupported in this PR
  • add fixed-bucket E2E and targeted regression tests

Tests

API and Format

Documentation

Copy link
Copy Markdown
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

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

Can we simplify some of the code? There's no need for so much validation; we just need to focus on E2E and functionality.

@QuakeWang QuakeWang changed the title feat: add partial-update merge engine groundwork feat: support fixed-bucket partial-update merge engine Apr 19, 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.

Support partial-update merge-engine

2 participants