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:
us...@infra.apache.org


Reply via email to