nickwallen commented on a change in pull request #1506: METRON-2188 Upgrade to 
HBase 2.0.2
URL: https://github.com/apache/metron/pull/1506#discussion_r323242629
 
 

 ##########
 File path: metron-platform/metron-hbase/metron-hbase-common/pom.xml
 ##########
 @@ -52,6 +52,10 @@
                     <groupId>org.slf4j</groupId>
                     <artifactId>slf4j-log4j12</artifactId>
                 </exclusion>
+                <exclusion>
 
 Review comment:
   I wouldn't say the process has changed that much; dependencies still leak, 
which needs to be hunted down.  The recent maven/pom changes just make it 
easier to see where the leaks are coming from.  At least that's my take after 
working through this.
   
   I can add some commentary on the various pom changes highlighting the issue 
that I encountered, if that helps.  Good thing I took good notes.

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