You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Michael Blow (Code Review)" <do...@asterixdb.incubator.apache.org> on 2016/05/18 01:02:57 UTC

Change in asterixdb[master]: Fix Maven Warnings

Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/859

Change subject: Fix Maven Warnings
......................................................................

Fix Maven Warnings

This change eliminates the following Maven warnings in the build:

- [WARNING] 'parent.relativePath' of POM org.apache.hyracks:apache-asterixdb-hyr
  acks:0.2.18-SNAPSHOT (/home/jenkins/workspace/asterix-gerrit-notopic/hyracks-f
  ullstack/pom.xml) points at org.apache:apache-asterixdb-fullstack instead of o
  rg.apache:apache, please verify your project structure @ org.apache.hyracks:ap
  ache-asterixdb-hyracks:0.2.18-SNAPSHOT, /home/jenkins/workspace/asterix-gerrit
  -notopic/hyracks-fullstack/pom.xml, line 28, column 11

- [WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must
  be unique: org.apache.hadoop:hadoop-hdfs:jar -> duplicate declaration of versi
  on (?) @ org.apache.asterix:asterix-app:[unknown-version], /home/jenkins/works
  pace/asterix-gerrit-notopic/asterixdb/asterix-app/pom.xml, line 258, column 17

- [WARNING] 'build.plugins.plugin.(groupId:artifactId)' must be unique but found
   duplicate declaration of plugin org.apache.maven.plugins:maven-jar-plugin @ o
  rg.apache.asterix:asterix-app:[unknown-version], /home/jenkins/workspace/aster
  ix-gerrit-notopic/asterixdb/asterix-app/pom.xml, line 54, column 15

- [WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must
  be unique: org.apache.asterix:asterix-app:test-jar -> duplicate declaration of
   version 0.8.9-SNAPSHOT @ org.apache.asterix:asterix-installer:[unknown-versio
  n], /home/jenkins/workspace/asterix-gerrit-notopic/asterixdb/asterix-installer
  /pom.xml, line 319, column 17

Change-Id: Ie984b7f0ea50b9688399fafdfc48057fdf46012d
---
M asterixdb/asterix-app/pom.xml
M asterixdb/asterix-installer/pom.xml
M asterixdb/pom.xml
M hyracks-fullstack/pom.xml
4 files changed, 3 insertions(+), 26 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/59/859/1

diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml
index 8870ce2..99f1c2f 100644
--- a/asterixdb/asterix-app/pom.xml
+++ b/asterixdb/asterix-app/pom.xml
@@ -52,18 +52,6 @@
         </executions>
       </plugin>
       <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
-        <artifactId>maven-jar-plugin</artifactId>
-        <executions>
-          <execution>
-            <goals>
-              <goal>jar</goal>
-              <goal>test-jar</goal>
-            </goals>
-          </execution>
-        </executions>
-      </plugin>
-      <plugin>
         <groupId>pl.project13.maven</groupId>
         <artifactId>git-commit-id-plugin</artifactId>
         <version>2.2.0</version>
@@ -253,13 +241,7 @@
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-hdfs</artifactId>
       <type>jar</type>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.hadoop</groupId>
-      <artifactId>hadoop-hdfs</artifactId>
-      <type>jar</type>
-      <scope>compile</scope>
+      <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>xerces</groupId>
diff --git a/asterixdb/asterix-installer/pom.xml b/asterixdb/asterix-installer/pom.xml
index 1fa3eb7..da93e65 100644
--- a/asterixdb/asterix-installer/pom.xml
+++ b/asterixdb/asterix-installer/pom.xml
@@ -318,13 +318,6 @@
     </dependency>
     <dependency>
       <groupId>org.apache.asterix</groupId>
-      <artifactId>asterix-app</artifactId>
-      <version>0.8.9-SNAPSHOT</version>
-      <type>test-jar</type>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.asterix</groupId>
       <artifactId>asterix-server</artifactId>
       <version>0.8.9-SNAPSHOT</version>
       <type>zip</type>
diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index 676683e..6fec98e 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -28,6 +28,7 @@
     <groupId>org.apache</groupId>
     <artifactId>apache</artifactId>
     <version>LATEST</version>
+    <relativePath/>
   </parent>
 
   <licenses>
diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml
index 7699520..59c30c1 100644
--- a/hyracks-fullstack/pom.xml
+++ b/hyracks-fullstack/pom.xml
@@ -29,6 +29,7 @@
     <groupId>org.apache</groupId>
     <artifactId>apache</artifactId>
     <version>LATEST</version>
+    <relativePath/>
   </parent>
 
   <licenses>

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/859
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie984b7f0ea50b9688399fafdfc48057fdf46012d
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mi...@couchbase.com>

Change in asterixdb[master]: Fix Maven Warnings

Posted by "Michael Blow (Code Review)" <do...@asterixdb.incubator.apache.org>.
Michael Blow has posted comments on this change.

Change subject: Fix Maven Warnings
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1381/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/859
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie984b7f0ea50b9688399fafdfc48057fdf46012d
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-HasComments: No

Change in asterixdb[master]: Fix Maven Warnings

Posted by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org>.
Till Westmann has posted comments on this change.

Change subject: Fix Maven Warnings
......................................................................


Patch Set 1: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/859
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie984b7f0ea50b9688399fafdfc48057fdf46012d
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Change in asterixdb[master]: Fix Maven Warnings

Posted by "Michael Blow (Code Review)" <do...@asterixdb.incubator.apache.org>.
Michael Blow has submitted this change and it was merged.

Change subject: Fix Maven Warnings
......................................................................


Fix Maven Warnings

This change eliminates the following Maven warnings in the build:

- [WARNING] 'parent.relativePath' of POM org.apache.hyracks:apache-asterixdb-hyr
  acks:0.2.18-SNAPSHOT (/home/jenkins/workspace/asterix-gerrit-notopic/hyracks-f
  ullstack/pom.xml) points at org.apache:apache-asterixdb-fullstack instead of o
  rg.apache:apache, please verify your project structure @ org.apache.hyracks:ap
  ache-asterixdb-hyracks:0.2.18-SNAPSHOT, /home/jenkins/workspace/asterix-gerrit
  -notopic/hyracks-fullstack/pom.xml, line 28, column 11

- [WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must
  be unique: org.apache.hadoop:hadoop-hdfs:jar -> duplicate declaration of versi
  on (?) @ org.apache.asterix:asterix-app:[unknown-version], /home/jenkins/works
  pace/asterix-gerrit-notopic/asterixdb/asterix-app/pom.xml, line 258, column 17

- [WARNING] 'build.plugins.plugin.(groupId:artifactId)' must be unique but found
   duplicate declaration of plugin org.apache.maven.plugins:maven-jar-plugin @ o
  rg.apache.asterix:asterix-app:[unknown-version], /home/jenkins/workspace/aster
  ix-gerrit-notopic/asterixdb/asterix-app/pom.xml, line 54, column 15

- [WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must
  be unique: org.apache.asterix:asterix-app:test-jar -> duplicate declaration of
   version 0.8.9-SNAPSHOT @ org.apache.asterix:asterix-installer:[unknown-versio
  n], /home/jenkins/workspace/asterix-gerrit-notopic/asterixdb/asterix-installer
  /pom.xml, line 319, column 17

Change-Id: Ie984b7f0ea50b9688399fafdfc48057fdf46012d
Reviewed-on: https://asterix-gerrit.ics.uci.edu/859
Tested-by: Michael Blow <mi...@couchbase.com>
Reviewed-by: Till Westmann <ti...@apache.org>
---
M asterixdb/asterix-app/pom.xml
M asterixdb/asterix-installer/pom.xml
M asterixdb/pom.xml
M hyracks-fullstack/pom.xml
4 files changed, 3 insertions(+), 26 deletions(-)

Approvals:
  Michael Blow: Verified
  Till Westmann: Looks good to me, approved



diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml
index 8870ce2..99f1c2f 100644
--- a/asterixdb/asterix-app/pom.xml
+++ b/asterixdb/asterix-app/pom.xml
@@ -52,18 +52,6 @@
         </executions>
       </plugin>
       <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
-        <artifactId>maven-jar-plugin</artifactId>
-        <executions>
-          <execution>
-            <goals>
-              <goal>jar</goal>
-              <goal>test-jar</goal>
-            </goals>
-          </execution>
-        </executions>
-      </plugin>
-      <plugin>
         <groupId>pl.project13.maven</groupId>
         <artifactId>git-commit-id-plugin</artifactId>
         <version>2.2.0</version>
@@ -253,13 +241,7 @@
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-hdfs</artifactId>
       <type>jar</type>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.hadoop</groupId>
-      <artifactId>hadoop-hdfs</artifactId>
-      <type>jar</type>
-      <scope>compile</scope>
+      <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>xerces</groupId>
diff --git a/asterixdb/asterix-installer/pom.xml b/asterixdb/asterix-installer/pom.xml
index 1fa3eb7..da93e65 100644
--- a/asterixdb/asterix-installer/pom.xml
+++ b/asterixdb/asterix-installer/pom.xml
@@ -318,13 +318,6 @@
     </dependency>
     <dependency>
       <groupId>org.apache.asterix</groupId>
-      <artifactId>asterix-app</artifactId>
-      <version>0.8.9-SNAPSHOT</version>
-      <type>test-jar</type>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.asterix</groupId>
       <artifactId>asterix-server</artifactId>
       <version>0.8.9-SNAPSHOT</version>
       <type>zip</type>
diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index 676683e..6fec98e 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -28,6 +28,7 @@
     <groupId>org.apache</groupId>
     <artifactId>apache</artifactId>
     <version>LATEST</version>
+    <relativePath/>
   </parent>
 
   <licenses>
diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml
index 7699520..59c30c1 100644
--- a/hyracks-fullstack/pom.xml
+++ b/hyracks-fullstack/pom.xml
@@ -29,6 +29,7 @@
     <groupId>org.apache</groupId>
     <artifactId>apache</artifactId>
     <version>LATEST</version>
+    <relativePath/>
   </parent>
 
   <licenses>

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/859
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie984b7f0ea50b9688399fafdfc48057fdf46012d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Michael Blow <mi...@couchbase.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>