[ 
https://issues.apache.org/jira/browse/HADOOP-18708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17744992#comment-17744992
 ] 

ASF GitHub Bot commented on HADOOP-18708:
-----------------------------------------

steveloughran commented on PR #5767:
URL: https://github.com/apache/hadoop/pull/5767#issuecomment-1643587230

   I'm thinking this move is time to revisit my refactoring plans and actually 
split s3afs into two layers, filesystem and store, as abfs does. we'd have all 
those callback interfaces done in the store class, which would also contain the 
threadpool, transfer manager etc -and all those `inner*()` ops in s3a fs would 
move down with the audit context passed in as an explicit param.
   
   lets make sure this pr is ready for it by using an interface for callbacks 
to create s3 client, ideally even invoke the region call. that way we can hide 
these classes from the details of the SDK




> AWS SDK V2 - Implement CSE
> --------------------------
>
>                 Key: HADOOP-18708
>                 URL: https://issues.apache.org/jira/browse/HADOOP-18708
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 3.4.0
>            Reporter: Ahmar Suhail
>            Priority: Major
>              Labels: pull-request-available
>
> S3 Encryption client for SDK V2 is now available, so add client side 
> encryption back in. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to