Github user zzcclp commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2397#discussion_r197372024
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/TestLoadDataWithUnsafeMemory.scala
---
@@ -64,6 +67,8 @@ class TestLoadDataWithUnsafeMemory extends QueryTest
.addProperty(CarbonCommonConstants.UNSAFE_WORKING_MEMORY_IN_MB,
"512")
CarbonProperties.getInstance()
.addProperty(CarbonCommonConstants.OFFHEAP_SORT_CHUNK_SIZE_IN_MB,
"512")
+ CarbonProperties.getInstance()
+
.addProperty(CarbonLoadOptionConstants.CARBON_LOAD_SORT_MEMORY_SPILL_PERCENTAGE,
"-1")
--- End diff --
why use "-1" here?
---
