[GitHub] [ozone] GlenGeng edited a comment on pull request #1996: HDDS-4912. Support inline upgrade from containerId, delTxnId, localId to SequenceIdGenerator.

[GitBox <gi...@apache.org>]

GlenGeng edited a comment on pull request #1996:
URL: https://github.com/apache/ozone/pull/1996#issuecomment-792598922


   cc @bshashikant @mukul1987 
   I've manually did an upgrade test from latest master to last commit of this PR, the upgrade logic for delTxnId, containerId and localId works as expected.
   
   - setup a cluster of latest master, use freon to write some data, manually remove some of the keys, manually close all the containers, use ldb of the debug tool to scan the containers cf and deletedBlocks cf.
   - stop the cluster, but keep the data
   - augment the PR to let `getNextId` to print a info trace.
   - setup a cluster of this PR,  use freon to write some data, manually remove some keys, checked from trace that, delTxnId, containerId and localId are correctly upgraded, and allocated.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org