date:20160713

Change in asterixdb[master]: minor POM cleanup

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: minor POM cleanup
..


Patch Set 4: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea7296c8cfd5d62df5c5419e69a4890e73c2b29a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: DO NOT REVIEW DO NOT REVIEW

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: DO NOT REVIEW DO NOT REVIEW
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9a859bb6b401dc5e463c8e9c506cc00b82befa70
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: minor POM cleanup

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: minor POM cleanup
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea7296c8cfd5d62df5c5419e69a4890e73c2b29a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: minor POM cleanup

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: minor POM cleanup
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea7296c8cfd5d62df5c5419e69a4890e73c2b29a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: minor POM cleanup

2016-07-13 Thread Michael Blow (Code Review)

Michael Blow has posted comments on this change.

Change subject: minor POM cleanup
..


Patch Set 4: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea7296c8cfd5d62df5c5419e69a4890e73c2b29a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-13 Thread Michael Blow (Code Review)

Michael Blow has posted comments on this change.

Change subject: Fulltext search initial implementation
..


Patch Set 4:

(2 comments)

https://asterix-gerrit.ics.uci.edu/#/c/989/4/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java:

Line 1349: return k == Kind.LITERAL_EXPRESSION || k == 
Kind.LIST_CONSTRUCTOR_EXPRESSION
I would replace all of this with a switch which returns true for all these 
kinds.


https://asterix-gerrit.ics.uci.edu/#/c/989/4/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java
File 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/expression/OperatorExpr.java:

Line 88: boolean cmp = t == OperatorType.EQ || t == OperatorType.NEQ || 
t == OperatorType.GT;
I would replace this with an enum switch.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I71887c2ea847e4488f4c98a11f8a5bcad02cac5a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Heri Ramampiaro 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: Yes

Change in asterixdb[master]: WIP: TESTING GERRIT SONARQUBE REVIEWS

2016-07-13 Thread Michael Blow (Code Review)

Michael Blow has abandoned this change.

Change subject: WIP: TESTING GERRIT SONARQUBE REVIEWS
..


Abandoned

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: Id5742e30fc90266d29354f2f092b021f95ec85ff
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow

Change in asterixdb[master]: DO NOT REVIEW DO NOT REVIEW

2016-07-13 Thread Michael Blow (Code Review)

Michael Blow has uploaded a new change for review.

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

Change subject: DO NOT REVIEW DO NOT REVIEW
..

DO NOT REVIEW DO NOT REVIEW

Testing git-gerrit changes

Change-Id: I9a859bb6b401dc5e463c8e9c506cc00b82befa70
---
M pom.xml
1 file changed, 1 insertion(+), 0 deletions(-)


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

diff --git a/pom.xml b/pom.xml
index 44ecd1b..3463993 100644
--- a/pom.xml
+++ b/pom.xml
@@ -31,6 +31,7 @@
 
   
 
+
   
 
   Apache License, Version 2.0

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9a859bb6b401dc5e463c8e9c506cc00b82befa70
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 8: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 8:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 7: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-13 Thread Till Westmann (Code Review)

Till Westmann has posted comments on this change.

Change subject: Fulltext search initial implementation
..


Patch Set 4:

(2 comments)

https://asterix-gerrit.ics.uci.edu/#/c/989/4/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/string/UTF8StringUtil.java
File 
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/string/UTF8StringUtil.java:

Line 382: if (lengthByteIncluded) {
The decision if the length byte is included is taken in the calling code and 
not at runtime. So it seems that it'd be better to have 2 different methods 
that first determine the lengths and then call a common method to do the actual 
work.


Line 428: if (lengthByteIncluded) {
see above


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I71887c2ea847e4488f4c98a11f8a5bcad02cac5a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Heri Ramampiaro 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: Yes

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 7:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Till Westmann (Code Review)

Till Westmann has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 7:

(4 comments)

https://asterix-gerrit.ics.uci.edu/#/c/990/7/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/util/SqlppVariableUtil.java
File 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/util/SqlppVariableUtil.java:

Line 74: }
remove this now?


Line 101: }
remove this now?


Line 108: }
remove this now?


https://asterix-gerrit.ics.uci.edu/#/c/990/7/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppFormatPrintVisitor.java
File 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppFormatPrintVisitor.java:

Line 132: out.print(" as " + projection.getName());
just print 'name' here?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: Yes

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Yingyi Bu (Code Review)

Yingyi Bu has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 6:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/990/6/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/util/ExpressionToVariableUtil.java
File 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/util/ExpressionToVariableUtil.java:

Line 42: return null;
> This seems to introduce the need for a number of null checks (and potential
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
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-HasComments: Yes

[jira] [Comment Edited] (ASTERIXDB-1506) non-responsive after some data is ingested

2016-07-13 Thread Christopher Green (JIRA)


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

Christopher Green edited comment on ASTERIXDB-1506 at 7/13/16 9:46 PM:
---

Oh no!  I misspelled the host name.  It's supposed to be 
mongodbtest.chtc.wisc.edu.  My apologies for the mistake. 

CJG 


was (Author: cjgreen):
Oh no!  I misspelled the host name.  It's suppose to be 
mongodbtest.chtc.wisc.edu.  My apologies for the mistake. 

CJG 

> non-responsive after some data is ingested
> --
>
> Key: ASTERIXDB-1506
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1506
> Project: Apache AsterixDB
>  Issue Type: Bug
>  Components: Feeds
>Reporter: Christopher Green
>Assignee: Abdullah Alamoudi
>
> I updated the code and recreated the dataset.  Once the feeder is started, it 
> will ingest a couple hundred entries and then the database becomes 
> non-responsive.  I can restart the instance, but about the same amount of 
> entries will be ingested before is stops working again.  
> Let what I need to provide to start working this issue and thank you for your 
> help. 



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

[jira] [Commented] (ASTERIXDB-1506) non-responsive after some data is ingested

2016-07-13 Thread Christopher Green (JIRA)


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

Christopher Green commented on ASTERIXDB-1506:
--

Oh no!  I misspelled the host name.  It's suppose to be 
mongodbtest.chtc.wisc.edu.  My apologies for the mistake. 

CJG 

> non-responsive after some data is ingested
> --
>
> Key: ASTERIXDB-1506
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1506
> Project: Apache AsterixDB
>  Issue Type: Bug
>  Components: Feeds
>Reporter: Christopher Green
>Assignee: Abdullah Alamoudi
>
> I updated the code and recreated the dataset.  Once the feeder is started, it 
> will ingest a couple hundred entries and then the database becomes 
> non-responsive.  I can restart the instance, but about the same amount of 
> entries will be ingested before is stops working again.  
> Let what I need to provide to start working this issue and thank you for your 
> help. 



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

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Fulltext search initial implementation
..


Patch Set 4: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/72/ : 
UNSTABLE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I71887c2ea847e4488f4c98a11f8a5bcad02cac5a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Heri Ramampiaro 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-Reviewer: Michael Blow 
Gerrit-HasComments: No

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Till Westmann (Code Review)

Till Westmann has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 6: Code-Review+2

(1 comment)

Just one question.

https://asterix-gerrit.ics.uci.edu/#/c/990/6/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/util/ExpressionToVariableUtil.java
File 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/util/ExpressionToVariableUtil.java:

Line 42: return null;
This seems to introduce the need for a number of null checks (and potential 
NPEs if we miss one). Would it make sense to introduce a special singleton? Or 
would that just replace all null checks with other checks?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: Yes

Change in asterixdb[master]: Remove NEST from the syntax.

2016-07-13 Thread Yingyi Bu (Code Review)

Yingyi Bu has submitted this change and it was merged.

Change subject: Remove NEST from the syntax.
..


Remove NEST from the syntax.

Change-Id: I0e982523fc3fbf916ae4dd1b14e64ecf12844bcb
Reviewed-on: https://asterix-gerrit.ics.uci.edu/992
Sonar-Qube: Jenkins 
Tested-by: Jenkins 
Reviewed-by: Till Westmann 
Integration-Tests: Jenkins 
---
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
2 files changed, 6 insertions(+), 34 deletions(-)

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



diff --git a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html 
b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
index 2f7b7c6..c651146 100644
--- a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
+++ b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
@@ -76,7 +76,6 @@
 | LIKE: "like"
 | LIMIT: "limit"
 | LOAD: "load"
-| NEST: "nest"
 | NODEGROUP: "nodegroup"
 | NGRAM: "ngram"
 | NOT: "not"
@@ -845,7 +844,7 @@
 
 FromTerm
 ::=
-Expression ( ( AS 
)? Variable )? ( AT Variable )? ( ( JoinType )? ( JoinClause | NestClause | UnnestClause ) )*
+Expression ( ( AS 
)? Variable )? ( AT Variable )? ( ( JoinType )? ( JoinClause | UnnestClause ) )*
 
 
 JoinClause
@@ -853,12 +852,7 @@
 JOIN Expression ( 
( AS )? Variable )? ( AT Variable )? ON Expression
 
 
-NestClause
-::=
-NEST Expression ( 
( AS )? Variable )? ( AT Variable )? ON Expression
-
-
-UnnestClause
+UnnestClause
 ::=
 ( UNNEST | CORRELATE | 
FLATTEN ) Expression ( ( AS )? Variable ) ( AT Variable 
)?
 
@@ -870,7 +864,7 @@
 
 LetClause
 ::=
-( ( LET | LETTING ) LetElement ( COMMA LetElement )* | WITH WithElement ( COMMA WithElement )* )
+( ( LET | LETTING ) LetElement ( COMMA LetElement )* | WITH WithElement ( COMMA WithElement )* )
 
 
 WhereClause
@@ -903,12 +897,12 @@
 ( ( SOME ) | ( EVERY ) ) Variable IN Expression ( 
COMMA Variable IN Expression )* SATISFIES Expression
 
 
-LetElement
+LetElement
 ::=
 Variable EQ Expression
 
 
-WithElement
+WithElement
 ::=
 Variable AS Expression
 
diff --git a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj 
b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
index 9f62ef2..dff69ab 100644
--- a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
+++ b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
@@ -130,7 +130,6 @@
 import org.apache.asterix.lang.sqlpp.clause.FromTerm;
 import org.apache.asterix.lang.sqlpp.clause.HavingClause;
 import org.apache.asterix.lang.sqlpp.clause.JoinClause;
-import org.apache.asterix.lang.sqlpp.clause.NestClause;
 import org.apache.asterix.lang.sqlpp.clause.Projection;
 import org.apache.asterix.lang.sqlpp.clause.SelectBlock;
 import org.apache.asterix.lang.sqlpp.clause.SelectClause;
@@ -2415,10 +2414,7 @@
AbstractBinaryCorrelateClause correlateClause = null;
  }
  (correlateClause = JoinClause(joinType)
-  |
-  correlateClause = NestClause(joinType)
-  |
-  correlateClause = UnnestClause(joinType)
+  | correlateClause = UnnestClause(joinType)
  )
  {
 correlateClauses.add(correlateClause);
@@ -2446,23 +2442,6 @@
 rightVar = ExpressionToVariableUtil.getGeneratedVariable(rightExpr);
 }
 return new JoinClause(joinType, rightExpr, rightVar, posVar, 
conditionExpr);
-  }
-}
-
-NestClause NestClause(JoinType joinType) throws ParseException :
-{
-Expression rightExpr = null;
-VariableExpr rightVar = null;
-VariableExpr posVar = null;
-Expression conditionExpr = null;
-}
-{
-   rightExpr = Expression() (()? rightVar = Variable())? ( 
posVar = Variable())?   conditionExpr = Expression()
-  {
-if(rightVar==null){
-rightVar = ExpressionToVariableUtil.getGeneratedVariable(rightExpr);
-}
-return new NestClause(joinType, rightExpr, rightVar, posVar, 
conditionExpr);
   }
 }
 
@@ -2845,7 +2824,6 @@
   | 
   | 
   | 
-  | 
   | 
   | 
   | 

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

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

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Fulltext search initial implementation
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I71887c2ea847e4488f4c98a11f8a5bcad02cac5a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Remove NEST from the syntax.

2016-07-13 Thread Till Westmann (Code Review)

Till Westmann has posted comments on this change.

Change subject: Remove NEST from the syntax.
..


Patch Set 1: Code-Review+2

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

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

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Fulltext search initial implementation
..


Patch Set 4:

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/989
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I71887c2ea847e4488f4c98a11f8a5bcad02cac5a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Fulltext search initial implementation
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I71887c2ea847e4488f4c98a11f8a5bcad02cac5a
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Remove NEST from the syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Remove NEST from the syntax.
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0e982523fc3fbf916ae4dd1b14e64ecf12844bcb
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Remove NEST from the syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Remove NEST from the syntax.
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0e982523fc3fbf916ae4dd1b14e64ecf12844bcb
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Remove NEST from the syntax.

2016-07-13 Thread Yingyi Bu (Code Review)

Yingyi Bu has uploaded a new change for review.

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

Change subject: Remove NEST from the syntax.
..

Remove NEST from the syntax.

Change-Id: I0e982523fc3fbf916ae4dd1b14e64ecf12844bcb
---
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
2 files changed, 6 insertions(+), 34 deletions(-)


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

diff --git a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html 
b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
index 2f7b7c6..c651146 100644
--- a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
+++ b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.html
@@ -76,7 +76,6 @@
 | LIKE: "like"
 | LIMIT: "limit"
 | LOAD: "load"
-| NEST: "nest"
 | NODEGROUP: "nodegroup"
 | NGRAM: "ngram"
 | NOT: "not"
@@ -845,7 +844,7 @@
 
 FromTerm
 ::=
-Expression ( ( AS 
)? Variable )? ( AT Variable )? ( ( JoinType )? ( JoinClause | NestClause | UnnestClause ) )*
+Expression ( ( AS 
)? Variable )? ( AT Variable )? ( ( JoinType )? ( JoinClause | UnnestClause ) )*
 
 
 JoinClause
@@ -853,12 +852,7 @@
 JOIN Expression ( 
( AS )? Variable )? ( AT Variable )? ON Expression
 
 
-NestClause
-::=
-NEST Expression ( 
( AS )? Variable )? ( AT Variable )? ON Expression
-
-
-UnnestClause
+UnnestClause
 ::=
 ( UNNEST | CORRELATE | 
FLATTEN ) Expression ( ( AS )? Variable ) ( AT Variable 
)?
 
@@ -870,7 +864,7 @@
 
 LetClause
 ::=
-( ( LET | LETTING ) LetElement ( COMMA LetElement )* | WITH WithElement ( COMMA WithElement )* )
+( ( LET | LETTING ) LetElement ( COMMA LetElement )* | WITH WithElement ( COMMA WithElement )* )
 
 
 WhereClause
@@ -903,12 +897,12 @@
 ( ( SOME ) | ( EVERY ) ) Variable IN Expression ( 
COMMA Variable IN Expression )* SATISFIES Expression
 
 
-LetElement
+LetElement
 ::=
 Variable EQ Expression
 
 
-WithElement
+WithElement
 ::=
 Variable AS Expression
 
diff --git a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj 
b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
index 9f62ef2..dff69ab 100644
--- a/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
+++ b/asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
@@ -130,7 +130,6 @@
 import org.apache.asterix.lang.sqlpp.clause.FromTerm;
 import org.apache.asterix.lang.sqlpp.clause.HavingClause;
 import org.apache.asterix.lang.sqlpp.clause.JoinClause;
-import org.apache.asterix.lang.sqlpp.clause.NestClause;
 import org.apache.asterix.lang.sqlpp.clause.Projection;
 import org.apache.asterix.lang.sqlpp.clause.SelectBlock;
 import org.apache.asterix.lang.sqlpp.clause.SelectClause;
@@ -2415,10 +2414,7 @@
AbstractBinaryCorrelateClause correlateClause = null;
  }
  (correlateClause = JoinClause(joinType)
-  |
-  correlateClause = NestClause(joinType)
-  |
-  correlateClause = UnnestClause(joinType)
+  | correlateClause = UnnestClause(joinType)
  )
  {
 correlateClauses.add(correlateClause);
@@ -2446,23 +2442,6 @@
 rightVar = ExpressionToVariableUtil.getGeneratedVariable(rightExpr);
 }
 return new JoinClause(joinType, rightExpr, rightVar, posVar, 
conditionExpr);
-  }
-}
-
-NestClause NestClause(JoinType joinType) throws ParseException :
-{
-Expression rightExpr = null;
-VariableExpr rightVar = null;
-VariableExpr posVar = null;
-Expression conditionExpr = null;
-}
-{
-   rightExpr = Expression() (()? rightVar = Variable())? ( 
posVar = Variable())?   conditionExpr = Expression()
-  {
-if(rightVar==null){
-rightVar = ExpressionToVariableUtil.getGeneratedVariable(rightExpr);
-}
-return new NestClause(joinType, rightExpr, rightVar, posVar, 
conditionExpr);
   }
 }
 
@@ -2845,7 +2824,6 @@
   | 
   | 
   | 
-  | 
   | 
   | 
   | 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0e982523fc3fbf916ae4dd1b14e64ecf12844bcb
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu

Change in incubator-asterixdb-site[asf-site]: Update pmc/committers

2016-07-13 Thread Till Westmann (Code Review)

Till Westmann has submitted this change and it was merged.

Change subject: Update pmc/committers
..


Update pmc/committers

Change-Id: I9bf7acf231f3869affae1a7fdd028a46ea9b2bcf
Reviewed-on: https://asterix-gerrit.ics.uci.edu/987
Reviewed-by: Yingyi Bu 
---
M community.md
M content/community.html
2 files changed, 70 insertions(+), 17 deletions(-)

Approvals:
  Yingyi Bu: Looks good to me, approved



diff --git a/community.md b/community.md
index 3e6b50c..0755fbc 100644
--- a/community.md
+++ b/community.md
@@ -29,31 +29,33 @@
 
 
 
- Committers
+ PMC + Committers
 
 {:.table .table-striped}
 || Name
  | Organization   | Apache ID  |
 
||---|||
 | | 
Abdullah Alamoudi | Couchbase  | amoudi |
+| | Ate 
Douma | Hippo  | ate|
 | | Cameron 
Samak | UC Irvine  | csamak |
 |   | Chen Li 
  | UC Irvine  | chenli |
 | | Chris 
Hillery | Couchbase  | ceej   |
+| | Chris 
Mattmann| NASA JPL   | mattmann   |
 |   | Heri 
Ramampiaro   | NTNU Trondheim | heri   |
 | | Ian 
Maxon | UC Irvine  | imaxon |
 |  | Ildar 
Absalyamov  | UC Riverside   | ildar  |
-| | Inci 
Cetindil | UC Irvine  | icetindil  |
 |  | 
Jianfeng Jia  | UC Irvine  | jianfeng   |
+| | Jochen 
Wiedmann   | Software AG| jochen |
 || Keren 
Ouaknine| Hebrew University  | kereno |
-| | Markus 
Dreseler   | HPI| mrks   |
 |  | Michael 
Blow  | Couchbase  | mblow  |
 | | Michael 
Carey | UC Irvine  | dtabass|
-|   | 
Murtadha Hubail   | UC Irvine  | mhubail|
+|   | 
Murtadha Hubail   | Saudi Aramco   | mhubail|
 |   | Pouria 
Pirzadeh   | UC Irvine  | pouria |
 | | Preston 
Carman| UC Riverside   | prestonc   |
 |  | Raman 
Grover  | Microsoft Research | raman  |
 |  | Sattam 
Alsubaiee  | KACST Saudi Arabia | salsubaiee |
 | | Steven 
Jacobs | UC Riverside   | sjaco002   |
 || Taewoo 
Kim| UC Irvine  | wangsaeu   |
+| | Ted 
Dunning   | MapR   | tdunning   |
 | | Till 
Westmann | Couchbase  | tillw  |
 |  | 
Vassilis Tsotras  | UC Riverside   | tsotras|
 || Vinayak 
Borkar| X15 Software   | vinayakb   |
@@ -63,6 +65,16 @@
 
 
 
+ Committers
+
+{:.table .table-striped}
+|| Name
  | Organization   | Apache ID  |
+||---|||
+| | Inci 
Cetindil | UC Irvine  | icetindil  |
+| | Markus 
Dreseler   | HPI| mrks   |
+
+
+
  Contributors
 
 {:.table .table-striped}
diff --git a/content/community.html b/content/community.html
index 7c7f599..c95bb47 100644
--- a/content/community.html
+++ b/content/community.html
@@ -162,7 +162,7 @@
 
 
 
-Committers
+PMC + Committers
 
 
   
@@ -179,6 +179,12 @@
   Abdullah Alamoudi
   Couchbase
   amoudi
+
+
+  
+  Ate Douma
+  Hippo
+  ate
 
 
   
@@ -199,6 +205,12 @@
   ceej
 
 
+  
+  Chris Mattmann
+  NASA JPL
+  mattmann
+
+
   
   Heri Ramampiaro
   NTNU Trondheim
@@ -217,28 +229,22 @@
   ildar
 
 
-  
-  Inci Cetindil
-  UC Irvine
-  icetindil
-
-
   
   Jianfeng Jia
   UC Irvine
   jianfeng
 
 
+  
+  Jochen Wiedmann
+  Software AG
+  jochen
+
+
   
   Keren Ouaknine
   Hebrew University
   kereno
-
-
-  
-  Markus Dreseler
-  HPI
-  mrks
 
 
   
@@ -295,6 +301,12 @@
   wangsaeu
 
 
+  
+  Ted Dunning
+  MapR
+  tdunning
+
+
   
   Till Westmann
   Couchbase
@@ -335,6 +347,35 @@
 
 
 
+Committers
+
+
+  
+
+   
+  Name
+  Organization
+  Apache ID
+
+  
+  
+
+  
+  Inci Cetindil
+  UC Irvine
+  icetindil
+
+
+  
+  Markus Dreseler
+  HPI
+  mrks
+
+  
+
+
+
+
 Contributors
 
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9bf7acf231f3869affae1a7fdd028a46ea9b2bcf
Gerrit-PatchSet: 3
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 6: Integration-Tests+1

Integration Tests Successful

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

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

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

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 6:

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

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

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

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Support SQL-compliant group-by syntax.
..


Patch Set 6:

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

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

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

Change in asterixdb[master]: Support SQL-compliant group-by syntax.

2016-07-13 Thread Yingyi Bu (Code Review)

Hello Jenkins,

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

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

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

Change subject: Support SQL-compliant group-by syntax.
..

Support SQL-compliant group-by syntax.

1. Add AST-level deep equality and expression subsutitions;
2. Fix DeepCopyVisitor and InlineColumnAliasVisitor for edge cases;
3. Add deep equality tests;
4. Add group-by tests for the SQL-compliant syntax.

Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f
---
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/sqlpp/ParserTestExecutor.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dapd/q2-6/q2-6.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dapd/q2-6/q2-6.2.update.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dapd/q2-6/q2-6.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr-2/gby-expr-2.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr-2/gby-expr-2.2.update.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr-2/gby-expr-2.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr-3/gby-expr-3.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr-3/gby-expr-3.2.update.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr-3/gby-expr-3.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr/gby-expr.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr/gby-expr.2.update.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/gby-expr/gby-expr.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-01-2/sugar-01-2.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-01-2/sugar-01-2.2.update.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-01-2/sugar-01-2.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-02-2/sugar-02-2.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-02-2/sugar-02-2.2.update.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-02-2/sugar-02-2.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-02-2/sugar-02-2.4.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-03-2/sugar-03-2.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-03-2/sugar-03-2.2.update.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-03-2/sugar-03-2.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-04-2/sugar-04-2.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-04-2/sugar-04-2.2.update.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-04-2/sugar-04-2.3.query.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-05-2/sugar-05-2.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-05-2/sugar-05-2.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/group-by/sugar-05-2/sugar-05-2.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/tpch-sql-sugar/q01_pricing_summary_report_2/q01_pricing_summary_report_2.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/tpch-sql-sugar/q01_pricing_summary_report_2/q01_pricing_summary_report_2.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/tpch-sql-sugar/q01_pricing_summary_report_2/q01_pricing_summary_report_2.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/group-by/gby-expr-2/gby-expr-2.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/group-by/gby-expr-3/gby-expr-3.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/group-by/gby-expr/gby-expr.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/clause/GroupbyClause.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/clause/LetClause.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/clause/LimitClause.java
M

Change in incubator-asterixdb-site[asf-site]: Update pmc/committers

2016-07-13 Thread Yingyi Bu (Code Review)

Yingyi Bu has posted comments on this change.

Change subject: Update pmc/committers
..


Patch Set 2: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9bf7acf231f3869affae1a7fdd028a46ea9b2bcf
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update pmc/committers

2016-07-13 Thread Till Westmann (Code Review)

Till Westmann has posted comments on this change.

Change subject: Update pmc/committers
..


Patch Set 1:

(2 comments)

https://asterix-gerrit.ics.uci.edu/#/c/987/1/community.md
File community.md:

Line 73: | | 
Inci Cetindil | UC Irvine  | icetindil  |
> I don't find this apache id here: http://people.apache.org/committer-index.
Indeed it seems that the ICLA was filed, but the account wasn't created. Need 
to find out what that means.


Line 74: | | 
Markus Dreseler   | HPI| mrks   |
> I don't find mrks listed as asterixdb committer here: http://people.apache.
Hmm, he does have an ICLA and a Apache account, I'm not completely sure, if the 
phonebook lists committers or PMC members ...


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9bf7acf231f3869affae1a7fdd028a46ea9b2bcf
Gerrit-PatchSet: 1
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: Yes

Change in incubator-asterixdb-site[asf-site]: Update pmc/committers

2016-07-13 Thread Jenkins (Code Review)

Jenkins has posted comments on this change.

Change subject: Update pmc/committers
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9bf7acf231f3869affae1a7fdd028a46ea9b2bcf
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in incubator-asterixdb-site[asf-site]: Update pmc/committers

2016-07-13 Thread Till Westmann (Code Review)

Till Westmann has uploaded a new patch set (#2).

Change subject: Update pmc/committers
..

Update pmc/committers

Change-Id: I9bf7acf231f3869affae1a7fdd028a46ea9b2bcf
---
M community.md
M content/community.html
2 files changed, 70 insertions(+), 17 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/incubator-asterixdb-site 
refs/changes/87/987/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/987
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9bf7acf231f3869affae1a7fdd028a46ea9b2bcf
Gerrit-PatchSet: 2
Gerrit-Project: incubator-asterixdb-site
Gerrit-Branch: asf-site
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi

Change in asterixdb[master]: ASTERIXDB-1269 Fix Extraction of Query Segment and Refactor ...

2016-07-13 Thread abdullah alamoudi (Code Review)

abdullah alamoudi has submitted this change and it was merged.

Change subject: ASTERIXDB-1269 Fix Extraction of Query Segment and Refactor AQL 
Parser
..


ASTERIXDB-1269 Fix Extraction of Query Segment and Refactor AQL Parser

This change fixes the issue ASTERIXDB-1269 and refactor aql.jj
removing qouted strings throughout the file and creating token's
definitions for them instead

Change-Id: I18ac4f8d86b3c5c7bfe226c98114499671649e93
Reviewed-on: https://asterix-gerrit.ics.uci.edu/988
Sonar-Qube: Jenkins 
Tested-by: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Till Westmann 
---
M asterixdb/asterix-app/pom.xml
M 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/query-issue697.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/aggregate-sql/sum_null-with-pred/sum_null-with-pred.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/aggregate/sum_null-with-pred/sum_null-with-pred.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/boolean/not_01/not_01.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/quantifiers/everysat_04/everysat_04.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/quantifiers/somesat_06/somesat_06.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/semistructured/has-param1/has-param1.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring-after-5/substring-after-5.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring-after-6/substring-after-6.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/tinysocial/tinysocial-suite/tinysocial-suite.13.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/tpch-sql-like/q13_customer_distribution/q13_customer_distribution.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/tpch-sql-like/q16_parts_supplier_relationship/q16_parts_supplier_relationship.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/tpch/q13_customer_distribution/q13_customer_distribution.3.query.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/tpch/q16_parts_supplier_relationship/q16_parts_supplier_relationship.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/user-defined-functions/single-line-definition/single-line-definition.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/user-defined-functions/single-line-definition/single-line-definition.2.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/user-defined-functions/single-line-definition/single-line-definition.3.ddl.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/user-defined-functions/udf18/udf18.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/user-defined-functions/single-line-definition/single-line-definition.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/user-defined-functions/single-line-definition/single-line-definition.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/user-defined-functions/single-line-definition/single-line-definition.3.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/user-defined-functions/single-line-definition/single-line-definition.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite.xml
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M asterixdb/asterix-lang-aql/src/main/javacc/AQL.jj
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/parser/ScopeChecker.java
27 files changed, 396 insertions(+), 123 deletions(-)

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



diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml
index 43adabf..7d0a9ba 100644
--- a/asterixdb/asterix-app/pom.xml
+++ b/asterixdb/asterix-app/pom.xml
@@ -145,6 +145,35 @@
 
   
 
+
+
+
+
+org.eclipse.m2e
+lifecycle-mapping
+1.0.0
+
+
+
+
+
+org.apache.asterix
+
asterix-test-datagenerator-maven-plugin
+
[0.8.9-SNAPSHOT,)
+
+generate-testdata
+
+
+
+
+

37 matches

Mail list logo