stoty commented on PR #5726: URL: https://github.com/apache/hbase/pull/5726#issuecomment-1973453219
* IMO it's always better not to have a dependency than it is to have one, when it is avoidable. * Curator libs are added to the HBase classpath. Curator libs are also present on the Hadoop Classpath, which is another opportunity for runtime classpath version conflicts. * It interacts with the Curator version from Hadoop in Maven projects * Increases the distribution size * It is not necessary for any HBase functionality Even if we ignore everything above, there is also a completely bogus curator-client dependency in hbase-endpoint, which gets added transitively to every project that depends on that. We should fix that one at the very 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org