duongkame commented on PR #5694: URL: https://github.com/apache/ozone/pull/5694#issuecomment-1840110805
There's an open point from this PR. Some S3 APIs like MultipartInfoInitiate or MultipartUploadComplete return unnecessary/redundant data like original volume, bucket and key names in their response. With the refactor to move bucket resolution to `preExecute`, we don't have the original bucket/volume name in `applyTransaction` to include in the response. Anyway, those are redundant data and are not used in S3G (only used in tests), I decided to just mark the data @Deprecated for compatibility, and removed all the relevant usage in tests. -- 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]
