Skip to content

Move appendable object creation outside transactions#25534

Open
jiangxinmeng1 wants to merge 8 commits into
matrixorigin:mainfrom
jiangxinmeng1:move-aobject-create-outside-txn
Open

Move appendable object creation outside transactions#25534
jiangxinmeng1 wants to merge 8 commits into
matrixorigin:mainfrom
jiangxinmeng1:move-aobject-create-outside-txn

Conversation

@jiangxinmeng1

Copy link
Copy Markdown
Contributor

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #25533

What this PR does / why we need it:

  • Create appendable objects as committed catalog entries from the appender path using TxnMgr.Now(), without attaching them to the active transaction.
  • Rebuild missing appendable objects during WAL replay from append command row IDs, then normalize their create timestamp after replay using the minimum observed commit timestamp.
  • Preserve apply-table-data semantics for appendable stats and reject unexpected transactional appendable-object create entries while keeping old WAL replay compatible.
  • Added/updated tests around appendable object replay and apply-table-data behavior.

@jiangxinmeng1 jiangxinmeng1 requested a review from XuPeng-SH as a code owner July 8, 2026 09:00
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature size/L Denotes a PR that changes [500,999] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants