Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2350#discussion_r191330085
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonProperties.java ---
@@ -1290,11 +1290,11 @@ public String getSortTempCompressor() {
String compressor =
getProperty(CarbonCommonConstants.CARBON_SORT_TEMP_COMPRESSOR,
CarbonCommonConstants.CARBON_SORT_TEMP_COMPRESSOR_DEFAULT).toUpperCase();
if (compressor.isEmpty() || "SNAPPY".equals(compressor) ||
"GZIP".equals(compressor)
- || "BZIP2".equals(compressor) || "LZ4".equals(compressor)) {
+ || "BZIP2".equals(compressor) || "LZ4".equals(compressor) ||
"ZSTD".equals(compressor)) {
return compressor;
} else {
LOGGER.warn("The
".concat(CarbonCommonConstants.CARBON_SORT_TEMP_COMPRESSOR)
- .concat(" configuration value is invalid. Only
snappy,gzip,bip2,lz4 and")
+ .concat(" configuration value is invalid. Only
snappy,gzip,bip2,lz4,zstd and")
--- End diff --
change to `, zstd`
---