sahilTakiar opened a new pull request #970: HADOOP-16371: Option to disable GCM 
for SSL connections when running on Java 8
URL: https://github.com/apache/hadoop/pull/970
 
 
   [HADOOP-16371: Option to disable GCM for SSL connections when running on 
Java 8](https://issues.apache.org/jira/browse/HADOOP-16371)
   
   Changes:
   * Patch is based on the merged patch from HADOOP-16050
   * Decided a better name for `SSLSocketFactoryEx` would be 
`DelegatingSSLSocketFactory` because the class is not OpenSSL specific (e.g. it 
is capable of just delegating to the JSSE)
   * Add a bunch of code comments to `DelegatingSSLSocketFactory`
   * Documented `fs.s3a.ssl.channel.mode` in `performance.md` and 
`core-default.xml`
   
   Testing Done:
   * Ran all S3 tests `mvn verify` and S3 scale tests `mvn verify 
-Dparallel-tests -Dscale -DtestsThreadCount=16` (did not have S3Guard or kms 
tests setup)
   * Ran `TestDelegatingSSLSocketFactory` on Ubuntu and OSX with `-Pnative` and 
confirmed the test passes on both systems (on OSX it is skipped, on Ubuntu it 
actually runs)
   * Ran the ABFS tests against "East US 2" and the only failure was 
`ITestGetNameSpaceEnabled.testNonXNSAccount` (known issue)
   * Ran `mvn package -Pdist -DskipTests -Dmaven.javadoc.skip=true 
-DskipShade`, un-tarred `hadoop-dist/target/hadoop-3.3.0-SNAPSHOT.tar.gz` and 
ran `./bin/hadoop fs -ls s3a://[my0bucket-name]/` successfully and that I could 
upload and read a file to S3 using the CLI without the wildlfy jar on the 
classpath

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