christophd commented on code in PR #1162:
URL: https://github.com/apache/camel-kamelets/pull/1162#discussion_r1032269912
##########
library/camel-kamelets-utils/pom.xml:
##########
@@ -71,12 +71,22 @@
<artifactId>camel-kafka</artifactId>
</dependency>
- <!-- AWS Dynamo DB camel component -->
+ <!-- Optional dependencies for data type conversion -->
<dependency>
<groupId>org.apache.camel</groupId>
<artifactId>camel-aws2-ddb</artifactId>
<scope>provided</scope>
</dependency>
+ <dependency>
+ <groupId>org.apache.camel</groupId>
+ <artifactId>camel-aws2-s3</artifactId>
+ <scope>provided</scope>
+ </dependency>
+ <dependency>
+ <groupId>org.apache.camel</groupId>
+ <artifactId>camel-cloudevents</artifactId>
+ <!--<scope>provided</scope>-->
Review Comment:
I had another look and the AWS S3 data type only uses some constants related
to header names in camel-cloudevents. I will use plain String header names and
remove the dependency for now.
--
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]
