adoroszlai commented on PR #5776: URL: https://github.com/apache/ozone/pull/5776#issuecomment-1858355012
> What about CapableOzoneFSInputStream and CapableOzoneFSInputStream? IIRC they were also added to workaround the lack of StreamCapabilities in Hadoop 2. Yes, they implement `StreamCapabilities` as subclasses, separate from the base `OzoneFS*Stream` classes due to compatibility. But their code is not copied from Hadoop, so we need to put it somewhere. I don't see much benefit in merging them into their base classes. -- 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]
