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

[improvement](statistics)Return -1 to neredis if report olap table row count for new table is not done for all tablets. (#40457) #40540

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

Jibing-Li
Copy link
Contributor

@Jibing-Li Jibing-Li commented Sep 9, 2024

backport: 40457
reverted

…w count for new table is not done for all tablets. (apache#40457)

Return -1 to neredis if report olap table row count for new table is not
done for all tablets.
After this change, nereids could know new table is empty or not. When
it's not empty but not reported yet, return -1 as row count to nereids.
@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.

@Jibing-Li Jibing-Li marked this pull request as ready for review September 9, 2024 07:34
Copy link
Contributor

github-actions bot commented Sep 9, 2024

clang-tidy review says "All clean, LGTM! 👍"

@Jibing-Li
Copy link
Contributor Author

run buildall

@Jibing-Li
Copy link
Contributor Author

run beut

@yiguolei yiguolei merged commit e43e6e2 into apache:branch-2.1 Sep 10, 2024
19 of 22 checks passed
yiguolei added a commit that referenced this pull request Sep 10, 2024
…table row count for new table is not done for all tablets. (#40457)" (#40616)

Reverts #40540
@Jibing-Li Jibing-Li deleted the row2.1 branch September 14, 2024 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants