swamirishi opened a new pull request, #5223: URL: https://github.com/apache/ozone/pull/5223
## What changes were proposed in this pull request? SnapshotCreateRequest creates snapshotInfo created time in pre-execute phase whose order need not be the same as the ratis log ordering. So 2 consecutive snapshot created in the snapshot info table need not be in increasing order of created time. SnapshotChainManager on om restart tries to load the snapshot in the increasing order of their created time. Thus throws a SnapshotChainCorruption exception in this case as it previous snapshot is not loaded but SnapshotChainManager expects that the previous snapshot be loaded before trying to load the current snapshot. Thus instead of ordering snapshot on their created time. The changes proposes to create a forward linked list for the snapshot chain based on the previousSnapshotGlobalId from the snapshotInfo and iterates through the linked list in the same order. Also added a few debug logs for the same. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-9215 ## How was this patch tested? Existing Unit tests & adding more Unit 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]
