Skip to content

[fix](regression) wait row count for partition_key_minmax#65124

Merged
hello-stephen merged 1 commit into
apache:masterfrom
shuke987:fix-partition-key-minmax-row-count-wait
Jul 2, 2026
Merged

[fix](regression) wait row count for partition_key_minmax#65124
hello-stephen merged 1 commit into
apache:masterfrom
shuke987:fix-partition-key-minmax-row-count-wait

Conversation

@shuke987

@shuke987 shuke987 commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Proposed changes

partition_key_minmax checks partition-pruned memo-plan statistics. In Cloud P0, this case can run the EXPLAIN before the physical index row count has finished reporting after insert/analyze. When that happens, the planner's selected-partition row count can be 0, so the existing ndv/count assertions fail even though the partition-key min/max is already correct.

This patch keeps the existing EXPLAIN assertions and adds a bounded wait for the base index strict row count via show index stats <table> <index> before running each memo-plan check.

Testing

  • git diff --check

No local regression environment was allocated in this session, so the targeted regression case was not run locally.

@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?

@shuke987

shuke987 commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator Author

run buildall

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

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@hello-stephen hello-stephen merged commit 23c42db into apache:master Jul 2, 2026
33 checks passed
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