Dzeri96 commented on code in PR #3599:
URL: https://github.com/apache/celeborn/pull/3599#discussion_r2811666173


##########
project/CelebornBuild.scala:
##########
@@ -132,6 +132,8 @@ object Dependencies {
   val hadoopAws = "org.apache.hadoop" % "hadoop-aws" % hadoopVersion 
excludeAll (
     ExclusionRule("com.amazonaws", "aws-java-sdk-bundle"))
   val awsS3 = "com.amazonaws" % "aws-java-sdk-s3" % awsS3Version
+  // Needed for com.amazonaws.auth.WebIdentityTokenCredentialsProvider
+  val awsSTS = "com.amazonaws" % "aws-java-sdk-sts" % awsS3Version

Review Comment:
   I added my changes. The NOTICE in the Amazon SDK had more info than what we 
had previously, so I added the missing content. That's how I interpreted 
Apache's guidelines at least.



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