colinmjj commented on PR #53: URL: https://github.com/apache/incubator-uniffle/pull/53#issuecomment-1211510881
@zuston Is it possible to split the PR according to following function: 1. register remote storage to shuffle server with user info and stores the mapping about `appId -> user` 2. Add an api in Storage to check if it is secured, eg, `isSecuredStorage()` 3. Create new writeHandler/readHandler to deal with secured write/read, eg, `SecuredHdfsShuffleWriteHandler extends HdfsShuffleWriteHandler` 4. For Shuffle Server, there has an executor service to deal with kerberos login if secured is enabled. Just for your reference to split the PR and will be easy to review. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
