[GitHub] ravipesala commented on a change in pull request #3099: [CARBONDATA-3268] Fix for query on Varchar Columns showing Null in Presto

[GitBox <gi...@apache.org>]

ravipesala commented on a change in pull request #3099: [CARBONDATA-3268] Fix for query on Varchar Columns showing Null in Presto
URL: https://github.com/apache/carbondata/pull/3099#discussion_r251221315
 
 

 ##########
 File path: integration/presto/src/test/scala/org/apache/carbondata/presto/integrationtest/PrestoTestNonTransactionalTableFiles.scala
 ##########
 @@ -97,7 +97,7 @@ class PrestoTestNonTransactionalTableFiles extends FunSuiteLike with BeforeAndAf
       sortColumns: List[String]): Any = {
     val schema = new StringBuilder()
       .append("[ \n")
-      .append("   {\"NaMe\":\"string\"},\n")
+      .append("   {\"name\":\"varchar\"},\n")
 
 Review comment:
   Instead of updating the existing column to varchar, add new varchar column type and insert the long string more than 32k and verify with testcase.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services