GeorgeJahad commented on PR #4294: URL: https://github.com/apache/ozone/pull/4294#issuecomment-1507779661
@prashantpogde @smengcl This PR of @Xushaohong 's appears to do most of the work necessary to support the incremental bootstrapping of OmSnapshots . It only requires about another 375 lines of changes, (excluding tests), to add incremental omSnapshots: https://github.com/Xushaohong/ozone/compare/HDDS-7995...GeorgeJahad:ozone:incrementalBootstrap?w=1 Note, I've left my tests out of that branch so you can see how little needs to be added to get our functionality. My new tests are here: https://github.com/GeorgeJahad/ozone/pull/16/files I need to clean those up and and add more, but I've done enough to convince myself that my changes on top of this PR provide a solid solution for our needs. Thus my recommendation is that we use this PR as the basis for the incremental snapshots. Then I'll add mine on top of it to add our functionality along with more 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]
