lhotari commented on code in PR #24946:
URL: https://github.com/apache/pulsar/pull/24946#discussion_r2494174239
##########
pulsar-io/hbase/pom.xml:
##########
@@ -77,6 +77,10 @@
<groupId>org.slf4j</groupId>
<artifactId>slf4j-log4j12</artifactId>
</exclusion>
+ <exclusion>
+ <groupId>org.apache.avro</groupId>
+ <artifactId>avro</artifactId>
+ </exclusion>
Review Comment:
That would work. However this current PR is problematic since it combines
changes for 2 separate connectors in a single PR. It would be better to close
this current PR and open a separate PR to upgrade hadoop3.version alone (since
it touches also tiered storage component), a separate PR for hbase.version +
the Avro exclusion and a separate PR to address the Flume connector. The reason
to keep things separately is that it's easier to make decisions whether to
include a change in maintenance versions or not.
--
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]
