bharatviswa504 commented on pull request #2448: URL: https://github.com/apache/ozone/pull/2448#issuecomment-885456153
>but I'm not sure we need to use another variable to control if we need to delete the old locations only for buckets with >versioning disabled. If Versioning is disabled, when overriding key the PR fix is to delete older version and have only one version. (And also it has this PR fix also.) >discussed in [#2261 (comment)](https://github.com/apache/ozone/pull/2261#issuecomment-843308499), if there are 1000 >versions, it's ok to have 1000 storage and process units for all versions, the issue we have now is its redundancy causing >500500 storage and process units. I think 1000 is acceptable with the advantage that we don't even have to change the >original design too much but restored all the original locations. As I said I am +1 to to remove the redundant blocks from the latest version and have in each version only that particular block version -- 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]
