Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread abdullah alamoudi (Code Review) 
abdullah alamoudi has posted comments on this change.

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..


Patch Set 6:

Integration tests failed due to connection refused!
I am going to trigger the build one more time.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..


Patch Set 6: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/54/ : 
SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype 
based on parallel streaming histogram.
..


Patch Set 35:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I450d0962fbeacfb2b6ab9fae0750f025ef17ba01
Gerrit-PatchSet: 35
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-Reviewer: Preston Carman 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Wenhai Li 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: Use more accurate name 'NCService' for HyracksVirtualCluster

2016-07-08 Thread Chris Hillery (Code Review) 
Chris Hillery has submitted this change and it was merged.

Change subject: Use more accurate name 'NCService' for HyracksVirtualCluster
..


Use more accurate name 'NCService' for HyracksVirtualCluster

Change-Id: I2c68c8a7abde3906df2961b214f046bdd9bfd426
Reviewed-on: https://asterix-gerrit.ics.uci.edu/982
Sonar-Qube: Jenkins 
Tested-by: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Till Westmann 
---
M 
asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/NCServiceExecutionIT.java
R 
hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksNCServiceProcess.java
M 
hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksVirtualCluster.java
M 
hyracks-fullstack/hyracks/hyracks-server/src/test/java/org/apache/hyracks/server/test/NCServiceIT.java
4 files changed, 11 insertions(+), 13 deletions(-)

Approvals:
  Till Westmann: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified



diff --git 
a/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/NCServiceExecutionIT.java
 
b/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/NCServiceExecutionIT.java
index c179103..696726f 100644
--- 
a/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/NCServiceExecutionIT.java
+++ 
b/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/NCServiceExecutionIT.java
@@ -111,11 +111,11 @@
 HDFSCluster.getInstance().setup(ASTERIX_APP_DIR + File.separator);
 
 cluster = new HyracksVirtualCluster(new File(APP_HOME), new 
File(ASTERIX_APP_DIR));
-cluster.addNC(
+cluster.addNCService(
 new File(CONF_DIR, "ncservice1.conf"),
 new File(LOG_DIR, "ncservice1.log")
 );
-cluster.addNC(
+cluster.addNCService(
 new File(CONF_DIR, "ncservice2.conf"),
 new File(LOG_DIR, "ncservice2.log")
 );
diff --git 
a/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksNCProcess.java
 
b/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksNCServiceProcess.java
similarity index 88%
rename from 
hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksNCProcess.java
rename to 
hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksNCServiceProcess.java
index 8bc1694..39971ad 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksNCProcess.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksNCServiceProcess.java
@@ -23,9 +23,9 @@
 import java.io.File;
 import java.util.List;
 
-public class HyracksNCProcess extends HyracksServerProcess {
+public class HyracksNCServiceProcess extends HyracksServerProcess {
 
-public HyracksNCProcess(File configFile, File logFile, File appHome, File 
workingDir) {
+public HyracksNCServiceProcess(File configFile, File logFile, File 
appHome, File workingDir) {
 this.configFile = configFile;
 this.logFile = logFile;
 this.appHome = appHome;
diff --git 
a/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksVirtualCluster.java
 
b/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksVirtualCluster.java
index f08bb43..cc0ecf7 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksVirtualCluster.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-server/src/main/java/org/apache/hyracks/server/process/HyracksVirtualCluster.java
@@ -30,7 +30,7 @@
 public class HyracksVirtualCluster {
 private final File appHome;
 private final File workingDir;
-private List ncProcs = new ArrayList<>(3);
+private List ncProcs = new ArrayList<>(3);
 private HyracksCCProcess ccProc = null;
 
 /**
@@ -51,15 +51,15 @@
  * @param configFile - full path to an ncservice.conf. May be null to 
accept all defaults.
  * @throws IOException - if there are errors starting the process.
  */
-public void addNC(File configFile, File logFile) throws IOException {
-HyracksNCProcess proc = new HyracksNCProcess(configFile, logFile, 
appHome, workingDir);
+public void addNCService(File configFile, File logFile) throws IOException 
{
+HyracksNCServiceProcess proc = new HyracksNCServiceProcess(configFile, 
logFile, appHome, workingDir);
 proc.start();
 ncProcs.add(proc);
 }
 
 /**
  * Starts the CC, initializing the cluster. Expects that any

Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype 
based on parallel streaming histogram.
..


Patch Set 35:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I450d0962fbeacfb2b6ab9fae0750f025ef17ba01
Gerrit-PatchSet: 35
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-Reviewer: Preston Carman 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Wenhai Li 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...

2016-07-08 Thread Yingyi Bu (Code Review) 
Yingyi Bu has posted comments on this change.

Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype 
based on parallel streaming histogram.
..


Patch Set 35:

(18 comments)

Here are some high-level comments:

1. Let partition number be the second argument of the range generation function;

2. Use JDK library as much as possible, e.g., binary search and priority queue;

3. Move histogram generation code from Hyracks to Asterix and remove type enums 
in the histogram processing code.

Detailed comments are inlined.

https://asterix-gerrit.ics.uci.edu/#/c/806/35/asterixdb/asterix-app/src/test/resources/runtimets/queries/aggregate/rg_double/rg_double.3.query.aql
File 
asterixdb/asterix-app/src/test/resources/runtimets/queries/aggregate/rg_double/rg_double.3.query.aql:

Line 20: set partitions '2'
can "partitions" be an argument of the function?


https://asterix-gerrit.ics.uci.edu/#/c/806/35/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/IHeapList.java
File 
hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/IHeapList.java:

Line 28: public void maintainHeap() throws HyracksDataException;
revisit the possibility of using priority queue and not exposing xxxHeap as 
public methods,  since xxxHeap seems to be implementation details rather than 
abstractions.


https://asterix-gerrit.ics.uci.edu/#/c/806/35/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/IHistogram.java
File 
hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/IHistogram.java:

Line 38: SHORT,
Those types are not AsterixDB types.  
Can we move all histogram code into asterixdb-runtime codebase and only handle 
asterixdb types.


Line 52: public void appendPair(E item, int count, boolean soft) throws 
HyracksDataException;
What's the difference between addPair and appendPair?  Do we need both at 
abstraction level?

What does soft mean?  Is it sth. necessary at the abstraction level?


Line 54: public List> generate(boolean isGlobal) throws 
HyracksDataException;
Must there be an final step to call generate? Can generate be called multiple 
times?  If it can be called multiple times, can we use "get" instead of 
"generate"?


https://asterix-gerrit.ics.uci.edu/#/c/806/35/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/IQuantileList.java
File 
hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/IQuantileList.java:

Line 29: public GenericQuantile get(int index);
Can you add some high-level annotation to describe what each method means?

Are all of those methods needed at the abstraction level?


https://asterix-gerrit.ics.uci.edu/#/c/806/35/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/structures/AbstractQuantileList.java
File 
hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/structures/AbstractQuantileList.java:

Line 34: private List> bins;
Use a fixed size array?


Line 38: protected DominantQuantile heapHead;
Use Java's PriorityQueue?


Line 40: protected Random prng;
prng is not used in this abstract class?

Move this to subclasses?  It might end up with duplications across different 
subclasses, but readability seems better to me.


Line 46: prng = new Random(31183);
Why is the random seed a fixed number?
Usually it's sth. like System.nanoTime().


Line 60: } else {
Use JDK library binarySearch?


Line 102: public void add(Double key, Integer value) {
Double->double?


Line 120: public void add(Double key) throws HyracksDataException {
Double->double?


Line 125: else if (bins.size() < cardinality) {
is the code formatted correctly? "else if" should be appended to the previous 
line.


Line 175: public void setKey(int index, Double key) {
Double->double?


https://asterix-gerrit.ics.uci.edu/#/c/806/35/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/structures/DominantQuantile.java
File 
hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/structures/DominantQuantile.java:

Line 70: return super.hashCode();
This doesn't seem right. hashCode implementation should be consistent with 
equals.

E.g.:
a.equals(b) --> a.hashCode() == b.hashCode()


https://asterix-gerrit.ics.uci.edu/#/c/806/35/hyracks-fullstack/hyracks/hyracks-dataflow-std/src/main/java/org/apache/hyracks/dataflow/std/range/structures/GenericQuantile.java
File

Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype 
based on parallel streaming histogram.
..


Patch Set 35:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/56/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I450d0962fbeacfb2b6ab9fae0750f025ef17ba01
Gerrit-PatchSet: 35
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-Reviewer: Preston Carman 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Wenhai Li 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: more graduation changes

2016-07-08 Thread Till Westmann (Code Review) 
Till Westmann has submitted this change and it was merged.

Change subject: more graduation changes
..


more graduation changes

Change-Id: Ic5ae8c7dc5698b2a309667d779e2a9eff4a892b7
Reviewed-on: https://asterix-gerrit.ics.uci.edu/984
Reviewed-by: Yingyi Bu 
---
M _config.yml
M content/download.html
M content/index.html
M download.md
M index.md
5 files changed, 19 insertions(+), 19 deletions(-)

Approvals:
  Yingyi Bu: Looks good to me, approved



diff --git a/_config.yml b/_config.yml
index 03926b4..a24ff52 100644
--- a/_config.yml
+++ b/_config.yml
@@ -3,9 +3,9 @@
 
 STABLE_VERSION: 0.8.8-incubating
 UNSTABLE_VERSION: 0.8.9-SNAPSHOT
-STABLE_DOWNLOAD_URL: 
"http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
+STABLE_DOWNLOAD_URL: 
"http://www.apache.org/dyn/closer.lua/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 UNSTABLE_DOWNLOAD_URL: 
"https://asterix-jenkins.ics.uci.edu/job/asterix-integration-tests/lastSuccessfulBuild/org.apache.asterix$asterix-installer/artifact/org.apache.asterix/asterix-installer/0.8.9-SNAPSHOT/asterix-installer-0.8.9-SNAPSHOT-binary-assembly.zip;
-STABLE_YARN_URL: 
"http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
+STABLE_YARN_URL: 
"http://www.apache.org/dyn/closer.lua/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
 UNSTABLE_YARN_URL: 
"https://asterix-jenkins.ics.uci.edu/job/asterix-integration-tests/lastSuccessfulBuild/org.apache.asterix$asterix-yarn/artifact/org.apache.asterix/asterix-yarn/0.8.9-SNAPSHOT/asterix-yarn-0.8.9-SNAPSHOT-binary-assembly.zip;
 DOCS_BASE_URL: "docs"
 UNSTABLE_DOCS_BASE_URL: "https://ci.apache.org/projects/asterixdb/;
diff --git a/content/download.html b/content/download.html
index 0f871c4..8c9cbff 100644
--- a/content/download.html
+++ b/content/download.html
@@ -119,7 +119,7 @@
   
 
 
-https://www.apache.org/dyn/closer.cgi/incubator/asterixdb; 
role="button">Source Tarballs 
+https://www.apache.org/dyn/closer.cgi/asterixdb; role="button">Source 
Tarballs 
 https://github.com/apache/asterixdb; role="button">AsterixDB on GitHub 
 
   
@@ -141,7 +141,7 @@
 
 
 
-http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
+http://www.apache.org/dyn/closer.lua/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
 
   
 
@@ -164,7 +164,7 @@
 
 
 
-http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
+http://www.apache.org/dyn/closer.lua/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
 
   
 
diff --git a/content/index.html b/content/index.html
index b2bd245..547457f 100644
--- a/content/index.html
+++ b/content/index.html
@@ -180,17 +180,17 @@
   
   
 Community and Support
-If you have any questions, please feel free to ask on our users
 mailing list, which you can subscribe to by sending an empty email to users-subscribe. 
Check out the Community page for more details.
+If you have any questions, please feel free to ask on our users
 mailing list, which you can subscribe to by sending an empty email to users-subscribe. Check out the 
Community page for more details.
 
 If you would like to know more about how to contribute to AsterixDB or 
any of its related components, head on over to the Contributing page to learn more.
   
   
 Download
-http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating 
-  https://github.com/apache/incubator-asterixdb/; role="button">GitHub 
+http://www.apache.org/dyn/closer.lua/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating 
+  https://github.com/apache/asterixdb/; role="button">GitHub 
 Installation Guide
 
-AsterixDB 101
+AsterixDB 101
   
 
 
diff --git a/download.md b/download.md
index 7165e3a..4063543 100644
--- a/download.md
+++ b/download.md
@@ -17,7 +17,7 @@
 
 
 
-https://www.apache.org/dyn/closer.cgi/incubator/asterixdb; 
role="button">Source Tarballs 
+https://www.apache.org/dyn/closer.cgi/asterixdb; role="button">Source 
Tarballs 
 https://github.com/apache/asterixdb; 
role="button">AsterixDB on GitHub 
 
 
diff --git a/index.md b/index.md
index 19e593b..50273a6 100644
--- a/index.md
+++ b/index.md
@@ -10,17 +10,17 @@
 
   
 ### Key Features ###
-* [Flexible](docs/0.8.8-incubating/aql/datamodel.html) data 
model
+* [Flexible]({{ site.DOCS_BASE_URL }}/{{ site.STABLE_VERSION

Change in incubator-asterixdb-site[asf-site]: more graduation changes

2016-07-08 Thread Yingyi Bu (Code Review) 
Yingyi Bu has posted comments on this change.

Change subject: more graduation changes
..


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5ae8c7dc5698b2a309667d779e2a9eff4a892b7
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: more graduation changes

2016-07-08 Thread Till Westmann (Code Review) 
Till Westmann has uploaded a new change for review.

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

Change subject: more graduation changes
..

more graduation changes

Change-Id: Ic5ae8c7dc5698b2a309667d779e2a9eff4a892b7
---
M _config.yml
M content/download.html
M content/index.html
M download.md
M index.md
5 files changed, 19 insertions(+), 19 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site 
refs/changes/84/984/1

diff --git a/_config.yml b/_config.yml
index 03926b4..a24ff52 100644
--- a/_config.yml
+++ b/_config.yml
@@ -3,9 +3,9 @@
 
 STABLE_VERSION: 0.8.8-incubating
 UNSTABLE_VERSION: 0.8.9-SNAPSHOT
-STABLE_DOWNLOAD_URL: 
"http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
+STABLE_DOWNLOAD_URL: 
"http://www.apache.org/dyn/closer.lua/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 UNSTABLE_DOWNLOAD_URL: 
"https://asterix-jenkins.ics.uci.edu/job/asterix-integration-tests/lastSuccessfulBuild/org.apache.asterix$asterix-installer/artifact/org.apache.asterix/asterix-installer/0.8.9-SNAPSHOT/asterix-installer-0.8.9-SNAPSHOT-binary-assembly.zip;
-STABLE_YARN_URL: 
"http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
+STABLE_YARN_URL: 
"http://www.apache.org/dyn/closer.lua/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
 UNSTABLE_YARN_URL: 
"https://asterix-jenkins.ics.uci.edu/job/asterix-integration-tests/lastSuccessfulBuild/org.apache.asterix$asterix-yarn/artifact/org.apache.asterix/asterix-yarn/0.8.9-SNAPSHOT/asterix-yarn-0.8.9-SNAPSHOT-binary-assembly.zip;
 DOCS_BASE_URL: "docs"
 UNSTABLE_DOCS_BASE_URL: "https://ci.apache.org/projects/asterixdb/;
diff --git a/content/download.html b/content/download.html
index 0f871c4..8c9cbff 100644
--- a/content/download.html
+++ b/content/download.html
@@ -119,7 +119,7 @@
   
 
 
-https://www.apache.org/dyn/closer.cgi/incubator/asterixdb; 
role="button">Source Tarballs 
+https://www.apache.org/dyn/closer.cgi/asterixdb; role="button">Source 
Tarballs 
 https://github.com/apache/asterixdb; role="button">AsterixDB on GitHub 
 
   
@@ -141,7 +141,7 @@
 
 
 
-http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
+http://www.apache.org/dyn/closer.lua/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
 
   
 
@@ -164,7 +164,7 @@
 
 
 
-http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
+http://www.apache.org/dyn/closer.lua/asterixdb/asterix-yarn-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating  
 
   
 
diff --git a/content/index.html b/content/index.html
index b2bd245..547457f 100644
--- a/content/index.html
+++ b/content/index.html
@@ -180,17 +180,17 @@
   
   
 Community and Support
-If you have any questions, please feel free to ask on our users
 mailing list, which you can subscribe to by sending an empty email to users-subscribe. 
Check out the Community page for more details.
+If you have any questions, please feel free to ask on our users
 mailing list, which you can subscribe to by sending an empty email to users-subscribe. Check out the 
Community page for more details.
 
 If you would like to know more about how to contribute to AsterixDB or 
any of its related components, head on over to the Contributing page to learn more.
   
   
 Download
-http://www.apache.org/dyn/closer.lua/incubator/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating 
-  https://github.com/apache/incubator-asterixdb/; role="button">GitHub 
+http://www.apache.org/dyn/closer.lua/asterixdb/asterix-installer-0.8.8-incubating-binary-assembly.zip;
 role="button">Download AsterixDB 0.8.8-incubating 
+  https://github.com/apache/asterixdb/; role="button">GitHub 
 Installation Guide
 
-AsterixDB 101
+AsterixDB 101
   
 
 
diff --git a/download.md b/download.md
index 7165e3a..4063543 100644
--- a/download.md
+++ b/download.md
@@ -17,7 +17,7 @@
 
 
 
-https://www.apache.org/dyn/closer.cgi/incubator/asterixdb; 
role="button">Source Tarballs 
+https://www.apache.org/dyn/closer.cgi/asterixdb; role="button">Source 
Tarballs 
 https://github.com/apache/asterixdb; 
role="button">AsterixDB on GitHub 
 
 
diff --git a/index.md b/index.md
index 19e593b..50273a6 100644
--- a/index.md
+++ b/index.md
@@ -10,17 +10,17 @@
 
   
 ### Key Features ###
-* [Flexible](docs/0.8.8-incubating/aql/datamodel.html) data 
model
+* [Flexible]({{ site.DOCS_BASE_URL }}/{{ site.STABLE_VERSION 
}}/aql/datamodel.html) data

Change in incubator-asterixdb-site[asf-site]: Update the dev-setup document to reflect recent changes - r...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Update the dev-setup document to reflect recent changes  - 
removed incubator string  - revised instruction to import asterixdb only
..


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/66/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iebee0da98ec4a7bc1a6e404293ae56bbb4390555
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update the dev-setup document to reflect recent changes - r...

2016-07-08 Thread Taewoo Kim (Code Review) 
Taewoo Kim has submitted this change and it was merged.

Change subject: Update the dev-setup document to reflect recent changes  - 
removed incubator string  - revised instruction to import asterixdb only
..


Update the dev-setup document to reflect recent changes
 - removed incubator string
 - revised instruction to import asterixdb only

Change-Id: Iebee0da98ec4a7bc1a6e404293ae56bbb4390555
Reviewed-on: https://asterix-gerrit.ics.uci.edu/983
Reviewed-by: Till Westmann 
---
M content/dev-setup.html
M dev-setup.md
2 files changed, 37 insertions(+), 54 deletions(-)

Approvals:
  Till Westmann: Looks good to me, approved



diff --git a/content/dev-setup.html b/content/dev-setup.html
index 57a8e68..0112075 100644
--- a/content/dev-setup.html
+++ b/content/dev-setup.html
@@ -167,45 +167,27 @@
 
 
   
-Check out Hyracks and Asterix master in two folders via git in the 
command line. Assume that the path is $HOME/workspace.
+Check out AsterixDB master in one folder via git in the command line. 
Assume that the path is $HOME/workspace.
 
- git clone https://github.com/apache/incubator-asterixdb/
- git clone https://github.com/apache/incubator-asterixdb-hyracks/
+ git clone https://github.com/apache/asterixdb/
 
 
-You will now have $HOME/workspace/incubator-asterixdb/ and 
$HOME/workspace/incubator-asterixdb-hyracks/.
+You will now have $HOME/workspace/asterixdb/.
   
   
-Go to the hyracks and install it’s artifacts to the local Maven 
repository by executing the following commands:
+Go to the asterixdb folder and install it’s artifacts to the local 
Maven repository by executing the following commands:
 
- cd incubator-asterixdb-hyracks/
+ cd asterixdb/
  mvn install -DskipTests
 
   
-  
-Go to the asterixdb folder and perform the same action:
-
-  cd ../incubator-asterixdb/asterix-maven-plugins/
-  mvn install -DskipTests
-  cd ../
-  mvn install -DskipTests
-
-  
-  In Eclipse, import hyracks as an existing Maven Project.
+  In Eclipse, import asterixdb as an existing Maven Project.
 
   File - Import - Maven - Existing Maven Projects 
- Next
-  Specify the Root directory as 
incubator-asterixdb-hyracks and then click Next until Finish is 
enabled.
+  Specify the Root directory as asterixdb and then click 
Next until Finish is enabled.
   If Eclipse tries to install the m2e (Maven To Eclipse) 
connector, let it do so.
   There might be some plugin errors; however, that is not a big deal. 
Wait until the job finishes.
-  Then, click Finish.
-
-  
-  Then import asterixdb as an existing Maven Project.
-
-  File - Import - Maven - Existing Maven Projects 
- Next
-  Specify root directory as $HOME/workspace/asterixdb and 
then click Next until Finish is enabled.
-  Eclipse may try installing m2e or other plugins again; 
let it do so.
-  Then, click Finish.
+  Then, click Finish. Restart Eclipse if asked to do so.
 
   
   Fix Eclipse’s build path to include compile-time generated sources.
@@ -213,12 +195,13 @@
   Right click the project where a red X mark is shown(e.g. 
asterix-algebra. Then resolve by applying the following:
 
   Right click asterix-algebra. Click Build Path and Configure 
Build Path. Click Add Folder.
-  Under the target - generated sources, check the 
parent folder of the edu folder and click OK.
+  Under the target - generated sources, check the 
parent folder of the org folder and click OK.
   For example, if the directory structure is “target - 
generated-sources - javacc - org - apache …”, check the javacc directory and 
click OK. Then X mark will dissapear.
 
   
   Repeat this step to all projects which show a red X mark except 
“asterix-fuzzyjoin” and “asterix-transactions”.
-  It may be the case that only “asterix-algebra” and “asterix-runtime” 
will require these steps.
+  It may be the case that only “asterix-algebra” and 
“asterix-external-data” will require these steps.
+  If you see “plugin execution not covered by lifecycle configuration” 
errors such as this one - “Plugin execution not covered by lifecycle 
configuration: 
org.apache.asterix:asterix-test-datagenerator-maven-plugin:0.8.9-SNAPSHOT:generate-testdata
 (execution: replace-template-data, phase: process-test-resources)”, try to 
resolve it using the quick fix feature. You can call this feature by right 
click on the error and select “quick fix”. And then choose “Mark goal 
generate-testdata as ignored in Eclipse build in Eclipse preferences”.
 
   
   Set up Eclipse code formatting rules
@@ -234,6 +217,13 @@
   
 
   
+  
+Lastly, go to the asterixdb folder and execute following command again. 
This is required since Eclipse might have cleaned the projects and rebuilt the 
them without creating all

Change in asterixdb[master]: RangeGenerator aggfunc for the numeric/asciiString datatype ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: RangeGenerator aggfunc for the numeric/asciiString datatype 
based on parallel streaming histogram.
..


Patch Set 35:

Integration Tests Aborted

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/56/ : 
ABORTED

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I450d0962fbeacfb2b6ab9fae0750f025ef17ba01
Gerrit-PatchSet: 35
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-Reviewer: Preston Carman 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Wenhai Li 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: some more graduation changes

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: some more graduation changes
..


Patch Set 1:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update the dev-setup document to reflect recent changes - r...

2016-07-08 Thread Taewoo Kim (Code Review) 
Taewoo Kim has uploaded a new change for review.

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

Change subject: Update the dev-setup document to reflect recent changes  - 
removed incubator string  - revised instruction to import asterixdb only
..

Update the dev-setup document to reflect recent changes
 - removed incubator string
 - revised instruction to import asterixdb only

Change-Id: Iebee0da98ec4a7bc1a6e404293ae56bbb4390555
---
M content/dev-setup.html
M dev-setup.md
2 files changed, 37 insertions(+), 54 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site 
refs/changes/83/983/1

diff --git a/content/dev-setup.html b/content/dev-setup.html
index 57a8e68..3dea0b2 100644
--- a/content/dev-setup.html
+++ b/content/dev-setup.html
@@ -167,45 +167,27 @@
 
 
   
-Check out Hyracks and Asterix master in two folders via git in the 
command line. Assume that the path is $HOME/workspace.
+Check out AsterixDB master in one folder via git in the command line. 
Assume that the path is $HOME/workspace.
 
- git clone https://github.com/apache/incubator-asterixdb/
- git clone https://github.com/apache/incubator-asterixdb-hyracks/
+ git clone https://github.com/apache/asterixdb/
 
 
-You will now have $HOME/workspace/incubator-asterixdb/ and 
$HOME/workspace/incubator-asterixdb-hyracks/.
+You will now have $HOME/workspace/asterixdb/.
   
   
-Go to the hyracks and install it’s artifacts to the local Maven 
repository by executing the following commands:
+Go to the asterixdb and install it’s artifacts to the local Maven 
repository by executing the following commands:
 
- cd incubator-asterixdb-hyracks/
+ cd asterixdb/
  mvn install -DskipTests
 
   
-  
-Go to the asterixdb folder and perform the same action:
-
-  cd ../incubator-asterixdb/asterix-maven-plugins/
-  mvn install -DskipTests
-  cd ../
-  mvn install -DskipTests
-
-  
-  In Eclipse, import hyracks as an existing Maven Project.
+  In Eclipse, import asterixdb as an existing Maven Project.
 
   File - Import - Maven - Existing Maven Projects 
- Next
-  Specify the Root directory as 
incubator-asterixdb-hyracks and then click Next until Finish is 
enabled.
+  Specify the Root directory as asterixdb and then click 
Next until Finish is enabled.
   If Eclipse tries to install the m2e (Maven To Eclipse) 
connector, let it do so.
   There might be some plugin errors; however, that is not a big deal. 
Wait until the job finishes.
-  Then, click Finish.
-
-  
-  Then import asterixdb as an existing Maven Project.
-
-  File - Import - Maven - Existing Maven Projects 
- Next
-  Specify root directory as $HOME/workspace/asterixdb and 
then click Next until Finish is enabled.
-  Eclipse may try installing m2e or other plugins again; 
let it do so.
-  Then, click Finish.
+  Then, click Finish. Restart Eclipse if asked to do so.
 
   
   Fix Eclipse’s build path to include compile-time generated sources.
@@ -213,12 +195,13 @@
   Right click the project where a red X mark is shown(e.g. 
asterix-algebra. Then resolve by applying the following:
 
   Right click asterix-algebra. Click Build Path and Configure 
Build Path. Click Add Folder.
-  Under the target - generated sources, check the 
parent folder of the edu folder and click OK.
+  Under the target - generated sources, check the 
parent folder of the org folder and click OK.
   For example, if the directory structure is “target - 
generated-sources - javacc - org - apache …”, check the javacc directory and 
click OK. Then X mark will dissapear.
 
   
   Repeat this step to all projects which show a red X mark except 
“asterix-fuzzyjoin” and “asterix-transactions”.
-  It may be the case that only “asterix-algebra” and “asterix-runtime” 
will require these steps.
+  It may be the case that only “asterix-algebra” and 
“asterix-external-data” will require these steps.
+  If you see “plugin execution not covered by lifecycle configuration” 
errors such as this one - “Plugin execution not covered by lifecycle 
configuration: 
org.apache.asterix:asterix-test-datagenerator-maven-plugin:0.8.9-SNAPSHOT:generate-testdata
 (execution: replace-template-data, phase: process-test-resources)”, try to 
resolve it using the quick fix feature. You can call this feature by right 
click on the error and select “quick fix”. And then choose “Mark goal 
generate-testdata as ignored in Eclipse build in Eclipse preferences”.
 
   
   Set up Eclipse code formatting rules
@@ -234,6 +217,13 @@
   
 
   
+  
+Lastly, go to the asterixdb and execute following command again. This 
is required since Eclipse might have cleaned the projects and rebuilt the them 
without creating all necessary classes. Currently, some

Change in incubator-asterixdb-site[asf-site]: more graduation changes

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: more graduation changes
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asf-site/65/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5ae8c7dc5698b2a309667d779e2a9eff4a892b7
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: some more graduation changes

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: some more graduation changes
..


Patch Set 1:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/57/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update the dev-setup document to reflect recent changes - r...

2016-07-08 Thread Till Westmann (Code Review) 
Till Westmann has posted comments on this change.

Change subject: Update the dev-setup document to reflect recent changes  - 
removed incubator string  - revised instruction to import asterixdb only
..


Patch Set 1:

(3 comments)

Just a few small fixes.

https://asterix-gerrit.ics.uci.edu/#/c/983/1/dev-setup.md
File dev-setup.md:

Line 48: 2. Go to the asterixdb and install it's artifacts to the local Maven 
repository by executing the following commands:
s/go to the asterixdb/go to the asterixdb folder/


Line 78: 6. Lastly, go to the asterixdb and execute following command again. 
This is required since Eclipse might have cleaned the projects and rebuilt the 
them without creating all necessary classes. Currently, some of the class files 
can be only built using mvn command.
s/go to the asterixdb/go to the asterixdb folder/


Line 82:  
WS


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iebee0da98ec4a7bc1a6e404293ae56bbb4390555
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update the dev-setup document to reflect recent changes - r...

2016-07-08 Thread Till Westmann (Code Review) 
Till Westmann has posted comments on this change.

Change subject: Update the dev-setup document to reflect recent changes  - 
removed incubator string  - revised instruction to import asterixdb only
..


Patch Set 2: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iebee0da98ec4a7bc1a6e404293ae56bbb4390555
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: some more graduation changes

2016-07-08 Thread Till Westmann (Code Review) 
Till Westmann has uploaded a new change for review.

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

Change subject: some more graduation changes
..

some more graduation changes

Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
---
M asterixdb/pom.xml
M hyracks-fullstack/pom.xml
2 files changed, 2 insertions(+), 12 deletions(-)


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

diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index 4d372d5..d7b0fed 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -206,7 +206,6 @@
   
${appendedResourcesDirectory}
   
 
org.apache:apache-jar-resource-bundle:1.4
-
org.apache:apache-incubator-disclaimer-resource-bundle:1.1
   
 
   
@@ -361,10 +360,9 @@
   
 
   
-
scm:git:https://github.com/apache/incubator-asterixdb
+scm:git:https://github.com/apache/asterixdb
 
scm:git:ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
-https://github.com/apache/incubator-asterixdb
-
+https://github.com/apache/asterixdb
 HEAD
   
 
diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml
index 8ea61dc..e0b12dd 100644
--- a/hyracks-fullstack/pom.xml
+++ b/hyracks-fullstack/pom.xml
@@ -275,7 +275,6 @@
 
   
 
org.apache:apache-jar-resource-bundle:1.4
-
org.apache:apache-incubator-disclaimer-resource-bundle:1.1
   
 
   
@@ -372,13 +371,6 @@
   
 
   
-
-  
-
scm:git:https://github.com/apache/incubator-asterixdb-hyracks
-
scm:git:ssh://fulliautomatix.ics.uci.edu:29418/hyracks
-https://github.com/apache/incubator-asterixdb-hyracks
-HEAD
-  
 
   
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann

Change in asterixdb[master]: some more graduation changes

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: some more graduation changes
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: some more graduation changes

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: some more graduation changes
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/57/ : 
SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: some more graduation changes

2016-07-08 Thread Till Westmann (Code Review) 
Till Westmann has submitted this change and it was merged.

Change subject: some more graduation changes
..


some more graduation changes

Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
Reviewed-on: https://asterix-gerrit.ics.uci.edu/985
Sonar-Qube: Jenkins 
Tested-by: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Yingyi Bu 
---
M asterixdb/pom.xml
M hyracks-fullstack/pom.xml
2 files changed, 2 insertions(+), 12 deletions(-)

Approvals:
  Yingyi Bu: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified



diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index 4d372d5..d7b0fed 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -206,7 +206,6 @@
   
${appendedResourcesDirectory}
   
 
org.apache:apache-jar-resource-bundle:1.4
-
org.apache:apache-incubator-disclaimer-resource-bundle:1.1
   
 
   
@@ -361,10 +360,9 @@
   
 
   
-
scm:git:https://github.com/apache/incubator-asterixdb
+scm:git:https://github.com/apache/asterixdb
 
scm:git:ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
-https://github.com/apache/incubator-asterixdb
-
+https://github.com/apache/asterixdb
 HEAD
   
 
diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml
index 8ea61dc..e0b12dd 100644
--- a/hyracks-fullstack/pom.xml
+++ b/hyracks-fullstack/pom.xml
@@ -275,7 +275,6 @@
 
   
 
org.apache:apache-jar-resource-bundle:1.4
-
org.apache:apache-incubator-disclaimer-resource-bundle:1.1
   
 
   
@@ -372,13 +371,6 @@
   
 
   
-
-  
-
scm:git:https://github.com/apache/incubator-asterixdb-hyracks
-
scm:git:ssh://fulliautomatix.ics.uci.edu:29418/hyracks
-https://github.com/apache/incubator-asterixdb-hyracks
-HEAD
-  
 
   
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia901209e62a6803817e63a8495b07987cd95e0e1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu

Change in asterixdb[master]: Index-only plan

2016-07-08 Thread Young-Seok Kim (Code Review) 
Young-Seok Kim has posted comments on this change.

Change subject: Index-only plan
..


Patch Set 11:

(21 comments)

The following three issues are discovered.
1. R-tree index on rectangle type is not correctly dealt with for the 
corresponding index-only plan case
   This case should be added as a test case.
2. limit-push-down for primary index scan doesn't seem to work.
3. limit-push-down for primary-index search doesn't seem to work.

Also, please address other comments.

https://asterix-gerrit.ics.uci.edu/#/c/744/11/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/AbstractReplicateOperator.java
File 
algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/AbstractReplicateOperator.java:

Line 36: private boolean[] outputMaterializationFlags = new 
boolean[outputArity];
I wonder why this flag is required.


https://asterix-gerrit.ics.uci.edu/#/c/744/11/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVarsRule.java
File 
algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVarsRule.java:

Line 45: public class RemoveRedundantGroupByDecorVarsRule implements 
IAlgebraicRewriteRule {
Is this class introduced for just for optimization, not for the correctness 
issue?


https://asterix-gerrit.ics.uci.edu/#/c/744/11/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/PrimaryIndexSearchOperationDatasetLevelCallback.java
File 
asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/PrimaryIndexSearchOperationDatasetLevelCallback.java:

Line 35: public class PrimaryIndexSearchOperationDatasetLevelCallback extends 
AbstractOperationCallback implements ISearchOperationCallback {
This class should be removed.


https://asterix-gerrit.ics.uci.edu/#/c/744/11/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/SecondaryIndexInstantSearchOperationTryLockOnlyOnPKCallback.java
File 
asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/SecondaryIndexInstantSearchOperationTryLockOnlyOnPKCallback.java:

Line 33: public class 
SecondaryIndexInstantSearchOperationTryLockOnlyOnPKCallback extends 
AbstractOperationCallback
Can we give a simpler class name, e.g., 
SecondaryIndexInstantSearchOperationCallback?


https://asterix-gerrit.ics.uci.edu/#/c/744/11/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java:

Line 644: 
.getIxJoinOuterAdditionalDataSourceVariables(i);
What's the difference between subTreeDataSourceVars and subTreePKs? why is it 
OK for subTreeDataSourceVars to replace some of subTreePks below?


Line 792: expr = (AbstractLogicalExpression) 
childFuncExpr.getArguments().get(0).getValue();
better be dealing with an else case explicitly, i.e., neither Assign nor unnest 
case.


https://asterix-gerrit.ics.uci.edu/#/c/744/11/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodAnalysisContext.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodAnalysisContext.java:

Line 65: //whether a verification (especially for R-Tree case) is 
required after the secondary index search?
what's the difference between 3. requireVerificationAfterSIdxSearch and 4. 
doesSIdxSearchGenerateNoFalsePositiveResults? It seems the same? It would be 
good to explain the difference with example cases.


Line 67: //Does the given index can cover all search predicates?
It seems that the line 67 and 69 should be switched??


https://asterix-gerrit.ics.uci.edu/#/c/744/11/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodJobGenParams.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodJobGenParams.java:

Line 44: protected boolean isPrimaryIndex;
Please say that "isPrimaryIndex is a derived parameter from the index name, so 
this parameter is not included in the 8 (NUM_PARAMS) parameters written and 
read."


https://asterix-gerrit.ics.uci.edu/#/c/744/11/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java:

Line 260: // If it is not granted, then we need to do a secondary index 
lookup, sort PKs,
Do we sort in this case? ( I thought we don't do sort for this trylockfailure 
case?)


Line 1301:

[jira] [Resolved] (ASTERIXDB-1513) Error in POM file after importing to eclipse

2016-07-08 Thread Taewoo Kim (JIRA) 

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-1513?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Taewoo Kim resolved ASTERIXDB-1513.
---
Resolution: Fixed

> Error in POM file after importing to eclipse
> 
>
> Key: ASTERIXDB-1513
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1513
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Mac OSX El Capitan 10.11.5
> Eclipse Mars.2 Release (4.5.2)
> Apache Maven 3.3.9
> m2e 1.7.0.20160603-1933
>Reporter: Vignesh Raghunathan
>Assignee: Taewoo Kim
>Priority: Blocker
> Attachments: Screen Shot 2016-07-03 at 3.17.01 PM.png, Screen Shot 
> 2016-07-06 at 1.05.09 PM.png, Screen Shot 2016-07-06 at 1.11.04 PM.png
>
>
> After importing hyracks and asterixdb projects into eclipse, there are errors 
> in the POM file in many hyracks projects. I followed all steps in 
> https://asterixdb.apache.org/dev-setup.html for setting up asterixdb 
> environment in eclipse



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (ASTERIXDB-1513) Error in POM file after importing to eclipse

2016-07-08 Thread Taewoo Kim (JIRA) 

[ 
https://issues.apache.org/jira/browse/ASTERIXDB-1513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15368905#comment-15368905
 ] 

Taewoo Kim commented on ASTERIXDB-1513:
---

https://asterixdb.apache.org/dev-setup.html is updated.

> Error in POM file after importing to eclipse
> 
>
> Key: ASTERIXDB-1513
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1513
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Mac OSX El Capitan 10.11.5
> Eclipse Mars.2 Release (4.5.2)
> Apache Maven 3.3.9
> m2e 1.7.0.20160603-1933
>Reporter: Vignesh Raghunathan
>Assignee: Taewoo Kim
>Priority: Blocker
> Attachments: Screen Shot 2016-07-03 at 3.17.01 PM.png, Screen Shot 
> 2016-07-06 at 1.05.09 PM.png, Screen Shot 2016-07-06 at 1.11.04 PM.png
>
>
> After importing hyracks and asterixdb projects into eclipse, there are errors 
> in the POM file in many hyracks projects. I followed all steps in 
> https://asterixdb.apache.org/dev-setup.html for setting up asterixdb 
> environment in eclipse



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Change in asterixdb[master]: Refactor General Active Classes

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Refactor General Active Classes
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/58/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0a8f33ee5b45c5e090b08c24a102e369aae43c04
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Steven Jacobs 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Remove duplicate operator symbol definition

2016-07-08 Thread Yingyi Bu (Code Review) 
Yingyi Bu has submitted this change and it was merged.

Change subject: Remove duplicate operator symbol definition
..


Remove duplicate operator symbol definition

Change-Id: I3cfcf9a7e131d60a9b97fbc9fae3b762031c8edb
Reviewed-on: https://asterix-gerrit.ics.uci.edu/979
Tested-by: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Yingyi Bu 
---
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/struct/OperatorType.java
2 files changed, 14 insertions(+), 65 deletions(-)

Approvals:
  Yingyi Bu: Looks good to me, approved
  Jenkins: Verified; Verified

Objections:
  Jenkins: Violations found



diff --git 
a/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
 
b/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
index 8f2374f..bf70df4 100644
--- 
a/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
+++ 
b/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
@@ -20,6 +20,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Optional;
 
 import org.apache.asterix.common.exceptions.AsterixException;
 import org.apache.asterix.lang.common.base.AbstractExpression;
@@ -90,71 +91,12 @@
 }
 
 public void addOperator(String strOp) {
-switch (strOp) {
-case "or":
-opList.add(OperatorType.OR);
-break;
-case "and":
-opList.add(OperatorType.AND);
-break;
-case "<":
-opList.add(OperatorType.LT);
-break;
-case ">":
-opList.add(OperatorType.GT);
-break;
-case "<=":
-opList.add(OperatorType.LE);
-break;
-case ">=":
-opList.add(OperatorType.GE);
-break;
-case "=":
-opList.add(OperatorType.EQ);
-break;
-case "!=":
-opList.add(OperatorType.NEQ);
-break;
-case "+":
-opList.add(OperatorType.PLUS);
-break;
-case "-":
-opList.add(OperatorType.MINUS);
-break;
-case "*":
-opList.add(OperatorType.MUL);
-break;
-case "/":
-opList.add(OperatorType.DIV);
-break;
-case "%":
-opList.add(OperatorType.MOD);
-break;
-case "^":
-opList.add(OperatorType.CARET);
-break;
-case "idiv":
-opList.add(OperatorType.IDIV);
-break;
-case "~=":
-opList.add(OperatorType.FUZZY_EQ);
-break;
-case "like":
-opList.add(OperatorType.LIKE);
-break;
-case "not_like":
-opList.add(OperatorType.NOT_LIKE);
-break;
-case "in":
-opList.add(OperatorType.IN);
-break;
-case "not_in":
-opList.add(OperatorType.NOT_IN);
-break;
-default:
-throw new UnsupportedOperationException("Unsupported operator: 
" + strOp);
+Optional op = OperatorType.fromSymbol(strOp);
+if (op.isPresent()) {
+opList.add(op.get());
+} else {
+throw new UnsupportedOperationException("Unsupported operator: " + 
strOp);
 }
-
 }
 
 @Override
diff --git 
a/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/struct/OperatorType.java
 
b/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/struct/OperatorType.java
index bb15ecd..6e2e3e2 100644
--- 
a/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/struct/OperatorType.java
+++ 
b/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/struct/OperatorType.java
@@ -18,6 +18,9 @@
  */
 package org.apache.asterix.lang.common.struct;
 
+import java.util.Arrays;
+import java.util.Optional;
+
 public enum OperatorType {
 OR("or"),
 AND("and"),
@@ -43,7 +46,7 @@
 
 private final String symbol;
 
-private OperatorType(String s) {
+OperatorType(String s) {
 symbol = s;
 }
 
@@ -51,4 +54,8 @@
 public String toString() {
 return symbol;
 }
+
+public static Optional fromSymbol(String symbol) {
+return

Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..


Patch Set 5:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

[jira] [Created] (ASTERIXDB-1524) No user level documentation for nested indexes

2016-07-08 Thread Abdullah Alamoudi (JIRA) 
Abdullah Alamoudi created ASTERIXDB-1524:


 Summary: No user level documentation for nested indexes
 Key: ASTERIXDB-1524
 URL: https://issues.apache.org/jira/browse/ASTERIXDB-1524
 Project: Apache AsterixDB
  Issue Type: Bug
  Components: Documentation
Reporter: Abdullah Alamoudi






--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..


Patch Set 6:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/53/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread abdullah alamoudi (Code Review) 
abdullah alamoudi has posted comments on this change.

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..


Patch Set 5: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Use more accurate name 'NCService' for HyracksVirtualCluster

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Use more accurate name 'NCService' for HyracksVirtualCluster
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2c68c8a7abde3906df2961b214f046bdd9bfd426
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Chris Hillery 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..


Patch Set 5: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/51/ : 
SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread Yingyi Bu (Code Review) 
Hello abdullah alamoudi, Till Westmann, Jenkins,

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

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

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

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..

ASTERIXDB-1494: refactor ExternalLibraryManager and make it non-static.

Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/ExternalDataLookupPOperator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/AsterixAppRuntimeContext.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/ExternalIndexingOperations.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/ExternalLibraryUtils.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/aql/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/NCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/external/TestLibrarian.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/RepeatedTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/IAsterixAppRuntimeContext.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/dataflow/IAsterixApplicationContextInfo.java
A 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/library/ILibraryManager.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/adapter/factory/GenericAdapterFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/adapter/factory/LookupAdapterFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IAdapterFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalFunction.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalLibraryManager.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/operators/FeedIntakeOperatorDescriptor.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/parser/factory/RSSParserFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/parser/factory/TweetParserFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/provider/AdapterFactoryProvider.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/provider/DatasourceFactoryProvider.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/provider/ParserFactoryProvider.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/ExternalDataUtils.java
M 
asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/library/adapter/TestTypedAdapterFactory.java
M 
asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/parser/factory/TestRecordWithPKParserFactory.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/AqlMetadataProvider.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/feeds/FeedMetadataUtil.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/util/AsterixAppContextInfo.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/application/CCApplicationContext.java
32 files changed, 337 insertions(+), 182 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/80/980/6
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/980
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi

Change in asterixdb[master]: ASTERIXDB-1494: refactor ExternalLibraryManager and make it ...

2016-07-08 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1494: refactor ExternalLibraryManager and make it 
non-static.
..


Patch Set 6:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c287a35ff90c3aea639d3069d5a842e28c5e508
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No