sodonnel commented on pull request #3151: URL: https://github.com/apache/ozone/pull/3151#issuecomment-1057168053
Lets see what @umamaheswararao thinks of the changes here. I am not against improvements, but we need to get the feature stabilised and refactoring things may not be helping with that. We could also do with improving unit / integration test coverage and scenarios. > we are going to implement cell rotation feature We are not planning to do that for now - the design needs updated. It just adds more complexity and in real world use, its hard to see it it will give us much, unless we are storing only tiny keys, where EC is not the correct use case. -- 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]
