@dlysnichenko 

I just arrived to change the code to have a provider here (in 
`KerberosKeytabController`). Unfortunately we have hundreds of references to 
`KerberosHelper` in our code with simple `@Inject` which should be changed to a 
provider since we can not use both (you either bind the interface to a class or 
you bind it to a provider; you cannot have both). In this case this would mean 
hundreds of changes; with so many modifications we would lose the main point of 
this PR.
So that I leave my patch as is...

@rlevas 
Any objection?

[ Full content available at: https://github.com/apache/ambari/pull/2359 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to