dongjoon-hyun commented on pull request #32826:
URL: https://github.com/apache/spark/pull/32826#issuecomment-859136503


   It's just a historical factor. IMO, I believe that we need to replace it to 
`zstd-jni`.
   > Does it make sense to use aircompressor for ZSTD in ORC, rather than the 
zstd-jni?
   
   Yes, `aircompressor` is behind and also has ZSTD bug. That's the reason why 
the community (not only Apache ORC, but also Presto) complains at the new 
version of aircompressor.
   - https://github.com/airlift/aircompressor/issues/122
   
   BTW, please note that your PR is merged to Apache ORC 1.7 which has no 
release plan yet. The situation is the same for the other community. Apache 
Kafka with ZSTD 1.5? Apache Avro with ZSTD 1.5? Apache Parquet with ZSTD 1.5? 
Apache Spark should embrace those Apache Projects together because our 
customers are able to use them together in a single app.


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to