Hemant Kumar
Hemant Kumar
> Maybe the updated PR template is a bit confusing, but "Title" is supposed to be entered only in the title field, not duplicated in the description field. > >...
@errose28 can you please review the latest pull? @xichen01 Can you please look into the workflow failures?
HDDS-10295. Provide an "ozone repair" subcommand to update the snapshot info in transactionInfoTable
@DaveTeng0 TestTransactionInfoRepair tests are failing due to NPE. Can you please fix that?
HDDS-10295. Provide an "ozone repair" subcommand to update the snapshot info in transactionInfoTable
@errose28 can you please take a look at the final PR?
HDDS-10295. Provide an "ozone repair" subcommand to update the snapshot info in transactionInfoTable
Thanks, @DaveTeng0 for the change and @errose28 for the review.
> how does this behave in case of new server & old client or vice-versa? [Protobuf change](https://github.com/apache/ozone/pull/6542/files#diff-31e34e282f9c79c316ed42775ccba5aba01cf10771bf8954e72ccc1e87d04a62) is backward and forward compatible in itself. I think the only change which...
> how does this behave in case of new server & old client or viceversa? @ayushtkn do you want to give it another look?
Thanks for the review @sumitagrawl. I've addressed or resolved most of the comments. Let me know if there is any compatibility break change otherwise, it should be good to merge.
Thanks @swamirishi @sumitagrawl and @ayushtkn for the review.
Thanks, @DaveTeng0 for the change and @errose28 for the review.