mchades commented on code in PR #8050:
URL: https://github.com/apache/gravitino/pull/8050#discussion_r2293550597


##########
bundles/aws/build.gradle.kts:
##########
@@ -40,15 +43,12 @@ dependencies {
     exclude("*")
   }
 
-  implementation(libs.aws.iam)
-  implementation(libs.aws.policy)

Review Comment:
   > Since lacking so many dependencies, how can we assure that it's easy for 
users to use the jar. 
   
   It is because we assume that the user environment already has access to 
cloud storage, so these users find it easy to use. If not, they should use 
`gravitino-aws-bundle.jar`.
   
   > is it reasonable to exclude dependencies like libs.aws.iam, 
libs.aws.policy?
   
   `libs.aws.iam` and `libs.aws.policy` are also a part of aws dependency, so I 
think it's reasonable to exclude them from `gravitino-aws.jar`
   



-- 
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]

Reply via email to