sidseth commented on issue #1332: HADOOP-16445. Allow separate custom signing 
algorithms for S3 and DDB
URL: https://github.com/apache/hadoop/pull/1332#issuecomment-532527888
 
 
   Removed the new method in S3AUtil, and introduced a SignerManager (more 
changes coming to this soon). Also incorporated one of the changes from the 
patch on HADOOP-16505 which uses the correct way to initialize signers.
   
   On the AwsConfigurationFactory - that makes sense. However, I don't think 
that should be in this patch. It's unrelated, and will end up moving more code 
from S3AUtils (including some public static methods). That's better handled in 
a separate refactoring only patch.

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