flyrain commented on code in PR #2680:
URL: https://github.com/apache/polaris/pull/2680#discussion_r2422241130
##########
polaris-core/build.gradle.kts:
##########
@@ -24,6 +24,7 @@ plugins {
dependencies {
implementation(project(":polaris-api-management-model"))
+ implementation(libs.apache.httpclient5)
Review Comment:
> Do we still need this dep. in core?
No. I think it’s perfectly fine to keep the OPA authorizer within the
`polaris-runtime-service` module rather than moving it to a separate extension
module.
The rationale for separating components like `hive` or `hadoop` federation
is that they introduce heavy and intrusive `Hadoop` dependencies that we don’t
want to pull into the service module.
The OPA authorizer, on the other hand, adds no additional dependencies, it
integrates cleanly within the service module, so there’s no downside to keeping
it there.
--
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]
