That would also work (as described [here](https://github.com/google/guice/wiki/CyclicDependencies#break-the-cycle-with-a-provider)); but - IMO - Guice providers are just hiding cyclic dependencies which is the real issue here. In this case it's not better or worse than the solution I applied (we have plenty of these in the code anyway); one advantage would be is to not add another static injection.
Long story short: I'll submit a new patch in 5-10 mins. [ Full content available at: https://github.com/apache/ambari/pull/2359 ] This message was relayed via gitbox.apache.org for [email protected]
