sungwy commented on code in PR #2680: URL: https://github.com/apache/polaris/pull/2680#discussion_r2444954281
########## gradle/projects.main.properties: ########## @@ -43,6 +43,8 @@ polaris-version=tools/version polaris-misc-types=tools/misc-types polaris-extensions-federation-hadoop=extensions/federation/hadoop polaris-extensions-federation-hive=extensions/federation/hive +polaris-extensions-auth-opa=extensions/auth/opa/impl +polaris-extensions-auth-opa-tests=extensions/auth/opa/tests Review Comment: I just had moved this out from the runtime module as a response to this [mailing list discussion thread here](https://lists.apache.org/thread/54qdbsxs3j7wwhv3tsccqj6qng5lqgmz). It looks like we are still working on finding a consensus on what the right approach to organize extensions would be. So I'd like to push for settling with a state that makes it easier to refactor to move into a different state without introducing breaking changes, to allow us to merge this PR. I think having it as a separate module is fine, and it'll also be relatively simple to get rid of it and move the integration tests into the runtime module in the future. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
