VenuReddy2103 commented on a change in pull request #4073:
URL: https://github.com/apache/carbondata/pull/4073#discussion_r556262687
##########
File path:
integration/presto/src/test/scala/org/apache/carbondata/presto/integrationtest/PrestoTestUsingSparkStore.scala
##########
@@ -325,4 +326,22 @@ class PrestoTestUsingSparkStore
}
+ test("Test decimal unscaled converter") {
+ prestoServer
+ .execute(
+ "create table presto_spark_db.array_decimal(salary
array(decimal(20,3)) ) with" +
+ "(format='CARBON') ")
+ copyStoreContents("array_decimal")
+ val result: List[Map[String, Any]] = prestoServer
+ .executeQuery("SELECT * FROM presto_spark_db.array_decimal")
+ assert(result.size == 1)
+ for (i <- 0 to 0) {
Review comment:
result size is only 1. Suggest to remove this redundant loop if not
required.
----------------------------------------------------------------
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:
[email protected]
