metron-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [metron] mmiklavc commented on a change in pull request #1551: METRON-2223 Reconcile Versions in Root POM with HDP 3.1
Date Fri, 08 Nov 2019 19:15:24 GMT
mmiklavc commented on a change in pull request #1551: METRON-2223 Reconcile Versions in Root
POM with HDP 3.1
URL: https://github.com/apache/metron/pull/1551#discussion_r344326846
 
 

 ##########
 File path: pom.xml
 ##########
 @@ -185,11 +167,6 @@
                 <artifactId>mockito-core</artifactId>
                 <version>${global_mockito_version}</version>
             </dependency>
-            <dependency>
-                <groupId>org.apache.storm</groupId>
-                <artifactId>storm-kafka</artifactId>
 
 Review comment:
   @justinleet @nickwallen - nope, looks like that was me - https://github.com/mmiklavc/metron/blame/7f1160401897c49738dcb79609661419d39e112d/pom.xml#L116.
The way the dependency management section works is to lay down versions and other details
about a dependency so you can include it in your child projects without having to explicitly
specify the version. Ideally, this would be the way we declare all shared dependencies in
Metron. Submodules are free to override the default if/when they choose by explicitly declaring
the version in their pom dep.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message