Thanks for the review Sravya Tirukkovalur. I agree 100%. There shouldn't be any Solr, Kafka, Sqoop classes in the PrivilegeOperatorPersistence. I left the existing classes in that map as-is and did not enforce them to be configured via sentry-site.xml primarily to avoid having to fix existing tests in this PR. If we're ok with this strategy, I could create a follow up jira/patch to remove those as well, so at least action factories are not tightly coupled. The eventual goal should be that the sentry-provider-db module should not depend on sentry-core-model-search and sentry-core-model-sqoop (and in future sentry-core-model-kafka.
I'm not familiar with the other hooks that you are talking about. Could you please elaborate?