You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by GitBox <gi...@apache.org> on 2020/10/14 09:48:59 UTC

[GitHub] [carbondata] akashrn5 opened a new pull request #3984: [WIP]Fix MV query issue with aggregation on decimal column

akashrn5 opened a new pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984


    ### Why is this PR needed?
    
    
    ### What changes were proposed in this PR?
   
       
    ### Does this PR introduce any user interface change?
    - No
    - Yes. (please explain the change and update document)
   
    ### Is any new testcase added?
    - No
    - Yes
   
       
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] akashrn5 commented on a change in pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
akashrn5 commented on a change in pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#discussion_r510104859



##########
File path: integration/spark/src/test/scala/org/apache/carbondata/view/rewrite/MVCreateTestCase.scala
##########
@@ -1471,6 +1471,16 @@ class MVCreateTestCase extends QueryTest with BeforeAndAfterAll {
     sql("drop table if exists t2")
   }
 
+  test("test sum aggregations on decimal columns") {
+    sql("drop table if exists sum_agg_decimal")
+    sql("create table sum_agg_decimal(salary1 decimal(7,2),salary2 decimal(7,2),salary3 decimal(7,2),salary4 decimal(7,2),empname string) stored as carbondata")
+    sql("drop materialized view if exists decimal_mv")
+    sql("create materialized view decimal_mv as select empname, sum(salary1 - salary2) from sum_agg_decimal group by empname")
+    sql("explain select empname, sum( salary1 - salary2) from sum_agg_decimal group by empname").show(false)

Review comment:
       done




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3984: [WIP]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#issuecomment-708363702


   Build Failed  with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4434/
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
Indhumathi27 commented on a change in pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#discussion_r507702557



##########
File path: integration/spark/src/test/scala/org/apache/carbondata/view/rewrite/MVCreateTestCase.scala
##########
@@ -1471,6 +1471,16 @@ class MVCreateTestCase extends QueryTest with BeforeAndAfterAll {
     sql("drop table if exists t2")
   }
 
+  test("test sum aggregations on decimal columns") {
+    sql("drop table if exists sum_agg_decimal")
+    sql("create table sum_agg_decimal(salary1 decimal(7,2),salary2 decimal(7,2),salary3 decimal(7,2),salary4 decimal(7,2),empname string) stored as carbondata")
+    sql("drop materialized view if exists decimal_mv")
+    sql("create materialized view decimal_mv as select empname, sum(salary1 - salary2) from sum_agg_decimal group by empname")
+    sql("explain select empname, sum( salary1 - salary2) from sum_agg_decimal group by empname").show(false)

Review comment:
       can revert this change




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] Indhumathi27 commented on pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
Indhumathi27 commented on pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#issuecomment-715202939


   LGTM


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#issuecomment-714585851


   Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4633/
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] akashrn5 commented on a change in pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
akashrn5 commented on a change in pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#discussion_r510104944



##########
File path: integration/spark/src/test/scala/org/apache/carbondata/view/rewrite/MVCreateTestCase.scala
##########
@@ -1471,6 +1471,16 @@ class MVCreateTestCase extends QueryTest with BeforeAndAfterAll {
     sql("drop table if exists t2")
   }
 
+  test("test sum aggregations on decimal columns") {
+    sql("drop table if exists sum_agg_decimal")

Review comment:
       done




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] asfgit closed pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
Indhumathi27 commented on a change in pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#discussion_r507703419



##########
File path: integration/spark/src/test/scala/org/apache/carbondata/view/rewrite/MVCreateTestCase.scala
##########
@@ -1471,6 +1471,16 @@ class MVCreateTestCase extends QueryTest with BeforeAndAfterAll {
     sql("drop table if exists t2")
   }
 
+  test("test sum aggregations on decimal columns") {
+    sql("drop table if exists sum_agg_decimal")

Review comment:
       can add drop to afterAll also




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3984: [CARBONDATA-4035]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#issuecomment-714559131


   Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/2877/
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3984: [WIP]Fix MV query issue with aggregation on decimal column

Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3984:
URL: https://github.com/apache/carbondata/pull/3984#issuecomment-708395934


   Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/2681/
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org