chesnokoff commented on code in PR #13091:
URL: https://github.com/apache/ignite/pull/13091#discussion_r3226203409


##########
modules/core/src/main/java/org/apache/ignite/internal/processors/query/running/GridRunningQueryInfo.java:
##########
@@ -33,9 +33,12 @@ public class GridRunningQueryInfo {
     /** */
     private final long id;
 
-    /** Originating Node ID. */
+    /** Node that owns query. */
     private final UUID nodeId;

Review Comment:
   Rolled back the `nodeId` & `originNodeId` split
   
   But for now map-query entries still use a locally generated running query 
id, so `SYS.SQL_QUERIES.QUERY_ID` for map rows is built as 
`<originNodeId>_<localMapQueryId>`. This means different map nodes can 
theoretically have the same `QUERY_ID` for their map rows



-- 
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]

Reply via email to