rahil-c commented on code in PR #8441:
URL: https://github.com/apache/hudi/pull/8441#discussion_r1171957728
##########
hudi-utilities/pom.xml:
##########
@@ -456,6 +468,14 @@
<scope>test</scope>
</dependency>
+ <!-- AWS Services -->
+ <!--
https://mvnrepository.com/artifact/software.amazon.awssdk/aws-java-sdk-sqs -->
+ <dependency>
+ <groupId>software.amazon.awssdk</groupId>
+ <artifactId>sqs</artifactId>
+ <version>${aws.sdk.version}</version>
+ </dependency>
+
Review Comment:
It seems that the events source classes inside the hudi utilities module
will require this dependency.
https://github.com/apache/hudi/blob/master/hudi-utilities/src/main/java/org/apache/hudi/utilities/sources/S3EventsSource.java
but i see what you mean we dont have to place this dependency inside the
utiltiies pom and instead we place it in hudi aws pom
https://github.com/apache/hudi/blob/master/hudi-aws/pom.xml#L144
--
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]
