Github user pepov commented on the issue:
https://github.com/apache/nifi/pull/2970
Nitpicking but now we do 2 lookups. First we look up the group to get the
identifier then we lookup the same group using that identifier. I beleive it
would be enough to store the group name property and look it up in the
populateNodes method only, but this is nitpicking, since there is no
performance or memory penalty in it, so overall this looks good to me.---
