You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/02/25 13:13:31 UTC

[GitHub] [spark] seayoun edited a comment on pull request #31645: [SPARK-30228][BUILD][2.4] Update zstd-jni to 1.4.4-3

seayoun edited a comment on pull request #31645:
URL: https://github.com/apache/spark/pull/31645#issuecomment-785884417


   > Usually, we don't upgrade dependencies in released branches. But this is a correctness bug and I think we can make an exception here.
   
   @cloud-fan It's hard to add an exception here because it is imperceptible for spark, spark use `BufferedInputStream` to deal with `ZstdInputStream` return 0, it's not in spark's core but the jdk.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org