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 and allocating 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 to learn about the 
largest contaienrId and delTxnId..
   - stop the cluster, but keep the data
   - augment the PR to let `getNextId` to print a info trace `LOG.info("next id 
for {} is {}", sequenceIdName, batch.nextId);`.
   - 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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to