korlov42 commented on a change in pull request #9369:
URL: https://github.com/apache/ignite/pull/9369#discussion_r702909999
##########
File path:
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/exec/ExecutionContext.java
##########
@@ -183,20 +201,46 @@ public MvccSnapshot mvccSnapshot() {
}
/**
- * @return Originating node ID.
+ * @return Local node ID.
+ */
+ public UUID localNodeId() {
+ return locNodeId;
+ }
+
+ /**
+ * @return Originating node ID (the node, who started the execution).
*/
public UUID originatingNodeId() {
- return planningContext().originatingNodeId();
+ return originatingNodeId == null ? locNodeId : originatingNodeId;
Review comment:
I would prefer to have `originatingNodeId` set explicitly
--
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]
