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 2018/03/14 02:21:03 UTC

Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...

Michael Blow has uploaded a new change for review.

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

Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/
......................................................................

Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/

- makes fuzzyjoin functions available to BAD tests

Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
---
M asterix-bad/pom.xml
1 file changed, 7 insertions(+), 9 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/81/2481/1

diff --git a/asterix-bad/pom.xml b/asterix-bad/pom.xml
index 992f844..2a8b7e3 100644
--- a/asterix-bad/pom.xml
+++ b/asterix-bad/pom.xml
@@ -154,6 +154,7 @@
           <configuration>
             <usedDependencies>
               <usedDependency>org.apache.hadoop:hadoop-minicluster</usedDependency>
+              <usedDependency>org.apache.asterix:asterix-fuzzyjoin</usedDependency>
             </usedDependencies>
             <ignoredUnusedDeclaredDependencies>
               <ignoredUnusedDeclaredDependency>org.apache.asterix:asterix-common</ignoredUnusedDeclaredDependency>
@@ -233,12 +234,10 @@
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>algebricks-common</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>hyracks-dataflow-common</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.asterix</groupId>
@@ -253,32 +252,26 @@
     <dependency>
       <groupId>log4j</groupId>
       <artifactId>log4j</artifactId>
-      <version>1.2.17</version>
     </dependency>
     <dependency>
       <groupId>org.apache.commons</groupId>
       <artifactId>commons-lang3</artifactId>
-      <version>3.5</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>algebricks-core</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>junit</groupId>
       <artifactId>junit</artifactId>
-      <version>4.12</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>hyracks-data-std</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>algebricks-runtime</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.asterix</groupId>
@@ -291,9 +284,14 @@
       <version>${asterix.version}</version>
     </dependency>
     <dependency>
+      <groupId>org.apache.asterix</groupId>
+      <artifactId>asterix-fuzzyjoin</artifactId>
+      <version>${asterix.version}</version>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>hyracks-api</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
   </dependencies>
 </project>

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>

Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...

Posted by "Michael Blow (Code Review)" <do...@asterixdb.incubator.apache.org>.
Hello Jenkins,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#2).

Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/
......................................................................

Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/

- makes fuzzyjoin functions available to BAD tests

Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
---
M asterix-bad/pom.xml
1 file changed, 7 insertions(+), 9 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad refs/changes/81/2481/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2481
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Xikui Wang <xk...@gmail.com>
Gerrit-Reviewer: abdullah alamoudi <ba...@gmail.com>

Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...

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

Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/
......................................................................


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/256/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Xikui Wang <xk...@gmail.com>
Gerrit-Reviewer: abdullah alamoudi <ba...@gmail.com>
Gerrit-HasComments: No

Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...

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

Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/
......................................................................


Patch Set 2: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Xikui Wang <xk...@gmail.com>
Gerrit-Reviewer: abdullah alamoudi <ba...@gmail.com>
Gerrit-HasComments: No

Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...

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

Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/
......................................................................


Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/

- makes fuzzyjoin functions available to BAD tests

Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
---
M asterix-bad/pom.xml
1 file changed, 7 insertions(+), 9 deletions(-)

Approvals:
  abdullah alamoudi: Looks good to me, approved
  Jenkins: Verified



diff --git a/asterix-bad/pom.xml b/asterix-bad/pom.xml
index 992f844..6f5cd32 100644
--- a/asterix-bad/pom.xml
+++ b/asterix-bad/pom.xml
@@ -154,6 +154,7 @@
           <configuration>
             <usedDependencies>
               <usedDependency>org.apache.hadoop:hadoop-minicluster</usedDependency>
+              <usedDependency>org.apache.asterix:asterix-fuzzyjoin</usedDependency>
             </usedDependencies>
             <ignoredUnusedDeclaredDependencies>
               <ignoredUnusedDeclaredDependency>org.apache.asterix:asterix-common</ignoredUnusedDeclaredDependency>
@@ -233,12 +234,10 @@
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>algebricks-common</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>hyracks-dataflow-common</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.asterix</groupId>
@@ -253,32 +252,26 @@
     <dependency>
       <groupId>log4j</groupId>
       <artifactId>log4j</artifactId>
-      <version>1.2.17</version>
     </dependency>
     <dependency>
       <groupId>org.apache.commons</groupId>
       <artifactId>commons-lang3</artifactId>
-      <version>3.5</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>algebricks-core</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>junit</groupId>
       <artifactId>junit</artifactId>
-      <version>4.12</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>hyracks-data-std</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>algebricks-runtime</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.asterix</groupId>
@@ -291,9 +284,14 @@
       <version>${asterix.version}</version>
     </dependency>
     <dependency>
+      <groupId>org.apache.asterix</groupId>
+      <artifactId>asterix-fuzzyjoin</artifactId>
+      <version>${asterix.version}</version>
+      <!-- TODO: scope could be test if BAD runtime does not depend on fuzzyjoin functions... -->
+    </dependency>
+    <dependency>
       <groupId>org.apache.hyracks</groupId>
       <artifactId>hyracks-api</artifactId>
-      <version>${hyracks.version}</version>
     </dependency>
   </dependencies>
 </project>

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Murtadha Hubail <mh...@apache.org>
Gerrit-Reviewer: Xikui Wang <xk...@gmail.com>
Gerrit-Reviewer: abdullah alamoudi <ba...@gmail.com>

Change in asterixdb-bad[master]: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/...

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

Change subject: Coordinated change for https://asterix-gerrit.ics.uci.edu/#/c/2480/
......................................................................


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/255/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifad55aea34603d1dbcd0b52bd7b29a7610e48b5e
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No