Skip to content

[fix](fe) Ignore obsolete table meta change journal - #66549

Closed
gavinchou wants to merge 1 commit into
apache:masterfrom
gavinchou:gavin-core-6125-op1102-compat
Closed

[fix](fe) Ignore obsolete table meta change journal#66549
gavinchou wants to merge 1 commit into
apache:masterfrom
gavinchou:gavin-core-6125-op1102-compat

Conversation

@gavinchou

Copy link
Copy Markdown
Contributor

Proposed changes

Add compatibility for obsolete OP_TABLE_META_CHANGE journal entries written by old 26.0 cloud builds. FE now recognizes opcode 1102, consumes the old JSON payload, and does nothing during replay.

This prevents upgrade replay from failing with UNKNOWN Operation Type 1102.

Testing

  • sh build.sh --fe

@gavinchou
gavinchou requested a review from mymeiyi as a code owner August 6, 2026 11:31
@hello-stephen

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@gavinchou

Copy link
Copy Markdown
Contributor Author

/review

@gavinchou

Copy link
Copy Markdown
Contributor Author

run buildall

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Codex automated review failed and did not complete.

Error: All Codex review accounts are usage-limited; earliest retry is 2026-08-08T03:32:00Z.
Workflow run: https://github.com/apache/doris/actions/runs/31099220637

Please trigger /review again after that time.

@gavinchou

Copy link
Copy Markdown
Contributor Author

This change is no longer needed.

@gavinchou gavinchou closed this Aug 6, 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