markobean commented on code in PR #11582:
URL: https://github.com/apache/nifi/pull/11582#discussion_r4011761214


##########
nifi-frontend/src/main/frontend/apps/nifi/src/app/pages/flow-designer/state/flow/flow.selectors.ts:
##########
@@ -294,3 +295,35 @@ export const selectOverlappingConnections = createSelector(
         return detectOverlappingConnections(connections, processGroupId);
     }
 );
+
+// maps the id of every group whose name is known in the current flow to that 
group's name: the current
+// process group, its ancestors via the breadcrumb chain, and the child 
process groups and remote process
+// groups it contains. Remote process groups are included alongside process 
groups because a connection to
+// one reports the remote process group's own id as the source/destination 
group id of the remote port it
+// terminates at, so the two kinds of id are looked up the same way.
+export const selectProcessGroupIdToNameMap = createSelector(

Review Comment:
   The map to process groups was refactored to allow proper handling of sibling 
process groups which also facilitates linking ports in on PG directly to ports 
in sibling PG.



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