sumitagrawl commented on code in PR #7602:
URL: https://github.com/apache/ozone/pull/7602#discussion_r1904980986
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java:
##########
@@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneManager
ozoneManager) {
*
* @return the response that will be returned to the client.
*/
- public abstract OMClientResponse validateAndUpdateCache(OzoneManager
ozoneManager, TermIndex termIndex);
+ public abstract OMClientResponse validateAndUpdateCache(OzoneManager
ozoneManager, ExecutionContext context);
/** For testing only. */
@VisibleForTesting
public OMClientResponse validateAndUpdateCache(OzoneManager ozoneManager,
long transactionLogIndex) {
- return validateAndUpdateCache(ozoneManager,
TransactionInfo.getTermIndex(transactionLogIndex));
+ ExecutionContext context = new ExecutionContext();
+ context.setTermIndex(TransactionInfo.getTermIndex(transactionLogIndex));
+ context.setIndex(transactionLogIndex);
Review Comment:
@adoroszlai
This will be different in next PR, so created having together. Only this is
to have most files impact to common place in this PR. So above can not be
followed.
index: OM managed index -- it will be different of ratis index.
termIndex: Ratis index
--
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]
