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

    https://github.com/apache/carbondata/pull/2863#discussion_r230257378
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datastore/compression/SnappyCompressor.java
 ---
    @@ -90,7 +90,7 @@ public String getName() {
         try {
           uncompressedLength = Snappy.uncompressedLength(compInput, offset, 
length);
           data = new byte[uncompressedLength];
    -      Snappy.uncompress(compInput, offset, length, data, 0);
    +      snappyNative.rawUncompress(compInput, offset, length, data, 0);
    --- End diff --
    
    is it safe to use `SnappyNative` class directly? It's documentation says we 
should not use this class directly


---

Reply via email to