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
[CARBONDATA-3899] Drop materialized view when executed concurrently from 4 concurrent client fails in all 4 clients. #3841
Conversation
Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1641/ |
Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/3382/ |
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1644/ |
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/3385/ |
@ShreelekhyaG Please create a jira and update the PR |
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1680/ |
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/3423/ |
integration/spark/src/test/scala/org/apache/carbondata/view/rewrite/MVCreateTestCase.scala
Show resolved
Hide resolved
integration/spark/src/test/scala/org/apache/carbondata/view/MVTest.scala
Outdated
Show resolved
Hide resolved
core/src/main/java/org/apache/carbondata/core/view/MVManager.java
Outdated
Show resolved
Hide resolved
...n/spark/src/main/scala/org/apache/spark/sql/execution/command/view/CarbonDropMVCommand.scala
Outdated
Show resolved
Hide resolved
integration/spark/src/main/scala/org/apache/carbondata/view/MVManagerInSpark.scala
Outdated
Show resolved
Hide resolved
integration/spark/src/main/scala/org/apache/carbondata/view/MVManagerInSpark.scala
Show resolved
Hide resolved
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/3457/ |
Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1715/ |
retest this please |
…lient fails in all 4 clients Added Drop MV check with ifexists
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/3473/ |
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1731/ |
LGTM |
1 similar comment
LGTM |
retest this please |
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/3486/ |
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1744/ |
Why is this PR needed?
Also, materialized view when created in one session and queried on another open session will not hit MV.
What changes were proposed in this PR?
Does this PR introduce any user interface change?
Is any new testcase added?