[ 
https://issues.apache.org/jira/browse/AVRO-2162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16410202#comment-16410202
 ] 

ASF GitHub Bot commented on AVRO-2162:
--------------------------------------

scottcarey commented on a change in pull request #303: AVRO-2162 Adds Zstandard 
compression to the Avro File Format (Java)
URL: https://github.com/apache/avro/pull/303#discussion_r176556430
 
 

 ##########
 File path: 
lang/java/avro/src/main/java/org/apache/avro/file/DataFileStream.java
 ##########
 @@ -351,16 +349,27 @@ ByteBuffer getAsByteBuffer() {
       return ByteBuffer.wrap(data, offset, blockSize);
     }
 
+    void setBytes(ByteBuffer block) {
 
 Review comment:
   this now supports Codecs that return Direct ByteBuffers.  Earlier versions 
of the ZstandardCodec were using APIs that returned direct buffers, which 
exploded here. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Add Zstandard compression to avro file format
> ---------------------------------------------
>
>                 Key: AVRO-2162
>                 URL: https://issues.apache.org/jira/browse/AVRO-2162
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>            Reporter: Scott Carey
>            Priority: Major
>
> I'd like to add Zstandard compression for Avro. 
> At compression level 1 It is almost as fast as Snappy at compression, with 
> compression ratios more like gzip.  At higher levels of compression, it is 
> more compact than gzip -9 with much lower CPU when compressing and roughly 3x 
> faster decompression.
>  
> Adding it to Java is fairly easy.  We'll need to say something about it in 
> the spec however, as an 'optinal' codec.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to