Re: [PR] IGNITE-22303 Retry operation when plan gets outdated by the time of execution [ignite-3]

["Copilot (via GitHub)" <gi...@apache.org>]

Copilot commented on code in PR #7309:
URL: https://github.com/apache/ignite-3/pull/7309#discussion_r2661992736


##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlOperationContext.java:
##########
@@ -80,12 +83,36 @@ private SqlOperationContext(
         this.errorListener = errorListener;
         this.userName = userName;
         this.topologyVersion = topologyVersion;
+        this.retryTxHolder = new AtomicReference<>(retryTx);
     }
 
     public static Builder builder() {
         return new Builder();
     }
 
+    /**
+     * Copies an existing context preserving the existing transaction.
+     *
+     * <p>Used in case of a retry. If the operation is repeated while preserving the running transaction,
+     *    the operation time is taken from this transaction.

Review Comment:
   The documentation comment has incorrect indentation. Line 96 should be aligned with line 95 to maintain consistent formatting in the javadoc comment.
   ```suggestion
        * the operation time is taken from this transaction.
   ```



-- 
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: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org