lokiore 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_r407216041
 
 

 ##########
 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:
   @ndimiduk Classes which we are moving are using many test classes of other 
modules, 
   we can move those classes to `src/main/java` but they have dependencies on 
many other test classes. So, to move these classes if we follow this route, how 
can we proceed without test dependency!?
   CC: @apurtell @busbey 

----------------------------------------------------------------
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

Reply via email to