dimas-b commented on code in PR #2680: URL: https://github.com/apache/polaris/pull/2680#discussion_r2453377643
########## 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: After re-reading the current code, I see that a lot of the integration tests are currently in the `runtime/service` module. With that in mind, plus CI time concerns, I'd be ok with keeping OPA ITs in `runtime/service` in this PR as long as it introduces only "test" dependencies to the `runtime/service` module. We can discuss moving IT tests into `runtime/server` later. -- 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]
