Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[opt](nereids) fix non-null selectivity computing #42286

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

xzj7019
Copy link
Contributor

@xzj7019 xzj7019 commented Oct 22, 2024

Introduced by pr: #40762
which doesn't update numNulls and other related column stats when they needs to be normalized.
This pr adds the updating trigger condition and makes the related case healthy.

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@xzj7019
Copy link
Contributor Author

xzj7019 commented Oct 22, 2024

run buildall

@xzj7019 xzj7019 force-pushed the refine_stats_derive_numnulls branch from ffb75e5 to d0d25ab Compare October 23, 2024 02:35
@xzj7019
Copy link
Contributor Author

xzj7019 commented Oct 23, 2024

run buildall

@xzj7019 xzj7019 force-pushed the refine_stats_derive_numnulls branch from d0d25ab to 437d65b Compare October 23, 2024 05:27
@xzj7019
Copy link
Contributor Author

xzj7019 commented Oct 23, 2024

run buildall

@xzj7019 xzj7019 force-pushed the refine_stats_derive_numnulls branch from 4165bd3 to 0dd643e Compare October 23, 2024 07:26
@xzj7019
Copy link
Contributor Author

xzj7019 commented Oct 23, 2024

run buildall

@xzj7019 xzj7019 force-pushed the refine_stats_derive_numnulls branch from 0dd643e to 552a70e Compare October 23, 2024 08:08
@xzj7019
Copy link
Contributor Author

xzj7019 commented Oct 23, 2024

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Oct 23, 2024
Copy link
Contributor

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

Copy link
Contributor

PR approved by anyone and no changes requested.

@morrySnow morrySnow merged commit 6dcc221 into apache:master Oct 24, 2024
25 of 27 checks passed
morrySnow pushed a commit that referenced this pull request Oct 24, 2024
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. dev/3.0.3-merged reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants