-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[BugFix] fix corrupted partition meta exception
when refreshing materialized views…
#54487
[BugFix] fix corrupted partition meta exception
when refreshing materialized views…
#54487
Conversation
77dfccb
to
a9a6f21
Compare
corrupted partition meta exception
when refreshing materialized views…
…erialized views using JDBC Catalog StarRocks#54486 Signed-off-by: AlgoLin <[email protected]>
a9a6f21
to
ad01594
Compare
Quality Gate passedIssues Measures |
[Java-Extensions Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
[FE Incremental Coverage Report]✅ pass : 5 / 5 (100.00%) file detail
|
[BE Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
This is just a walk-aroud, since it will be triggered to be refreshed for each refresh because we can not deduce the partition's real refresh time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@Mergifyio backport branch-3.4 |
@Mergifyio backport branch-3.3 |
@Mergifyio backport branch-3.2 |
✅ Backports have been created
|
✅ Backports have been created
|
✅ Backports have been created
|
…erialized views… (#54487) Signed-off-by: AlgoLin <[email protected]> (cherry picked from commit 9478087)
…erialized views… (#54487) Signed-off-by: AlgoLin <[email protected]> (cherry picked from commit 9478087)
…erialized views… (#54487) Signed-off-by: AlgoLin <[email protected]> (cherry picked from commit 9478087)
yes, when we cannot get any partition statistics from the external catalog, it may be easier for users to accept this way. I see that SR uses the same method when querying paimon or PostgreSQL. |
fix the unexpected errors when refreshing materialized views using JDBC Catalog
Why I'm doing:
see more #54486
What I'm doing:
Fixes #54486
What type of PR is this:
Does this PR entail a change in behavior?
Checklist:
Bugfix cherry-pick branch check: