akashrn5 commented on a change in pull request #3851:
URL: https://github.com/apache/carbondata/pull/3851#discussion_r455977271



##########
File path: 
integration/spark/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/TestLoadDataGeneral.scala
##########
@@ -234,9 +234,25 @@ class TestLoadDataGeneral extends QueryTest with 
BeforeAndAfterEach {
       CarbonCommonConstants.BLOCKLET_SIZE_DEFAULT_VAL)
   }
 
+  test("test decimal value as null with global sort load") {

Review comment:
       @kunal642 had already fixed one issue regarding null value for string, 
now we got for decimal. He has added test case with string, int, double, int, 
bigint having null values in the test case, now decimal is added, please better 
to add a test case for all complex types also once. 




----------------------------------------------------------------
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