Skip to content

branch-4.0: [fix](iceberg) Invalidate related-table cache on external table refresh #60867#60885

Merged
yiguolei merged 1 commit intobranch-4.0from
auto-pick-60867-branch-4.0
Mar 1, 2026
Merged

branch-4.0: [fix](iceberg) Invalidate related-table cache on external table refresh #60867#60885
yiguolei merged 1 commit intobranch-4.0from
auto-pick-60867-branch-4.0

Conversation

@github-actions
Copy link
Contributor

Cherry-picked from #60867

…sh (#60867)

Related PR: #57972 

Problem Summary:

In multi-FE deployments, Iceberg partition evolution (ADD/REPLACE/DROP
PARTITION KEY) can leave some FEs with a stale
`IcebergExternalTable.isValidRelatedTableCached` value. This stale cache
may cause incorrect partitioned/unpartitioned
judgment and query failures until the table is refreshed.

This PR:
- Clears the related-table validation cache during
`OP_REFRESH_EXTERNAL_TABLE` replay
  (`RefreshManager.refreshTableInternal()`).
- Removes the redundant per-operation manual cache clearing in
`IcebergMetadataOps`.
@github-actions github-actions bot requested a review from yiguolei as a code owner February 27, 2026 09:35
@Thearas
Copy link
Contributor

Thearas commented Feb 27, 2026

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?

@dataroaring dataroaring reopened this Feb 27, 2026
@Thearas
Copy link
Contributor

Thearas commented Feb 27, 2026

run buildall

@hello-stephen
Copy link
Contributor

FE UT Coverage Report

Increment line coverage 0.00% (0/2) 🎉
Increment coverage report
Complete coverage report

@yiguolei
Copy link
Contributor

skip buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Feb 28, 2026
@github-actions
Copy link
Contributor Author

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

@github-actions
Copy link
Contributor Author

PR approved by anyone and no changes requested.

@yiguolei
Copy link
Contributor

skip buildall

@yiguolei yiguolei merged commit 3f86bec into branch-4.0 Mar 1, 2026
28 of 31 checks passed
@github-actions github-actions bot deleted the auto-pick-60867-branch-4.0 branch March 1, 2026 03:37
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.

5 participants