lhotari commented on code in PR #24946:
URL: https://github.com/apache/pulsar/pull/24946#discussion_r2491265483
##########
pulsar-io/flume/pom.xml:
##########
@@ -61,6 +61,18 @@
<artifactId>avro</artifactId>
<groupId>org.apache.avro</groupId>
</exclusion>
+ <exclusion>
+ <groupId>org.apache.tomcat.embed</groupId>
+ <artifactId>tomcat-embed-core</artifactId>
+ </exclusion>
+ <exclusion>
+ <groupId>org.apache.mina</groupId>
+ <artifactId>mina-core</artifactId>
Review Comment:
Removing mina-core would most likely break the Flume connector. A better
approach might be to remove the connector completely since we don't have
integration tests to validate that it would really work when using a real
network connection.
The current Flume connector tests use org.apache.flume.channel.MemoryChannel
and there are not end-to-end integration tests that would use a real Flume
instance running in a container.
--
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]
