Skip to content

branch-4.0: [fix](fe) Skip dropped columns in follower stats sync #63882#63947

Open
yujun777 wants to merge 1 commit into
apache:branch-4.0from
yujun777:branch-4.0-pick-63882
Open

branch-4.0: [fix](fe) Skip dropped columns in follower stats sync #63882#63947
yujun777 wants to merge 1 commit into
apache:branch-4.0from
yujun777:branch-4.0-pick-63882

Conversation

@yujun777
Copy link
Copy Markdown
Contributor

@yujun777 yujun777 commented Jun 1, 2026

cherry-pick: #63882

### What problem does this PR solve?

FollowerColumnSender drains queued column references on follower FEs and
syncs the columns that still need analysis to the master. A queued
column can become stale after DDL changes. If the table still exists but
the queued column has been dropped, table.getColumn(column.colName)
returns null and the sender throws a NullPointerException while reading
the type.

This patch skips dropped columns before checking the column type, so the
daemon does not emit periodic ERROR logs and can continue processing the
remaining queued columns.
@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?

@yujun777
Copy link
Copy Markdown
Contributor Author

yujun777 commented Jun 1, 2026

run buildall

@yujun777
Copy link
Copy Markdown
Contributor Author

yujun777 commented Jun 1, 2026

run cloud_p0

@yujun777
Copy link
Copy Markdown
Contributor Author

yujun777 commented Jun 2, 2026

run nonConcurrent

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