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


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

Review Comment:
   please see the updated doc for `gravitino-aws.jar` and 
`gravitino-aws-bundle.jar` in this PR:
   
   > - 
[`gravitino-aws-bundle-${gravitino-version}.jar`](https://mvnrepository.com/artifact/org.apache.gravitino/gravitino-aws-bundle)
 is a bundle that 
   >   contains all the dependencies needed to access S3. It is recommended to 
use this JAR when your environment does not include cloud storage dependencies. 
   >   You can use this bundle to access cloud storage through Gravitino 
directly without downloading extra dependencies.
   > - 
[`gravitino-aws-${gravitino-version}.jar`](https://mvnrepository.com/artifact/org.apache.gravitino/gravitino-aws)
 is a condensed JAR. 
   >   It is recommended to use this JAR when your environment can already 
access cloud storage. It only contains Gravitino's own code and shaded basic 
dependencies,
   >   which can ensure that there will be no conflicts with the dependencies 
of cloud storage.



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