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 2019/04/26 23:06:49 UTC

[GitHub] [spark] vanzin opened a new pull request #24476: [SPARK-25535][CORE][branch-2.4] Work around bad error handling in commons-crypto.

vanzin opened a new pull request #24476: [SPARK-25535][CORE][branch-2.4] Work around bad error handling in commons-crypto.
URL: https://github.com/apache/spark/pull/24476
 
 
   The commons-crypto library does some questionable error handling internally,
   which can lead to JVM crashes if some call into native code fails and cleans
   up state it should not.
   
   While the library is not fixed, this change adds some workarounds in Spark code
   so that when an error is detected in the commons-crypto side, Spark avoids
   calling into the library further.
   
   Tested with existing and added unit tests.

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


With regards,
Apache Git Services

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