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 check sync materialized view exist bug #30038
[BugFix] Fix check sync materialized view exist bug #30038
Conversation
81c4cf2
to
382dba9
Compare
Signed-off-by: shuming.li <ming.moriarty@gmail.com>
382dba9
to
1a1d380
Compare
SonarCloud Quality Gate failed. 3 Bugs 0.0% Coverage The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
[FE Incremental Coverage Report]😍 pass : 12 / 12 (100.00%) file detail
|
[BE Incremental Coverage Report]😍 pass : 0 / 0 (0%) |
@Mergifyio backport branch-3.1 |
@Mergifyio backport branch-3.0 |
✅ Backports have been created
|
✅ Backports have been created
|
* Fix check sync materialized view exist bug Signed-off-by: shuming.li <ming.moriarty@gmail.com> * fix bugs Signed-off-by: shuming.li <ming.moriarty@gmail.com> --------- Signed-off-by: shuming.li <ming.moriarty@gmail.com> (cherry picked from commit d299ddd) # Conflicts: # fe/fe-core/src/main/java/com/starrocks/alter/MaterializedViewHandler.java
* Fix check sync materialized view exist bug Signed-off-by: shuming.li <ming.moriarty@gmail.com> * fix bugs Signed-off-by: shuming.li <ming.moriarty@gmail.com> --------- Signed-off-by: shuming.li <ming.moriarty@gmail.com> (cherry picked from commit d299ddd) # Conflicts: # fe/fe-core/src/main/java/com/starrocks/alter/MaterializedViewHandler.java # fe/fe-core/src/test/java/com/starrocks/analysis/CreateMaterializedViewTest.java
* Fix check sync materialized view exist bug Signed-off-by: shuming.li <ming.moriarty@gmail.com> * fix bugs Signed-off-by: shuming.li <ming.moriarty@gmail.com> --------- Signed-off-by: shuming.li <ming.moriarty@gmail.com>
((OlapTable) tbl).getIndexNameById(mvMeta.getIndexId())
is null (there maybe some corrupt in the table), create sync mv will always failWhat type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check: