stoty commented on PR #6584: URL: https://github.com/apache/hbase/pull/6584#issuecomment-2661557452
This makes my head hurt, but let's try and enumerate the use cases: - Production HBase install : Includes dependency managed ZK 3.8.4 + dependency managed Curator. - Production BYO Hadoop install: Includes dependency managed ZK 3.8.4 + possibly ancient Curator from Hadoop - Developer classpath: Includes dependency managed ZK 3.8.4 + dependency managed Curator 4.2.0 - hbase-shaded-client: Includes relocated dependency managed ZK 3.8.4 + relocated dependency managed Curator 4.2.0 - hbase-shaded-client-byo: Includes relocated dependency managed ZK 3.8.4 , plus whatever ZK+curator Hadoop brings. Which means that * we either have unrelocated ZK 3.8.4 (and no Curator from HBase) on the classpath, which works with Curator 5, * or HBase uses relocated ZK 3.8.4, and relocated curator libraries, and so does Hadoop. (hbase-shaded-client) * or HBase uses the relocated ZK 3.8.4, but Hadoop uses its own ZK and curator. (hbase-shaded-client-byo-hadoop) Based on this, Hadoop either uses a completely different set of ZK and Curator libraries, or its ZK version is replaced by ours, so upgrading Curator should be OK. -- 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]
