dimas-b commented on code in PR #2680: URL: https://github.com/apache/polaris/pull/2680#discussion_r2453086104
########## 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: My suggestion was to run integration tests inside the OPA modules (by including runtime-service and making a dedicated Quarkus server) to improve code isolation. If that adds too much CI time, I'm ok with including OPA as a test-only dependency into runtime-service. Even better, why not create a dedicated integration tests module? This will avoid any semi-related deps in runtime-service and still allow for one Quarkus server build for running all integration tests? -- 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]
