Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/2634
@ottobackwards Thanks for mentioning about tests. I've added tests for
Funnels.
@markap14 I've removed caching `hasNonLoopConnection` because it can be
left unchanged when connections are added while a component is running.
@markobean I grouped some conditions to clearly express that a component is
treated as a 'source' component, I hope it becomes more understandable now.
Thank you all for giving your comments. I've tested adding brand-new
funnels and other components as well as loading existing components by
restarting NiFi. It looks work as expected. Hope it is ready to be merged.
---
