steveloughran commented on pull request #2675: URL: https://github.com/apache/hadoop/pull/2675#issuecomment-780659540
Latest PR moves the prepareRequest() callback of the active span into the AWS SDK. HEAD^ does it in the request factory construction -that's still supported, but if we can keep the AWS SDK wiring -that will be better * guarantee of being invoked on all S3 operations, even those done in the client during startup * means we get audit/log of calls like doesBucketExistV2 without us reimplementing them * means we can simplify the work I've just done regarding marking operations as OnceRaw and walking up/down the callstack But: this does mean we need to solve the problem of endpoints and regions ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
