Till Westmann has posted comments on this change. Change subject: Cleanup AsterixDB Dependencies ......................................................................
Patch Set 12: (8 comments) https://asterix-gerrit.ics.uci.edu/#/c/1421/12/asterixdb/asterix-algebra/pom.xml File asterixdb/asterix-algebra/pom.xml: Line 181: <groupId>org.apache.hyracks</groupId> Ok, I feel bad as I write this, but if we ordered grouped them by Hyracks and AsterixDB ... https://asterix-gerrit.ics.uci.edu/#/c/1421/12/asterixdb/asterix-app/pom.xml File asterixdb/asterix-app/pom.xml: Line 333: <version>1.2.17</version> Do we only have log4j in this module? https://asterix-gerrit.ics.uci.edu/#/c/1421/12/asterixdb/asterix-installer/pom.xml File asterixdb/asterix-installer/pom.xml: Line 484: <version>1.2.15</version> another log4j https://asterix-gerrit.ics.uci.edu/#/c/1421/12/asterixdb/asterix-lang-sqlpp/pom.xml File asterixdb/asterix-lang-sqlpp/pom.xml: Line 140: <version>1.2.17</version> and another log4j https://asterix-gerrit.ics.uci.edu/#/c/1421/12/asterixdb/asterix-metadata/pom.xml File asterixdb/asterix-metadata/pom.xml: Line 89: <version>0.2.18-SNAPSHOT</version> Why do we have explicit hyracks versions here? Line 104: <version>0.8.9-SNAPSHOT</version> ... and AsterixDB versions? https://asterix-gerrit.ics.uci.edu/#/c/1421/12/asterixdb/asterix-yarn/pom.xml File asterixdb/asterix-yarn/pom.xml: Line 417: <version>1.2.9</version> log4j Line 427: <version>2.2.0</version> Is this the same hadoop-version we use in the root pom? -- To view, visit https://asterix-gerrit.ics.uci.edu/1421 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3cc9d4db57b2138931baa3ef51ea569c45b79a2f Gerrit-PatchSet: 12 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-HasComments: Yes
