Russole commented on PR #9370: URL: https://github.com/apache/ozone/pull/9370#issuecomment-3583488875
Thanks for the feedback, @Gargi-jais11. I checked the current construction paths and Ozone only applies one wrapper layer, e.g.: new CryptoOutputStream(keyOutputStream, ...) Nested wrapping never occurs today, and the original code in both getKeyOutputStream() and getMetadata() also assumes a single layer. Since this PR is just a refactor to remove duplicate instanceof checks, I kept the existing behavior unchanged. If nested wrapping is introduced in the future, unwrap() can be extended easily. -- 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]
