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

Reply via email to