HeartSaVioR commented on pull request #2466: URL: https://github.com/apache/hadoop/pull/2466#issuecomment-728411313
Btw, while we can't set max date on AbstractS3ATokenIdentifier, there looks to be the chance for SessionTokenIdentifier to set max date as same as expiry time if marshalledCredentials is provided. Would it make sense? I also have the fix for this, though I'm not 100% sure my theory is correct so minimized the fix for HADOOP-17379. https://github.com/HeartSaVioR/hadoop/commit/b7e1d170ce3f8a50fa5953a53a73b1b08d3b54d5 If the fix makes sense, either updating the PR or having a separate JIRA issue is fine for me. Please guide which is preferred for you. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
