Skip to content

branch-4.1: [fix](meta) Rebuild view column lookup after metadata recovery #69150 - #69482

Merged
yiguolei merged 1 commit into
branch-4.1from
auto-pick-69150-branch-4.1
Oct 23, 2026
Merged

yiguolei merged 1 commit into
branch-4.1from
auto-pick-69150-branch-4.1

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Cherry-picked from #69150

### What problem does this PR solve?
After FE metadata recovery, creating a row policy on a view can fail
with `column not exist`, even though `DESC` shows the column.

`View.gsonPostProcess()` overrides the parent method without calling it,
leaving the non-persisted `nameToColumn` map empty. This affects both
image loading and create-table journal replay.

Call `super.gsonPostProcess()` to rebuild the column lookup from the
restored schema.

### Release note
Fix CREATE ROW POLICY incorrectly reporting missing view columns after
FE metadata recovery.
@github-actions
github-actions Bot requested a review from yiguolei as a code owner October 22, 2026 17:49
@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?

@hello-stephen

Copy link
Copy Markdown
Contributor

run buildall

@hello-stephen

Copy link
Copy Markdown
Contributor

FE UT Coverage Report

Increment line coverage 100.00% (1/1) 🎉
Increment coverage report
Complete coverage report

@hello-stephen

Copy link
Copy Markdown
Contributor

FE Regression Coverage Report

Increment line coverage 100.00% (1/1) 🎉
Increment coverage report
Complete coverage report

@yiguolei
yiguolei merged commit 00a9e1c into branch-4.1 Oct 23, 2026
35 of 36 checks passed
@github-actions

Copy link
Copy Markdown
Contributor Author

PR approved by anyone and no changes requested.

@github-actions github-actions Bot added the approved Indicates a PR has been approved by one committer. label Oct 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor Author

PR approved by at least one committer and no changes requested.

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

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants