virajjasani commented on code in PR #1967:
URL: https://github.com/apache/phoenix/pull/1967#discussion_r1750976499
##########
phoenix-core-client/src/main/java/org/apache/phoenix/execute/MutationState.java:
##########
@@ -187,6 +187,9 @@ public class MutationState implements SQLCloseable {
private final boolean indexRegionObserverEnabledAllTables;
+ private ReturnResult returnResult;
Review Comment:
We can re-set it as part of `reset` but i think it is not strictly necessary
because `PhoenixStatement` sets it only for auto-commit based new API usage,
but its still good practice to reset it as part of the `MutationState` reset.
--
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]
