sumitagrawl opened a new pull request, #5811:
URL: https://github.com/apache/ozone/pull/5811
## What changes were proposed in this pull request?
- take snapshot should not over-write the db transaction with
getLastAppliedIndex() which might be old
As this was causing replay of transaction if db transaction is
over-written on startup.
This can get trigger from
1. db snapshot -- need wait for double buffer flush to complete on-going
update
2. upgrade prepare -- there is already a check with wait for update prepare
index, so no need update back with LastAppliedTransactionId. Further code is
simplified.
- simplify updateLastAppliedIndex() call updated only using double buffer
flush. This update is required for Leader state isReady() by ratis as present
in comment.
a. notifyTermIndexUpdated() should be updated using double buffer so that
term index always in sync with db transaction info
b. doubleBuffer flush simplify update the transaction index as present in
applyTransactionMap or ratisTransactionMap
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9342
## How was this patch tested?
- existing test cases should cover impact
--
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]
