Skip to content

[fix](regression) unstable case: partition_key_minmax#46597

Merged
englefly merged 1 commit intoapache:masterfrom
englefly:min-max-case
Jan 9, 2025
Merged

[fix](regression) unstable case: partition_key_minmax#46597
englefly merged 1 commit intoapache:masterfrom
englefly:min-max-case

Conversation

@englefly
Copy link
Copy Markdown
Contributor

@englefly englefly commented Jan 8, 2025

What problem does this PR solve?

root cause:
by default partition column stats analyze is disabled.
but in case partition_col_stats, it is turned on.

  1. make partition_key_minmax support partition column stats
  2. set nonConcurrent for partition_col_stats

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason
  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@englefly
Copy link
Copy Markdown
Contributor Author

englefly commented Jan 8, 2025

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Jan 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 8, 2025

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 8, 2025

PR approved by anyone and no changes requested.

@englefly
Copy link
Copy Markdown
Contributor Author

englefly commented Jan 8, 2025

run p0

@englefly
Copy link
Copy Markdown
Contributor Author

englefly commented Jan 8, 2025

run cloud_p0

@englefly englefly merged commit 0d784c0 into apache:master Jan 9, 2025
@englefly englefly deleted the min-max-case branch January 9, 2025 02:14
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