- 
                Notifications
    
You must be signed in to change notification settings  - Fork 9.1k
 
HADOOP-18742. AWS v2 SDK: stabilise dependencies with rest of hadoop libraries #5739
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
See aws_sdk_v2_changelog.md for details. Co-authored-by: Ahmar Suhail <[email protected]> Co-authored-by: Alessandro Passaro <[email protected]>
addresses review comments + yetus errors Co-authored-by: Ahmar Suhail <[email protected]>
…pache#5421) Changes include * use bundled transfer manager * adds transfer listener to upload * adds support for custom signers * don't set default endpoint * removes v1 sdk bundle, only use core package * implements region caching + many more Note: spotbugs is warning about inconsistent synchronization in accessing a new s3a FS field. This will be fixed in a follow-up patch. Contributed by Ahmar Suhail
…libraries * explicit jackson declaration in hadoop-aws * cut jackson, eventstream and ion from aws SDK dependencies Change-Id: I47f6e10d42c8067df8255eca69799469d7252480
| 
           not tested this yet; this is a cherrypick of my existing branch onto the rebased sdk branch  | 
    
| 
           💔 -1 overall 
 
 This message was automatically generated.  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not tested this yet
if you test it and things work then this LGTM
75220b7    to
    acb31b6      
    Compare
  
    | 
           pulling in to #5872 which downgrades the core lib to provided; we gain consistency by excluding unshaded stuff  | 
    
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?