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`


---

Reply via email to