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-533187731
 
 
   The change the latest commit (SignerManager) removes the code from S3AUtils 
and moves it to a SignerManager class (non-static). The actual signer 
registration has picked up bits from the patch on HADOOP-16505 (duplicate of 
this jira), since that was doing the registration in a better way (once per 
unique signer per JVM, rather than each time a FS instances is created). 
   
   I do plan to make additional changes to this class in a follow up patch, but 
it's essentially the same as the old patch, with a non-static Closeable class, 
without adding an extra static method to S3AUtils. Please let me know if 
there's specific concerns around this (i.e. change back to what?)

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