ndimiduk commented on a change in pull request #1201: HBASE-23639 : Moving
classes out of hbase-it /test for direct API use of chaos.
URL: https://github.com/apache/hbase/pull/1201#discussion_r408990532
##########
File path: hbase-it/pom.xml
##########
@@ -154,10 +154,29 @@
<type>test-jar</type>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.apache.hbase</groupId>
+ <artifactId>hbase-server</artifactId>
+ <type>test-jar</type>
Review comment:
This dependency graph goes deep, and makes this a non-trivial challenge to
unravel. I agree we should be shipping a solid suite of tools for testing HBase
and HBase applications. Sadly changing only the leaves of that graph is not
sufficient. IMHO, you have to start at the root classes and work out to the
leaves. I don't know if that effort is worth it when compared to building a new
testing harness from scratch that's designed from the outset to be user-facing.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
