sanjeet006py commented on code in PR #2064: URL: https://github.com/apache/phoenix/pull/2064#discussion_r1931100900
########## phoenix-core-client/src/main/java/org/apache/phoenix/execute/MutationState.java: ########## @@ -1530,7 +1531,7 @@ public List<Mutation> getMutationList() { // REPLAY_ONLY_INDEX_WRITES for first batch // only in case of 1121 SQLException itrListMutation.remove(); - + mutationBatchSuccessCounter++; Review Comment: Then to know the successful mutation batches, I would have to first keep total batches initially and have to compute the diff to get batches successfully committed. This I feel is a bit indirect way to compute same thing. WDYT? -- 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: issues-unsubscr...@phoenix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org