srowen commented on a change in pull request #25932: 
[SPARK-29250][BUILD][test-hadoop3.2][test-java11][test-maven] Upgrade to Hadoop 
3.2.1
URL: https://github.com/apache/spark/pull/25932#discussion_r328585374
 
 

 ##########
 File path: dev/deps/spark-deps-hadoop-3.2
 ##########
 @@ -22,6 +22,8 @@ automaton-1.11-8.jar
 avro-1.8.2.jar
 avro-ipc-1.8.2.jar
 avro-mapred-1.8.2-hadoop2.jar
+bcpkix-jdk15on-1.60.jar
+bcprov-jdk15on-1.60.jar
 
 Review comment:
   For the record, the process for the last part is 
https://www.apache.org/dev/crypto.html#sources
   I can do it afterwards, it's not hard.
   
   However, hm, I wonder if Hadoop needs a similar disclosure at 
http://www.apache.org/licenses/exports/ ? It's possible that somehow it isn't 
distributed directly by Hadoop, but would be surprised if it's a first-class 
dep and it makes binary releases.
   
   Maybe, eh, CC @steveloughran in case he knows anything about this angle.

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


With regards,
Apache Git Services

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

Reply via email to