flyrain commented on code in PR #2680:
URL: https://github.com/apache/polaris/pull/2680#discussion_r2453092210
##########
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:
Looking into the pipeline, I noticed that the OPA integration tests are
still being executed within the `runtime-service` module, specifically via this
command:
```
./gradlew \
intTest \
-x :polaris-runtime-service:intTest
```
This behavior stems from the change in
[`[runtime/server/build.gradle.kts](https://github.com/apache/polaris/pull/2680/files#diff-9895ab333561c6f07093e25cd9915f15caa3901efe46cdf0dffdf4c8b34cb5ad)`](https://github.com/apache/polaris/pull/2680/files#diff-9895ab333561c6f07093e25cd9915f15caa3901efe46cdf0dffdf4c8b34cb5ad).
I’m not particularly a fan of this setup, but I’m OK with it, and I haven’t
observed any performance issues.
For reference:
With OPA ~12 mins,
https://github.com/apache/polaris/actions/runs/18717788753/job/53381604223?pr=2680
Without OPA ~11mins,
https://github.com/apache/polaris/actions/runs/18701076539/job/53329779421?pr=2852
--
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]
