Github user KanakaKumar commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2525#discussion_r203637393
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datastore/filesystem/LocalCarbonFile.java
 ---
    @@ -293,7 +291,12 @@ public boolean delete() {
         } else if ("LZ4".equalsIgnoreCase(compressor)) {
           inputStream = new LZ4BlockInputStream(new FileInputStream(path));
         } else if ("ZSTD".equalsIgnoreCase(compressor)) {
    -      inputStream = new ZstdInputStream(new FileInputStream(path));
    +      try {
    +        inputStream = (InputStream) 
Class.forName("com.github.luben.zstd.ZstdInputStream")
    +            .getConstructors()[0].newInstance(new FileInputStream(path));
    +      } catch (ReflectiveOperationException e) {
    +        throw new RuntimeException(e);
    --- End diff --
    
    Throw IOException with a hint to  "Ensure ZSTD lib is in class path or not"


---

Reply via email to