ndimiduk commented on code in PR #5680:
URL: https://github.com/apache/hbase/pull/5680#discussion_r2100115341


##########
hbase-http/pom.xml:
##########
@@ -458,6 +398,19 @@
               </execution>
             </executions>
           </plugin>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-compiler-plugin</artifactId>
+            <configuration>
+              <!--
+                Below tests fail with hadoop 2 due to test dependency 
versioning issues.  Hence, skip LDAP tests for Hadoop 2,

Review Comment:
   This approach is clever but quite unexpected for the average dev and thus a 
bit scary to me.



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

Reply via email to