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