[GitHub] [incubator-kyuubi] pan3793 commented on pull request #3877: [KYUUBI #3865] Arrow-based result compression support

[GitBox <gi...@apache.org>]

pan3793 commented on PR #3877:
URL: https://github.com/apache/incubator-kyuubi/pull/3877#issuecomment-1336126170

   Forwarding informations from the offline discussion:
   
   1. we plan to defer this feature after Spark Connect implemented arrow compression, because there are some different ways(e.g. arrow native, stream compression, block compression) to implement compression, we'd better to keep consistent w/ upstream.
   2. [aircompressor](https://github.com/airlift/aircompressor) is good option for client side, the roughly testing shows it has same decompression performance as lz4-java and zstd-jni, but there are some memory allocation issue on lz4 stream decompression.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org