Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2274#discussion_r187530598
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -340,7 +342,13 @@ private CarbonLoadModel createLoadModel() throws
IOException, InvalidLoadOptionE
// we are still using the traditional carbon table folder structure
persistSchemaFile(table, CarbonTablePath.getSchemaFilePath(path));
}
-
+ if (!table.isTransactionalTable()) {
+ CarbonProperties.getInstance()
+ .addProperty(CarbonCommonConstants.ENABLE_OFFHEAP_SORT, "false");
--- End diff --
But this will overwrite the ENABLE_OFFHEAP_SORT is the user has set it.
Need to check if this is already set. If not then set to false otherwise use
what the user has set.
---
