Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..


Patch Set 5: Integration-Tests-1

Integration Tests Timed Out

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1571: fix and add regression tests.
..


Patch Set 2: Integration-Tests-1

Integration Tests Timed Out

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

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

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

Change in asterixdb[master]: ASTERIXDB-1375 - New Query UI implementation

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1375 - New Query UI implementation
..


Patch Set 13:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9aa6fb0a6bebaaa58f288e621dbfe76a604c689
Gerrit-PatchSet: 13
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Kaveen Rodrigo 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1375 - New Query UI implementation

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1375 - New Query UI implementation
..


Patch Set 13:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9aa6fb0a6bebaaa58f288e621dbfe76a604c689
Gerrit-PatchSet: 13
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Kaveen Rodrigo 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1375 - New Query UI implementation

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1375 - New Query UI implementation
..


Patch Set 13:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9aa6fb0a6bebaaa58f288e621dbfe76a604c689
Gerrit-PatchSet: 13
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Kaveen Rodrigo 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1375 - New Query UI implementation

[Kaveen Rodrigo (Code Review)]

Hello Ian Maxon, Jenkins,

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

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

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

Change subject: ASTERIXDB-1375 - New Query UI implementation
..

ASTERIXDB-1375 - New Query UI implementation

Implemented UI using bootstrap according to specification
Wrote the client side logic using Angular JS
Using CodeMirror and custom mode to do syntax highlighting.
Modified HTTP API to support CORS.
Custom servlet to host new Web Ui on port 19006.
CHANGES
Changed resource folder name to "queryui"
Fixed SonarQube problems
Dynamic HTTP API port, instead of hardcoded value
... Jul 5 2016
Using JSONObject for Dynamic HTTP APi Port
Improved Animations
Added Collapsable side panal
Updated LICENSE file
... Aug 7 2016
Fixed Logger level to Error
Change-Id: Id9aa6fb0a6bebaaa58f288e621dbfe76a604c689
---
M .gitignore
M asterixdb/LICENSE
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/jobgen/QueryLogicalExpressionJobGen.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByIndexFieldAccessRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/CheckFilterExpressionTypeRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/FuzzyJoinRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/InlineUnnestFunctionRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceAutogenerateIDRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceDynamicTypeCastForExternalFunctionRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceDynamicTypeCastRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceEnforcedListTypeRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceStaticTypeCastForInsertRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/PushAggregateIntoGroupbyRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/RemoveRedundantListifyRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ReplaceSinkOpWithCommitOpRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/SetClosedRecordConstructorsRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/IntroduceJoinAccessMethodRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/InvertedIndexAccessMethod.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/subplan/InlineSubplanInputForNestedTupleSourceRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/typecast/StaticTypeCastUtil.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/ConstantHelper.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/TypeTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/ValidateUtil.java
M asterixdb/asterix-app/pom.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryWebInterfaceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/file/SecondaryIndexOperationsHelper.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
A asterixdb/asterix-app/src/main/resources/queryui/css/bootstrap-theme.min.css
A asterixdb/asterix-app/src/main/resources/queryui/css/bootstrap.min.css
A asterixdb/asterix-app/src/main/resources/queryui/css/codemirror.css
A asterixdb/asterix-app/src/main/resources/queryui/css/json-formatter.min.css
A 

Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..


Patch Set 5:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1571: fix and add regression tests.
..


Patch Set 2:

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

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

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

Change in asterixdb[master]: Index-only plan

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Index-only plan
..


Patch Set 26:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa02c13d4fddd880e1ee9e85eef6577301fb4560
Gerrit-PatchSet: 26
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Index-only plan

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Index-only plan
..


Patch Set 26:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa02c13d4fddd880e1ee9e85eef6577301fb4560
Gerrit-PatchSet: 26
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Wenhai Li (Code Review)]

Hello Jenkins,

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

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

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

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..

Initial fuzzy join framework with multiple split and temporary ngram fix.


Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/FuzzyUtils.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/AsterixExtractFunctionsFromJoinConditionRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/FuzzyJoinRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/subplan/InlineSubplanInputForNestedTupleSourceRule.java
A asterixdb/asterix-app/data/dblp-small/csx-small-multi-id.txt
A asterixdb/asterix-app/data/dblp-small/dblp-small-multi-id.txt
A asterixdb/asterix-app/data/pub-small/csx-small-multi-id.txt
A asterixdb/asterix-app/data/pub-small/dblp-small-multi-id.txt
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join-noeqjoin/ngram-jaccard-inline.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join-noeqjoin/word-jaccard-inline.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/leftouterjoin-probe-pidx-with-join-jaccard-check-idx_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-fuzzyeq-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-fuzzyeq-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-fuzzyeq-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-fuzzyeq-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-fuzzyeq-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-fuzzyeq-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check-after-btree-access.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/leftouterjoin-probe-pidx-with-join-jaccard-check-idx_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/ngram-fuzzyeq-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/ngram-jaccard-check_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/ngram-jaccard-inline.plan
M 

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1571: fix and add regression tests.
..


Patch Set 2:

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

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

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1571: fix and add regression tests.
..


Patch Set 2:

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

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

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

Change in asterixdb[master]: Index-only plan

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Index-only plan
..


Patch Set 25:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa02c13d4fddd880e1ee9e85eef6577301fb4560
Gerrit-PatchSet: 25
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Yingyi Bu (Code Review)]

Hello Jenkins,

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

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

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

Change subject: ASTERIXDB-1571: fix and add regression tests.
..

ASTERIXDB-1571: fix and add regression tests.

- also fix the left outer type propagation policy.

Change-Id: I45d6dbabdef709c9ba3cb883412301ab7fd69a85
---
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/DeepCopyVisitor.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppFormatPrintVisitor.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/TypePropagationPolicy.java
20 files changed, 392 insertions(+), 2 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I45d6dbabdef709c9ba3cb883412301ab7fd69a85
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 

Change in asterixdb[master]: Index-only plan

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Index-only plan
..


Patch Set 25:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa02c13d4fddd880e1ee9e85eef6577301fb4560
Gerrit-PatchSet: 25
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Index-only plan

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Index-only plan
..


Patch Set 24:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa02c13d4fddd880e1ee9e85eef6577301fb4560
Gerrit-PatchSet: 24
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1571: fix and add regression tests.
..


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

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

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1571: fix and add regression tests.
..


Patch Set 1:

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

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

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

Change in asterixdb[master]: ASTERIXDB-1571: fix and add regression tests.

[Yingyi Bu (Code Review)]

Yingyi Bu has uploaded a new change for review.

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

Change subject: ASTERIXDB-1571: fix and add regression tests.
..

ASTERIXDB-1571: fix and add regression tests.

- also fix the left outer type propagation policy.

Change-Id: I45d6dbabdef709c9ba3cb883412301ab7fd69a85
---
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.1.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.2.update.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.3.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/query-ASTERIXDB-1571-3/query-ASTERIXDB-1571-3.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/query-ASTERIXDB-1571-4/query-ASTERIXDB-1571-4.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/query-ASTERIXDB-1571/query-ASTERIXDB-1571.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/DeepCopyVisitor.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppFormatPrintVisitor.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/TypePropagationPolicy.java
20 files changed, 391 insertions(+), 2 deletions(-)


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

diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.1.ddl.sqlpp
 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.1.ddl.sqlpp
new file mode 100644
index 000..07c39e3
--- /dev/null
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.1.ddl.sqlpp
@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+drop dataverse test if exists;
+create dataverse test;
+use test;
+
+drop dataset samptable if exists;
+drop type samptabletype if exists;
+
+create type samptabletype as closed {
+  id: int8
+};
+
+create dataset samptable(samptabletype) primary key id;
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.2.update.sqlpp
 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/query-ASTERIXDB-1571-2/query-ASTERIXDB-1571-2.2.update.sqlpp
new file mode 100644
index 000..d1ad8b5
--- /dev/null
+++ 

Change in asterixdb[master]: ConstantExpressionUtil improvements

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ConstantExpressionUtil improvements
..


Patch Set 1:

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

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

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

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ConstantExpressionUtil improvements

[Yingyi Bu (Code Review)]

Yingyi Bu has posted comments on this change.

Change subject: ConstantExpressionUtil improvements
..


Patch Set 1: Code-Review+2

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

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

Change in asterixdb[master]: ConstantExpressionUtil improvements

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ConstantExpressionUtil improvements
..


Patch Set 1:

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

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

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

Change in asterixdb[master]: ConstantExpressionUtil improvements

[Till Westmann (Code Review)]

Till Westmann has uploaded a new change for review.

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

Change subject: ConstantExpressionUtil improvements
..

ConstantExpressionUtil improvements

Change-Id: I636538d6ea6c6a527aac1eb9cacdcc40a8489c02
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByIndexFieldAccessRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/PushFieldAccessRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/UnnestToDataScanRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/functions/MetadataBuiltinFunctions.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/ClosedRecordConstructorResultType.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/OpenRecordConstructorResultType.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/RecordAddFieldsTypeComputer.java
R 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/util/ConstantExpressionUtil.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/formats/NonTaggedDataFormat.java
11 files changed, 53 insertions(+), 110 deletions(-)


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

diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByIndexFieldAccessRule.java
 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByIndexFieldAccessRule.java
index 6e8bd69..3ba7084 100644
--- 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByIndexFieldAccessRule.java
+++ 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByIndexFieldAccessRule.java
@@ -23,10 +23,9 @@
 import java.util.Collections;
 
 import org.apache.asterix.algebra.base.AsterixOperatorAnnotations;
+import org.apache.asterix.om.util.ConstantExpressionUtil;
 import org.apache.asterix.lang.common.util.FunctionUtil;
 import org.apache.asterix.om.base.AInt32;
-import org.apache.asterix.om.base.AString;
-import org.apache.asterix.om.base.IAObject;
 import org.apache.asterix.om.constants.AsterixConstantValue;
 import org.apache.asterix.om.functions.AsterixBuiltinFunctions;
 import org.apache.asterix.om.types.ARecordType;
@@ -141,7 +140,7 @@
 @SuppressWarnings("unchecked")
 private static ILogicalExpression createFieldAccessByIndex(ARecordType 
recType,
 AbstractFunctionCallExpression fce) {
-String s = getStringSecondArgument(fce);
+String s = ConstantExpressionUtil.getStringArgument(fce, 1);
 if (s == null) {
 return null;
 }
@@ -151,22 +150,6 @@
 }
 return new ScalarFunctionCallExpression(
 
FunctionUtil.getFunctionInfo(AsterixBuiltinFunctions.FIELD_ACCESS_BY_INDEX), 
fce.getArguments().get(0),
-new MutableObject(new 
ConstantExpression(new AsterixConstantValue(new AInt32(k);
-}
-
-private static String 
getStringSecondArgument(AbstractFunctionCallExpression expr) {
-ILogicalExpression e2 = expr.getArguments().get(1).getValue();
-if (e2.getExpressionTag() != LogicalExpressionTag.CONSTANT) {
-return null;
-}
-ConstantExpression c = (ConstantExpression) e2;
-if (!(c.getValue() instanceof AsterixConstantValue)) {
-return null;
-}
-IAObject v = ((AsterixConstantValue) c.getValue()).getObject();
-if (v.getType().getTypeTag() != ATypeTag.STRING) {
-return null;
-}
-return ((AString) v).getStringValue();
+new MutableObject<>(new ConstantExpression(new 
AsterixConstantValue(new AInt32(k);
 }
 }
diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
index 9d9dda0..fec564e 100644
--- 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
+++ 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
@@ -23,6 +23,7 @@
 import java.nio.ByteBuffer;
 import java.util.List;
 
+import org.apache.asterix.om.util.ConstantExpressionUtil;
 import org.apache.asterix.common.config.GlobalConfig;
 import org.apache.asterix.dataflow.data.common.AqlExpressionTypeComputer;
 import org.apache.asterix.dataflow.data.nontagged.AqlMissingWriterFactory;
@@ -35,7 +36,6 @@
 import 

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Taewoo Kim (Code Review)]

Hello Jenkins,

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

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

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

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..

ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths 
handling

 - ASTERIXDB-1081: Fixed RemoveUnusedAssignAndAggregateRule to reflect multiple 
paths in the plan.
 - ASTERIXDB-1086: Fixed IntroduceProjectsRule to reflect multiples paths in 
the plan.
 - ASTERIXDB-1246: Fixed RemoveRedundantGroupByDecorVarsRule to remove decor 
variables
   before IntroduceProjects rule fires.

Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1020.plan
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/IntroduceProjectsRule.java
D 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVars.java
A 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVarsRule.java
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveUnusedAssignAndAggregateRule.java
M 
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/StorageUtil.java
8 files changed, 439 insertions(+), 198 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/73/1073/4
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1073
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


Patch Set 3: Integration-Tests-1

Integration Tests Timed Out

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

[jira] [Updated] (ASTERIXDB-1572) Multiple and/or conditions in where clause inside a subquery in from clause throws AlgebricksException

[Till (JIRA)]

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

Till updated ASTERIXDB-1572:

Assignee: Yingyi Bu

> Multiple and/or conditions in where clause inside a subquery in from clause 
> throws AlgebricksException
> --
>
> Key: ASTERIXDB-1572
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1572
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Master branch with latest commit id: 
> bd3a535b171a5cc7486175
>Reporter: Vignesh Raghunathan
>Assignee: Yingyi Bu
>
> To reproduce the issue run the following sqlpp statements:
> {code}
> drop dataverse sampdb if exists;
> create dataverse sampdb;
> use sampdb;
> drop dataset samptable if exists;
> drop type samptabletype if exists;
> create type samptabletype as closed {
>   id: int8
> };
> create dataset samptable(samptabletype) primary key id;
> insert into samptable ({'id' : 0});
> insert into samptable ({'id' : 1});
> select *
> from
> (
> select id from samptable
> where (id in [0] and id in [1]) 
> or (id in [1] and id in [2]) 
> ) st1;
> {code}
> Result:
> {code}
> org.apache.hyracks.algebricks.common.exceptions.AlgebricksException: 
> Attempting to construct a nested plan with 4 operator descriptors. Currently, 
> nested plans can only consist in linear pipelines of Asterix micro operators.
>   at 
> org.apache.hyracks.algebricks.core.algebra.operators.physical.AbstractPhysicalOperator.buildPipelineWithProjection(AbstractPhysicalOperator.java:144)
>   at 
> org.apache.hyracks.algebricks.core.algebra.operators.physical.AbstractPhysicalOperator.compileSubplans(AbstractPhysicalOperator.java:125)
>   at 
> org.apache.hyracks.algebricks.core.algebra.operators.physical.SubplanPOperator.contributeRuntimeOperator(SubplanPOperator.java:92)
>   at 
> org.apache.hyracks.algebricks.core.algebra.operators.logical.AbstractLogicalOperator.contributeRuntimeOperator(AbstractLogicalOperator.java:166)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:98)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> org.apache.hyracks.algebricks.core.jobgen.impl.PlanCompiler.compileOpRef(PlanCompiler.java:85)
>   at 
> 

Change in asterixdb[master]: small refactoring

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: small refactoring
..


Patch Set 3:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I37eab1645416e3aad6119bba527c5e3b4b98fddc
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: small refactoring

[Till Westmann (Code Review)]

Hello Jenkins,

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

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

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

Change subject: small refactoring
..

small refactoring

Change-Id: I37eab1645416e3aad6119bba527c5e3b4b98fddc
---
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/dataflow/FeedRecordDataFlowController.java
1 file changed, 21 insertions(+), 15 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/52/1052/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1052
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I37eab1645416e3aad6119bba527c5e3b4b98fddc
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: abdullah alamoudi 

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


Patch Set 3:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


Patch Set 3:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


Patch Set 2:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..


Patch Set 3:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

[Taewoo Kim (Code Review)]

Hello Jenkins,

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

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

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

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple 
plan paths handling
..

ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths 
handling

 - ASTERIXDB-1081: Fixed RemoveUnusedAssignAndAggregateRule to reflect multiple 
paths in the plan.
 - ASTERIXDB-1086: Fixed IntroduceProjectsRule to reflect multiples paths in 
the plan.
 - ASTERIXDB-1246: Fixed RemoveRedundantGroupByDecorVarsRule to remove decor 
variables
   before IntroduceProjects rule fires.

Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1020.plan
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/IntroduceProjectsRule.java
D 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVars.java
A 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVarsRule.java
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveUnusedAssignAndAggregateRule.java
7 files changed, 439 insertions(+), 196 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/73/1073/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1073
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

[Taewoo Kim (Code Review)]

Taewoo Kim has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths 
in a plan can be handled.
..


Patch Set 2:

(6 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1073/2/hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/IntroduceProjectsRule.java
File 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/IntroduceProjectsRule.java:

Line 76: Set parentUsedVars = new 
HashSet();
> MAJOR SonarQube violation:
Done


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

Line 64: if (op.getOperatorTag() != LogicalOperatorTag.SINK) {
> MAJOR SonarQube violation:
Done


Line 72: boolean planTransformed = false;
> MAJOR SonarQube violation:
Done


Line 94: GroupByOperator groupByOp = (GroupByOperator) op;
> MAJOR SonarQube violation:
Done


https://asterix-gerrit.ics.uci.edu/#/c/1073/2/hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveUnusedAssignAndAggregateRule.java
File 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveUnusedAssignAndAggregateRule.java:

Line 68: Set survivedUnionSourceVarSet = new 
HashSet();
> MAJOR SonarQube violation:
Done


Line 298: case ASSIGN: {
> MAJOR SonarQube violation:
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: Yes

Change in asterixdb[master]: Add upsert option for feed

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add upsert option for feed
..


Patch Set 7: Integration-Tests-1

Integration Tests Timed Out

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Xikui Wang 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Add standalone performance test.

[Chris Hillery (Code Review)]

Chris Hillery has submitted this change and it was merged.

Change subject: Add standalone performance test.
..


Add standalone performance test.

Change-Id: Iaffcbd2f798402742740b8ac12f9cf41a52c
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1070
Tested-by: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Ian Maxon 
---
A 
asterixdb/asterix-experiments/src/main/java/org/apache/asterix/experiment/builder/PresetClusterPerfBuilder.java
M 
asterixdb/asterix-experiments/src/main/java/org/apache/asterix/experiment/client/LSMExperimentSetRunner.java
A 
asterixdb/asterix-experiments/src/main/resources/ingestion-experiment-binary-and-configs/aql/bench_3_load_2.aql
3 files changed, 145 insertions(+), 2 deletions(-)

Approvals:
  Ian Maxon: Looks good to me, approved
  Jenkins: Verified; Verified

Objections:
  Jenkins: Violations found



diff --git 
a/asterixdb/asterix-experiments/src/main/java/org/apache/asterix/experiment/builder/PresetClusterPerfBuilder.java
 
b/asterixdb/asterix-experiments/src/main/java/org/apache/asterix/experiment/builder/PresetClusterPerfBuilder.java
new file mode 100644
index 000..5c136d6
--- /dev/null
+++ 
b/asterixdb/asterix-experiments/src/main/java/org/apache/asterix/experiment/builder/PresetClusterPerfBuilder.java
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.asterix.experiment.builder;
+
+import org.apache.asterix.experiment.action.base.SequentialActionList;
+import org.apache.asterix.experiment.action.derived.RunAQLFileAction;
+import org.apache.asterix.experiment.action.derived.RunSQLPPFileAction;
+import org.apache.asterix.experiment.action.derived.SleepAction;
+import org.apache.asterix.experiment.action.derived.TimedAction;
+import org.apache.asterix.experiment.client.LSMExperimentConstants;
+import 
org.apache.asterix.experiment.client.LSMExperimentSetRunner.LSMExperimentSetRunnerConfig;
+import org.apache.asterix.experiment.client.LSMPerfConstants;
+import org.apache.http.client.HttpClient;
+import org.apache.http.impl.client.DefaultHttpClient;
+
+import javax.xml.bind.JAXBException;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.Scanner;
+
+/**
+ * This experiment simply runs the performance benchmark queries against
+ * a cluster which is presumed to already be running.
+ * Also, there is no orchestration server involved in this experiment builder.
+ *
+ * The only LSMExperimentSetRunner config command-line options used by this
+ * experiment are:
+ *   -ler  -rh  -rp
+ */
+public class PresetClusterPerfBuilder extends AbstractExperimentBuilder {
+
+protected final HttpClient httpClient;
+
+protected final String restHost;
+
+protected final int restPort;
+
+protected final Path localExperimentRoot;
+
+private final String countFileName;
+
+protected final String loadAQLFilePath;
+
+protected final String querySQLPPFileName;
+
+public PresetClusterPerfBuilder(LSMExperimentSetRunnerConfig config) {
+super("PresetClusterPerfBuilder");
+this.httpClient = new DefaultHttpClient();
+this.restHost = config.getRESTHost();
+this.restPort = config.getRESTPort();
+this.localExperimentRoot = Paths.get(config.getLocalExperimentRoot());
+this.countFileName = "bench_count.aql";
+this.loadAQLFilePath = "bench_3_load_2.aql";
+this.querySQLPPFileName = "agg_bench";
+}
+
+@Override
+protected void doBuild(Experiment e) throws IOException, JAXBException {
+SequentialActionList execs = new SequentialActionList();
+
+//ddl statements
+execs.add(new SleepAction(15000));
+// TODO: implement retry handler
+execs.add(new RunAQLFileAction(httpClient, restHost, restPort, 
localExperimentRoot.resolve(
+

Change in asterixdb[master]: Add standalone performance test.

[Chris Hillery (Code Review)]

Chris Hillery has posted comments on this change.

Change subject: Add standalone performance test.
..


Patch Set 1:

My thoughts exactly. :) Thanks.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaffcbd2f798402742740b8ac12f9cf41a52c
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Chris Hillery 
Gerrit-Reviewer: Chris Hillery 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Add standalone performance test.

[Ian Maxon (Code Review)]

Ian Maxon has posted comments on this change.

Change subject: Add standalone performance test.
..


Patch Set 1: Code-Review+2

Sonarqube comments seem kind of spurious to me, one is thinking ++ means code 
and on the others you're fixing a typo, so...

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaffcbd2f798402742740b8ac12f9cf41a52c
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Chris Hillery 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths 
in a plan can be handled.
..


Patch Set 2:

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

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

Change in asterixdb[master]: Index-only plan

[Taewoo Kim (Code Review)]

Taewoo Kim has posted comments on this change.

Change subject: Index-only plan
..


Patch Set 23:

(3 comments)

Thanks Till. I will breakdown this big change into smaller pieces as much as 
possible and upload them as a new patch.

https://asterix-gerrit.ics.uci.edu/#/c/744/23/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.java:

Line 280: ISearchOperationCallbackFactory searchCallbackFactory = 
null;
> MAJOR SonarQube violation:
Done


Line 289: ArrayBackedValueStorage castBuffer = null;
> MAJOR SonarQube violation:
Done


Line 289: ArrayBackedValueStorage castBuffer = null;
> MAJOR SonarQube violation:
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa02c13d4fddd880e1ee9e85eef6577301fb4560
Gerrit-PatchSet: 23
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: Young-Seok Kim 
Gerrit-HasComments: Yes

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths 
in a plan can be handled.
..


Patch Set 2:

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

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

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

Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..


Patch Set 4: Integration-Tests-1

Integration Tests Timed Out

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

[Taewoo Kim (Code Review)]

Hello Jenkins,

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

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

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

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths 
in a plan can be handled.
..

ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths in a plan can be 
handled.

 - ASTERIXDB-1081: Fixed RemoveUnusedAssignAndAggregateRule to reflect multiple 
paths in the plan.
 - ASTERIXDB-1086: Fixed IntroduceProjectsRule to reflect multiples paths in 
the plan.
 - ASTERIXDB-1246: Fixed RemoveRedundantGroupByDecorVarsRule to remove decor 
variables
   before IntroduceProjects rule fires.

Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1020.plan
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/IntroduceProjectsRule.java
D 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVars.java
A 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVarsRule.java
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveUnusedAssignAndAggregateRule.java
7 files changed, 432 insertions(+), 180 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths 
in a plan can be handled.
..


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

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

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

[Taewoo Kim (Code Review)]

Taewoo Kim has posted comments on this change.

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths 
in a plan can be handled.
..


Patch Set 1:

This patch set is uploaded as a subset of Index-only plan.

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

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

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

[Taewoo Kim (Code Review)]

Taewoo Kim has uploaded a new change for review.

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

Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths 
in a plan can be handled.
..

ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths in a plan can be 
handled.

 - ASTERIXDB-1081: Fixed RemoveUnusedAssignAndAggregateRule to reflect multiple 
paths in the plan.
 - ASTERIXDB-1086: Fixed IntroduceProjectsRule to reflect multiples paths in 
the plan.
 - ASTERIXDB-1246: Fixed RemoveRedundantGroupByDecorVarsRule to remove decor 
variables
   before IntroduceProjects rule fires.

Change-Id: I69e055572f024f28a857d4e64916b4806e63c083
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1020.plan
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/IntroduceProjectsRule.java
D 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVars.java
A 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveRedundantGroupByDecorVarsRule.java
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/RemoveUnusedAssignAndAggregateRule.java
7 files changed, 432 insertions(+), 180 deletions(-)


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

diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
index eb4751d..cf3fb8a 100644
--- 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
+++ 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
@@ -112,7 +112,7 @@
 import 
org.apache.hyracks.algebricks.rewriter.rules.PushUnnestDownThroughUnionRule;
 import org.apache.hyracks.algebricks.rewriter.rules.ReinferAllTypesRule;
 import 
org.apache.hyracks.algebricks.rewriter.rules.RemoveCartesianProductWithEmptyBranchRule;
-import 
org.apache.hyracks.algebricks.rewriter.rules.RemoveRedundantGroupByDecorVars;
+import 
org.apache.hyracks.algebricks.rewriter.rules.RemoveRedundantGroupByDecorVarsRule;
 import 
org.apache.hyracks.algebricks.rewriter.rules.RemoveRedundantVariablesRule;
 import 
org.apache.hyracks.algebricks.rewriter.rules.RemoveUnnecessarySortMergeExchange;
 import 
org.apache.hyracks.algebricks.rewriter.rules.RemoveUnusedAssignAndAggregateRule;
@@ -251,7 +251,7 @@
 consolidation.add(new IntroduceAggregateCombinerRule());
 consolidation.add(new CountVarToCountOneRule());
 consolidation.add(new RemoveUnusedAssignAndAggregateRule());
-consolidation.add(new RemoveRedundantGroupByDecorVars());
+consolidation.add(new RemoveRedundantGroupByDecorVarsRule());
 //PushUnnestDownUnion => RemoveRedundantListifyRule cause these rules 
are correlated
 consolidation.add(new PushUnnestDownThroughUnionRule());
 consolidation.add(new RemoveRedundantListifyRule());
diff --git 
a/asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
 
b/asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
index 5b08bf5..4e40dd2 100644
--- 
a/asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
+++ 
b/asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
@@ -13,33 +13,20 @@
   -- STABLE_SORT [$$25(ASC)]  |PARTITIONED|
 -- HASH_PARTITION_EXCHANGE [$$25]  |PARTITIONED|
   -- STREAM_PROJECT  |PARTITIONED|
--- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
-  -- HYBRID_HASH_JOIN [$$36][$$25]  |PARTITIONED|
+-- STREAM_SELECT  |PARTITIONED|
+  -- STREAM_PROJECT  |PARTITIONED|
 -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
-  -- STREAM_PROJECT  |PARTITIONED|
--- STREAM_SELECT  |PARTITIONED|
-  -- STREAM_PROJECT  |PARTITIONED|
--- ASSIGN  |PARTITIONED|
-  -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- DATASOURCE_SCAN  |PARTITIONED|
-  -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
--- EMPTY_TUPLE_SOURCE  |PARTITIONED|
- 

Change in asterixdb[master]: AsterixDB Schema Inferencer

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: AsterixDB Schema Inferencer
..


Patch Set 4: Integration-Tests-1

Integration Tests Failed

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6077216ba457a182e8034ed47536fc5f4dcb639
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Add upsert option for feed

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add upsert option for feed
..


Patch Set 7:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Xikui Wang 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: AsterixDB Schema Inferencer

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: AsterixDB Schema Inferencer
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6077216ba457a182e8034ed47536fc5f4dcb639
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Add upsert option for feed

[Xikui Wang (Code Review)]

Xikui Wang has posted comments on this change.

Change subject: Add upsert option for feed
..


Patch Set 7:

@Till

Thanks for your comments. It's good to have someone helping me improve my 
coding style and quality. I added one test case for this option. It includes 
insert & upsert case. Feel free to leave any comments. :)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Xikui Wang 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Add upsert option for feed

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add upsert option for feed
..


Patch Set 7:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: AsterixDB Schema Inferencer

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: AsterixDB Schema Inferencer
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6077216ba457a182e8034ed47536fc5f4dcb639
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Add upsert option for feed

[Xikui Wang (Code Review)]

Hello Jenkins,

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

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

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

Change subject: Add upsert option for feed
..

Add upsert option for feed

For ASTERIXDB-1567. Provide "upsert-feed"="true" for
feed configuration, which changes the default record
insert to upsert.

Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
A asterixdb/asterix-app/data/tinysocial/twu_update.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.10.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.12.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.3.server.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.4.sleep.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.5.server.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.6.server.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.7.sleep.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.8.server.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/feeds/upsert-feed/upsert-feed.9.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/feeds/upsert-feed/upsert-feed.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite.xml
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/ExternalDataConstants.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/ExternalDataUtils.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/feeds/FeedMetadataUtil.java
18 files changed, 350 insertions(+), 13 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/68/1068/7
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1068
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 

Change in asterixdb[master]: introduce ConstantExpressionUtil

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: introduce ConstantExpressionUtil
..


Patch Set 2:

Integration Tests Timed Out

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

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

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

Change in asterixdb[master]: Add upsert option for feed

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add upsert option for feed
..


Patch Set 6:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: AsterixDB Schema Inferencer

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: AsterixDB Schema Inferencer
..


Patch Set 3: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6077216ba457a182e8034ed47536fc5f4dcb639
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: AsterixDB Schema Inferencer

[Wail Alkowaileet (Code Review)]

Hello Jenkins,

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

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

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

Change subject: AsterixDB Schema Inferencer
..

AsterixDB Schema Inferencer

Add schema builder to the clean JSON output format.

Change-Id: Ia6077216ba457a182e8034ed47536fc5f4dcb639
---
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/SessionConfig.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/APIServlet.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultSchemaAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.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/main/java/org/apache/asterix/messaging/CCMessageBroker.java
M asterixdb/asterix-app/src/main/java/org/apache/asterix/result/ResultUtils.java
M asterixdb/asterix-app/src/main/resources/webui/querytemplate.html
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/messaging/api/IApplicationMessage.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/ServletUtil.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/dataflow/FeedRecordDataFlowController.java
M asterixdb/asterix-om/pom.xml
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AFlatPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AObjectPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AOptionalFieldPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AOrderedlistPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/ARecordPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AUnionPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AUnorderedlistPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AbstractPrinterWithSchemaFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/IPrinterWithSchemaFactory.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/base/IDataFormat.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/nontagged/AqlCleanJSONWithSchemaPrinterFactoryProvider.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/AListPrinter.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/ARecordPrinter.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/json/clean/schema/AListPrinterSchema.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/json/clean/schema/APrintVisitor.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/json/clean/schema/ARecordPrinterSchema.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/schema/builder/HeterogeneousTypeComputerLoader.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/schema/builder/SchemaBuilder.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/formats/NonTaggedDataFormat.java
A asterixdb/asterix-schema-spark/pom.xml
A 
asterixdb/asterix-schema-spark/src/main/java/org/apache/asterix/schema/spark/SparkHeterogeneousTypeComputer.java
A asterixdb/asterix-schema/pom.xml
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/message/SchemaMessage.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/message/SchemaRegisterMessage.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/node/AbstractNestedSchemaNode.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/node/AbstractSchemaNode.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/node/FlatSchemaNode.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/node/IHeterogeneousTypeComputer.java
A 

Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!

[Wenhai Li (Code Review)]

Wenhai Li has uploaded a new patch set (#3).

Change subject: PLEASE EDIT to provide a meaningful commit message!
..

PLEASE EDIT to provide a meaningful commit message!

The following commits from your working branch will be included:

commit 30342904b2311900fe9fb8bc27dc1d1f7a8e81b7
Merge: 35116f1 9d63f9f
Author: Michael 
Date:   Fri Aug 12 11:16:32 2016 -0700

Merge branch 'master' into fuzzy

commit 35116f170749638467663d7e572792d3e7f74325
Merge: e7e11bf bd3a535
Author: Michael 
Date:   Fri Aug 12 07:45:33 2016 -0700

Merge branch 'master' into fuzzy

commit e7e11bf820895fb27031b59eca3056dbc93eb579
Merge: c40d969 112b39e
Author: Michael 
Date:   Tue Aug 9 00:20:46 2016 -0700

Merge branch 'master' into fuzzy

commit c40d9691a3085f864647137df516b98a63439b7c
Author: Michael 
Date:   Fri Aug 5 11:20:24 2016 -0700

Merge with master

commit 7f7a93aea5bc2ad6799735c412dc7308acb1fe8c
Merge: fce8b4b 9dfebec
Author: Michael 
Date:   Fri Aug 5 10:45:35 2016 -0700

Merge branch 'master' into fuzzy

Conflicts:

asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java

commit fce8b4bad075b27dd18c2747b8f27d25350177bd
Merge: 2ddd1a3 516d1a8
Author: Michael 
Date:   Tue Aug 2 21:19:08 2016 -0700

Merge branch 'master' into fuzzy

commit 2ddd1a331d295ed33f0081e11a3239e65e891c6b
Author: Michael 
Date:   Tue Aug 2 21:16:43 2016 -0700

Add rules

commit 395099b89c498f49ae119ec2b939b32e3770f0a6
Merge: 36dd404 dbfd22e
Author: Michael 
Date:   Wed Jul 27 20:49:56 2016 -0700

Merge branch 'master' into fuzzy

Conflicts:

asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java

commit 36dd4046aca20da739d61fbd003a351796010541
Merge: b4d9d5a a3603ac
Author: Michael 
Date:   Sun Jul 24 09:29:02 2016 -0700

Merge branch 'master' into fuzzy

commit b4d9d5a56fb258b2872743840b96e5133f8ead74
Merge: e712468 806f7d2
Author: Michael 
Date:   Sat Jul 23 23:40:11 2016 -0700

Merge branch 'master' into fuzzy

commit e712468dc63cabe05121ce4068a04d29a5a7400e
Author: Michael 
Date:   Sat Jul 23 23:38:15 2016 -0700

UNION fuzzy type.

commit 8646b803427075b73ad3ea96d6225eb288c4c66e
Author: Michael 
Date:   Thu Jul 14 20:36:03 2016 -0700

SQ again!

commit 93a3483d9c84d9b39f7c41dd493be8b823237a1d
Merge: 3ec9542 8166898
Author: Michael 
Date:   Thu Jul 14 20:11:30 2016 -0700

Merge branch 'master' into fuzzy

commit 3ec9542740b44fd22634bf7f5ac48fa923ffc2e3
Author: Michael 
Date:   Thu Jul 14 20:10:16 2016 -0700

SQ warnings;

commit 55319fc23fe162b88740960a3392664751c7bd5a
Author: Michael 
Date:   Thu Jul 14 10:55:24 2016 -0700

Another SQ warning.

commit 2edc0c700dced2c9870a94dbaa03dec6e396c8d8
Author: Michael 
Date:   Thu Jul 14 10:50:49 2016 -0700

Removed the white spaces and the SQ warnings.

commit 31d09b5650087f04005d76c29b2352b3022aaff9
Merge: 3754969 c1f984e
Author: Michael 
Date:   Thu Jul 14 10:36:40 2016 -0700

Merge branch 'master' into fuzzy

commit 375496918229fca473b5ff9172b697f7ed413266
Author: Michael 
Date:   Thu Jul 14 10:34:37 2016 -0700

Polished the code for SQ warning.

commit 439cd7c8b024f59319b2eb5b86b7229ccac8165f
Merge: 03eea46 5b2d4c8
Author: Michael 
Date:   Thu Jul 14 07:34:57 2016 -0700

Merge branch 'master' into fuzzy

commit 03eea46623e81b88a502d3bb4e8dee51e3feb7dc
Merge: 41b84c8 f09607d
Author: Michael 
Date:   Fri Jul 8 09:13:45 2016 -0700

Merge branch 'master' into fuzzy

commit 41b84c837fb890e48eb5cc11482408e174c91c6b
Author: Michael 
Date:   Fri Jul 8 09:04:56 2016 -0700

substitue the table with spaces.

commit 3d7f75b5b620c577b1677999d8918970a4b1ec8c
Merge: 8b93e8a 5a12edf
Author: Michael 
Date:   Fri Jul 1 01:26:19 2016 -0700

Merge branch 'master' into fuzzy

commit 8b93e8a857b54b93069cb42bb5bc08f9d8569a74
Merge: 63ef80f c7599b4
Author: Michael 
Date:   Sat Jun 18 06:57:03 2016 -0700

Merge branch 'master' into fuzzy

commit 63ef80f4acc056fe20ff0502f2e7112badda24c8
Merge: 96ea5b8 98ddabc
Author: Michael 
Date:   Thu Jun 16 21:30:07 2016 -0700

Merge branch 'master' into fuzzy

commit 96ea5b822012a2aa4e9bd96bcf1d06601bfa78ff
Merge: 3d5f73c fd1d460
Author: Michael 
Date:   Thu Jun 16 00:51:04 2016 -0700

Merge branch 'master' into fuzzy

commit 3d5f73c942b7247baafd30a59edc16de3e41ae91
Merge: 7d980ef 67dabe3
Author: Michael 

Change in asterixdb[master]: Fuzzyjoin rules enabled with top-level CommonOperators and t...

[Wenhai Li (Code Review)]

Wenhai Li has abandoned this change.

Change subject: Fuzzyjoin rules enabled with top-level CommonOperators and the 
testCases covering multiway fuzzyJoin and joinSelect cases.
..


Abandoned

Be replaced by refs/changes/72/1072/2

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: Ia829133808deeaba3c847ac383aa8ae76fa5d87e
Gerrit-PatchSet: 49
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Chen Li 
Gerrit-Reviewer: Inci Cetindil 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Yingyi Bu 

Change in asterixdb[master]: Initial fuzzy join framework with multiple split and tempora...

[Wenhai Li (Code Review)]

Wenhai Li has uploaded a new patch set (#2).

Change subject: Initial fuzzy join framework with multiple split and temporary 
ngram fix.
..

Initial fuzzy join framework with multiple split and temporary ngram fix.

commit d9feb3ee13f907a0dcc27c0ad91dce92ad16c9a0
Author: Michael 
Date:   Wed Dec 2 03:22:49 2015 -0800

Initial fuzzyjoin setup for inci

Change-Id: I0db68cb30bc986cf3b5d5f671694d77b5cbcf3de
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/FuzzyUtils.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/AsterixExtractFunctionsFromJoinConditionRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/FuzzyJoinRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/subplan/InlineSubplanInputForNestedTupleSourceRule.java
A asterixdb/asterix-app/data/dblp-small/csx-small-multi-id.txt
A asterixdb/asterix-app/data/dblp-small/dblp-small-multi-id.txt
A asterixdb/asterix-app/data/pub-small/csx-small-multi-id.txt
A asterixdb/asterix-app/data/pub-small/dblp-small-multi-id.txt
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join-noeqjoin/ngram-jaccard-inline.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join-noeqjoin/word-jaccard-inline.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/leftouterjoin-probe-pidx-with-join-jaccard-check-idx_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-fuzzyeq-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-fuzzyeq-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-fuzzyeq-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard-check_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/ngram-jaccard_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-fuzzyeq-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-fuzzyeq-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-fuzzyeq-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check-after-btree-access.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard-check_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_02.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_03.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/word-jaccard_04.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/leftouterjoin-probe-pidx-with-join-jaccard-check-idx_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/ngram-fuzzyeq-jaccard_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/ngram-jaccard-check_01.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/nested-index/inverted-index-join/ngram-jaccard-inline.plan
M 

Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!

[Wenhai Li (Code Review)]

Wenhai Li has uploaded a new change for review.

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

Change subject: PLEASE EDIT to provide a meaningful commit message!
..

PLEASE EDIT to provide a meaningful commit message!

The following commits from your working branch will be included:

commit 30342904b2311900fe9fb8bc27dc1d1f7a8e81b7
Merge: 35116f1 9d63f9f
Author: Michael 
Date:   Fri Aug 12 11:16:32 2016 -0700

Merge branch 'master' into fuzzy

commit 35116f170749638467663d7e572792d3e7f74325
Merge: e7e11bf bd3a535
Author: Michael 
Date:   Fri Aug 12 07:45:33 2016 -0700

Merge branch 'master' into fuzzy

commit e7e11bf820895fb27031b59eca3056dbc93eb579
Merge: c40d969 112b39e
Author: Michael 
Date:   Tue Aug 9 00:20:46 2016 -0700

Merge branch 'master' into fuzzy

commit c40d9691a3085f864647137df516b98a63439b7c
Author: Michael 
Date:   Fri Aug 5 11:20:24 2016 -0700

Merge with master

commit 7f7a93aea5bc2ad6799735c412dc7308acb1fe8c
Merge: fce8b4b 9dfebec
Author: Michael 
Date:   Fri Aug 5 10:45:35 2016 -0700

Merge branch 'master' into fuzzy

Conflicts:

asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java

commit fce8b4bad075b27dd18c2747b8f27d25350177bd
Merge: 2ddd1a3 516d1a8
Author: Michael 
Date:   Tue Aug 2 21:19:08 2016 -0700

Merge branch 'master' into fuzzy

commit 2ddd1a331d295ed33f0081e11a3239e65e891c6b
Author: Michael 
Date:   Tue Aug 2 21:16:43 2016 -0700

Add rules

commit 395099b89c498f49ae119ec2b939b32e3770f0a6
Merge: 36dd404 dbfd22e
Author: Michael 
Date:   Wed Jul 27 20:49:56 2016 -0700

Merge branch 'master' into fuzzy

Conflicts:

asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java

commit 36dd4046aca20da739d61fbd003a351796010541
Merge: b4d9d5a a3603ac
Author: Michael 
Date:   Sun Jul 24 09:29:02 2016 -0700

Merge branch 'master' into fuzzy

commit b4d9d5a56fb258b2872743840b96e5133f8ead74
Merge: e712468 806f7d2
Author: Michael 
Date:   Sat Jul 23 23:40:11 2016 -0700

Merge branch 'master' into fuzzy

commit e712468dc63cabe05121ce4068a04d29a5a7400e
Author: Michael 
Date:   Sat Jul 23 23:38:15 2016 -0700

UNION fuzzy type.

commit 8646b803427075b73ad3ea96d6225eb288c4c66e
Author: Michael 
Date:   Thu Jul 14 20:36:03 2016 -0700

SQ again!

commit 93a3483d9c84d9b39f7c41dd493be8b823237a1d
Merge: 3ec9542 8166898
Author: Michael 
Date:   Thu Jul 14 20:11:30 2016 -0700

Merge branch 'master' into fuzzy

commit 3ec9542740b44fd22634bf7f5ac48fa923ffc2e3
Author: Michael 
Date:   Thu Jul 14 20:10:16 2016 -0700

SQ warnings;

commit 55319fc23fe162b88740960a3392664751c7bd5a
Author: Michael 
Date:   Thu Jul 14 10:55:24 2016 -0700

Another SQ warning.

commit 2edc0c700dced2c9870a94dbaa03dec6e396c8d8
Author: Michael 
Date:   Thu Jul 14 10:50:49 2016 -0700

Removed the white spaces and the SQ warnings.

commit 31d09b5650087f04005d76c29b2352b3022aaff9
Merge: 3754969 c1f984e
Author: Michael 
Date:   Thu Jul 14 10:36:40 2016 -0700

Merge branch 'master' into fuzzy

commit 375496918229fca473b5ff9172b697f7ed413266
Author: Michael 
Date:   Thu Jul 14 10:34:37 2016 -0700

Polished the code for SQ warning.

commit 439cd7c8b024f59319b2eb5b86b7229ccac8165f
Merge: 03eea46 5b2d4c8
Author: Michael 
Date:   Thu Jul 14 07:34:57 2016 -0700

Merge branch 'master' into fuzzy

commit 03eea46623e81b88a502d3bb4e8dee51e3feb7dc
Merge: 41b84c8 f09607d
Author: Michael 
Date:   Fri Jul 8 09:13:45 2016 -0700

Merge branch 'master' into fuzzy

commit 41b84c837fb890e48eb5cc11482408e174c91c6b
Author: Michael 
Date:   Fri Jul 8 09:04:56 2016 -0700

substitue the table with spaces.

commit 3d7f75b5b620c577b1677999d8918970a4b1ec8c
Merge: 8b93e8a 5a12edf
Author: Michael 
Date:   Fri Jul 1 01:26:19 2016 -0700

Merge branch 'master' into fuzzy

commit 8b93e8a857b54b93069cb42bb5bc08f9d8569a74
Merge: 63ef80f c7599b4
Author: Michael 
Date:   Sat Jun 18 06:57:03 2016 -0700

Merge branch 'master' into fuzzy

commit 63ef80f4acc056fe20ff0502f2e7112badda24c8
Merge: 96ea5b8 98ddabc
Author: Michael 
Date:   Thu Jun 16 21:30:07 2016 -0700

Merge branch 'master' into fuzzy

commit 96ea5b822012a2aa4e9bd96bcf1d06601bfa78ff
Merge: 3d5f73c fd1d460
Author: Michael 
Date:   Thu Jun 16 00:51:04 2016 -0700

Merge branch 'master' into fuzzy

commit 

Change in asterixdb[master]: Add upsert option for feed

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add upsert option for feed
..


Patch Set 4:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Ensure RMI host is set to the cluster ip address

[Chris Hillery (Code Review)]

Chris Hillery has posted comments on this change.

Change subject: Ensure RMI host is set to the cluster ip address
..


Patch Set 2:

Mike - the client and cluster IPs for the CC are only the "public" IPs already. 
The client IP is obviously the one that external clients can see. As for 
cluster IP, the CC listens on all interfaces; the specific cluster IP is the 
one which is passed to NCs for connecting.

FYI, though, I'm no longer sure the whole public/private divide is really 
desirable. It was there primarily to support earlier limitations in Docker, but 
that's long ago. It'd still be useful if you were setting up a cluster where 
some nodes were behind NAT or something, but I can't imagine that's really a 
common use case. It might be worth stripping it out to avoid confusions.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0fbda4ce14c892e66595d7a1cc3801f0fc2b8fba
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Chris Hillery 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

[jira] [Commented] (ASTERIXDB-1478) using localfs goes wrong

[Vignesh Raghunathan (JIRA)]

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

Vignesh Raghunathan commented on ASTERIXDB-1478:


I was facing a similar issue earlier and I did a cursory analysis of the code 
dealing with reading data from the local file system.

My input file had the characters CÔTE D'IVOIRE. When the character Ô is 
encountered, the decoder.decode() method returns a CoderResult object 
indicating a malformed input. However, this CoderResult object is ignored. This 
results in charbuffer's position staying at 0 which results in an infinite 
loop. 

I created a temporary fix by making the decoder ignore malformed input.

I haven't looked at the code thoroughly but I could see that 
AsterixInputStreamReader uses AsterixInputStream which is a byte stream. 
Neither the read method in AsterixInputStream nor AsterixInputStreamReader have 
checks for partial reads of bytes corresponding to utf8 characters between 
calls to the read method.

> using localfs goes wrong
> 
>
> Key: ASTERIXDB-1478
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1478
> Project: Apache AsterixDB
>  Issue Type: Bug
>  Components: Error Reporting
> Environment: AsterixDB 0.8.9(pulled from github today)
> Ubuntu 14.0
> Java -version 1.8.0_79
> Computer has 1 ssd and 1 hhd
>Reporter: yipzifai
>Assignee: Wenhai
>  Labels: features
> Attachments: cc.log, cpu.jpg, execute.log, my_asterix_nc1.log, 
> my_asterix_nc2.log
>
>
> I load data file using localfs but AsterixDB keeps running for a long long 
> time.(ps:CPU utilization ~= 100%) I have been waiting over 8 hours 
> yesterday.Then i found that the storage folder in workingdir didn't add any 
> files.
> Exectly, I try to use feeding or AQL "Insert", but it still goes wrong.
> AQL:
> use dataverse test;
> drop dataset relationship_B if exists;
> drop type relationshipB if exists;
> create type relationshipB as closed{
> pkId:int64,
> who_id:int64,
> who_name:string,
> id:int64,
> name:string,
> code:string,
> shortname:string,
> brief:string,
> validateDate:date?,
> gfType:string,
> shijizengjianchi:string,
> chiyoushuliang:double,
> chigubianhua:string,
> zhanbi:double,
> konggugongsiID:int64?,
> konggugongsiName:string?,
> rType:string,
> source:string,
> method:string
> };
> create dataset relationship_B(relationshipB) primary key pkId;
> use dataverse test;
> load dataset relationship_B
> using localfs
> (("path"="127.0.0.1:///home/to/adbSrc/ddl/relationshipB.adm"),("format"="adm"));
> About data files:
> one file is big object insertion(each record has over 30,000 characters); 
> each record of another file has 100~200 characters.
> {code}
> local.xml
> 
> 
> /usr/local/jdk1.8.0_91/jre
> 
> /home/to/adbSrc/asterix-mgmt/clusters/local/working_dir/logs
> 
> /home/to/adbSrc/asterix-mgmt/clusters/local/working_dir/txnLogs
> storage
> 
> /home/to/adbSrc/asterix-mgmt/clusters/local/working_dir
> true
> 
>   
> /home/to/adbSrc/asterix-mgmt/clusters/local/working_dir/nc/p1,/home/to/adbSrc/asterix-mgmt/clusters/local/working_dir/nc/p2
> 
> master
> 127.0.0.1
> 127.0.0.1
> 1098
> 1099
> 
> 8900
> 
> 
> nc1
> 127.0.0.1
> 
> /home/to/adbSrc/asterix-mgmt/clusters/local/working_dir/nc1/txnLogs
>  
> 8901
> 
> 
> nc2
> 127.0.0.1
> 
> /home/to/adbSrc/asterix-mgmt/clusters/local/working_dir/nc2/txnLogs
>   
> 8902
> 
> 1
> 5
> 0
> 5
> 10
> 8640
> 6
> 
> {code}



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

[jira] [Updated] (ASTERIXDB-1575) TwitterParser doesn't support non-ascii strings.

[Jianfeng Jia (JIRA)]

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

Jianfeng Jia updated ASTERIXDB-1575:

Assignee: Xikui Wang

> TwitterParser doesn't support non-ascii strings.
> 
>
> Key: ASTERIXDB-1575
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1575
> Project: Apache AsterixDB
>  Issue Type: Bug
>Reporter: Wail Alkowaileet
>Assignee: Xikui Wang
>
> Hi,
> When I tried to run the TwitterFeed to collect Arabic tweets, I got malformed 
> strings. 
> it seems that JObjectUtil.getNormalizedString() discard all Arabic letters in 
> UTF-8.



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

[jira] [Commented] (ASTERIXDB-1571) Subquery in from clause throws NullPointerException

[Yingyi Bu (JIRA)]

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

Yingyi Bu commented on ASTERIXDB-1571:
--

Thanks, I'll look into the 2nd query.
However, note that "SELECT *" creates nested tuples. Therefore the query you 
commented might not express what you expect.  The following query might express 
what you want:

{noformat}
use sampdb;

with samptable1 as
(
  select value samptable from samptable
)

select *
from
(
  select * from
  samptable s1
  left outer join
  samptable s2
on s1.id = s2.id
) t ;
{noformat}

> Subquery in from clause throws NullPointerException
> ---
>
> Key: ASTERIXDB-1571
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1571
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Master branch with latest commit id: 
> bd3a535b171a5cc7486175
>Reporter: Vignesh Raghunathan
>Assignee: Yingyi Bu
>
> To reproduce the issue, run the following sqlpp statements
> {code}
> drop dataverse sampdb if exists;
> create dataverse sampdb;
> use sampdb;
> drop dataset samptable if exists;
> drop type samptabletype if exists;
> create type samptabletype as closed {
>   id: int8
> };
> create dataset samptable(samptabletype) primary key id;
> insert into samptable ({'id' : 0});
> insert into samptable ({'id' : 1});
> select *
> from
> (
> select * from samptable
> );
> {code}
> Result:
> {code}
> Aug 12, 2016 1:57:08 PM org.apache.asterix.api.http.servlet.APIServlet doPost
> SEVERE: null
> java.lang.NullPointerException
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:130)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.Projection.accept(Projection.java:45)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:193)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectRegular.accept(SelectRegular.java:40)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:169)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectClause.accept(SelectClause.java:42)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:140)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.struct.SetOperationInput.accept(SetOperationInput.java:56)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:201)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectSetOperation.accept(SelectSetOperation.java:47)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:223)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.expression.SelectExpression.accept(SelectExpression.java:55)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:90)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:76)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedIdentifier(ExpressionToVariableUtil.java:48)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedVariable(ExpressionToVariableUtil.java:91)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromTerm(SQLPPParser.java:3767)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromClause(SQLPPParser.java:3674)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectBlock(SQLPPParser.java:3342)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectSetOperation(SQLPPParser.java:3255)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectExpression(SQLPPParser.java:3226)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.Query(SQLPPParser.java:2176)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SingleStatement(SQLPPParser.java:414)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.Statement(SQLPPParser.java:325)
>   at 
> 

Change in asterixdb[master]: Ensure RMI host is set to the cluster ip address

[Michael Blow (Code Review)]

Michael Blow has submitted this change and it was merged.

Change subject: Ensure RMI host is set to the cluster ip address
..


Ensure RMI host is set to the cluster ip address

By default RMI stubs reference InetAddress.getLocalHost(), which may not
be reachable from the CC / NC.  Unless explicitly set, bind it to the
cluster net ip address, public in case of NC.

Change-Id: I0fbda4ce14c892e66595d7a1cc3801f0fc2b8fba
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1065
Sonar-Qube: Jenkins 
Tested-by: Jenkins 
Integration-Tests: Jenkins 
Reviewed-by: Chris Hillery 
Reviewed-by: Till Westmann 
---
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
2 files changed, 11 insertions(+), 0 deletions(-)

Approvals:
  Chris Hillery: Looks good to me, approved
  Till Westmann: Looks good to me, but someone else must approve
  Jenkins: Verified; No violations found; Verified



diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
index b6ce217..28cbe11 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
@@ -96,6 +96,11 @@
 AsterixAppContextInfo.initialize(appCtx, 
getNewHyracksClientConnection(), GlobalRecoveryManager.INSTANCE,
 libraryManager);
 
+if (System.getProperty("java.rmi.server.hostname") == null) {
+System.setProperty("java.rmi.server.hostname",
+((ClusterControllerService) 
ccAppCtx.getControllerService()).getCCConfig().clusterNetIpAddress);
+}
+
 proxy = AsterixStateProxy.registerRemoteObject();
 appCtx.setDistributedState(proxy);
 
diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/NCApplicationEntryPoint.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/NCApplicationEntryPoint.java
index 2555b5a..4cb9d70 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/NCApplicationEntryPoint.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/NCApplicationEntryPoint.java
@@ -95,6 +95,12 @@
 LOGGER.info("Starting Asterix node controller: " + nodeId);
 }
 
+if (System.getProperty("java.rmi.server.hostname") == null) {
+System.setProperty("java.rmi.server.hostname",
+((NodeControllerService) ncAppCtx.getControllerService())
+.getConfiguration().clusterNetPublicIPAddress);
+}
+
 runtimeContext = new AsterixAppRuntimeContext(ncApplicationContext, 
metadataRmiPort);
 AsterixMetadataProperties metadataProperties = 
((IAsterixPropertiesProvider) runtimeContext)
 .getMetadataProperties();

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0fbda4ce14c892e66595d7a1cc3801f0fc2b8fba
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Chris Hillery 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 

[jira] [Commented] (ASTERIXDB-1571) Subquery in from clause throws NullPointerException

[Vignesh Raghunathan (JIRA)]

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

Vignesh Raghunathan commented on ASTERIXDB-1571:


Could you please try the following query?

{code}
drop dataverse sampdb if exists;
create dataverse sampdb;
use sampdb;

drop dataset samptable if exists;
drop type samptabletype if exists;

create type samptabletype as closed {
  id: int8
};

create dataset samptable(samptabletype) primary key id;

insert into samptable ({'id' : 0});
insert into samptable ({'id' : 1});

with samptable1 as 
(
select * from samptable
)

select *
from
(
select * from 
samptable1 s1 
left outer join 
samptable1 s2
on s1.id = s2.id
) t ;
{code}

I tried to simplify my original query. Since the previous query was throwing a 
NPE, I assumed they were due to same reason without looking at the stack trace.

> Subquery in from clause throws NullPointerException
> ---
>
> Key: ASTERIXDB-1571
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1571
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Master branch with latest commit id: 
> bd3a535b171a5cc7486175
>Reporter: Vignesh Raghunathan
>Assignee: Yingyi Bu
>
> To reproduce the issue, run the following sqlpp statements
> {code}
> drop dataverse sampdb if exists;
> create dataverse sampdb;
> use sampdb;
> drop dataset samptable if exists;
> drop type samptabletype if exists;
> create type samptabletype as closed {
>   id: int8
> };
> create dataset samptable(samptabletype) primary key id;
> insert into samptable ({'id' : 0});
> insert into samptable ({'id' : 1});
> select *
> from
> (
> select * from samptable
> );
> {code}
> Result:
> {code}
> Aug 12, 2016 1:57:08 PM org.apache.asterix.api.http.servlet.APIServlet doPost
> SEVERE: null
> java.lang.NullPointerException
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:130)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.Projection.accept(Projection.java:45)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:193)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectRegular.accept(SelectRegular.java:40)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:169)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectClause.accept(SelectClause.java:42)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:140)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.struct.SetOperationInput.accept(SetOperationInput.java:56)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:201)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectSetOperation.accept(SelectSetOperation.java:47)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:223)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.expression.SelectExpression.accept(SelectExpression.java:55)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:90)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:76)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedIdentifier(ExpressionToVariableUtil.java:48)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedVariable(ExpressionToVariableUtil.java:91)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromTerm(SQLPPParser.java:3767)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromClause(SQLPPParser.java:3674)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectBlock(SQLPPParser.java:3342)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectSetOperation(SQLPPParser.java:3255)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectExpression(SQLPPParser.java:3226)
>   at 
> 

Change in asterixdb[master]: small refactoring

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: small refactoring
..


Patch Set 2:

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

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

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

Change in asterixdb[master]: Ensure RMI host is set to the cluster ip address

[Michael Blow (Code Review)]

Michael Blow has posted comments on this change.

Change subject: Ensure RMI host is set to the cluster ip address
..


Patch Set 1:

Chris, one thing that's not clear to me is why we don't want to have a "public" 
cluster IP address for CCs like we have for NCs.  It seems the same situation 
could happen that direction as well.

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

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

Change in asterixdb[master]: AsterixDB Schema Inferencer

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: AsterixDB Schema Inferencer
..


Patch Set 3:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6077216ba457a182e8034ed47536fc5f4dcb639
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: AsterixDB Schema Inferencer

[Wail Alkowaileet (Code Review)]

Hello Jenkins,

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

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

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

Change subject: AsterixDB Schema Inferencer
..

AsterixDB Schema Inferencer

Add schema builder to the clean JSON output format.

Change-Id: Ia6077216ba457a182e8034ed47536fc5f4dcb639
---
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/SessionConfig.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/APIServlet.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultLocationsAPIServlets.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultSchemaAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.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/main/java/org/apache/asterix/messaging/CCMessageBroker.java
M asterixdb/asterix-app/src/main/java/org/apache/asterix/result/ResultUtils.java
M asterixdb/asterix-app/src/main/resources/webui/querytemplate.html
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/messaging/api/IApplicationMessage.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/ServletUtil.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/dataflow/FeedRecordDataFlowController.java
M asterixdb/asterix-om/pom.xml
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AFlatPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AObjectPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AOptionalFieldPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AOrderedlistPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/ARecordPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AUnionPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AUnorderedlistPrinterFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/AbstractPrinterWithSchemaFactory.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/printers/json/clean/schema/IPrinterWithSchemaFactory.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/base/IDataFormat.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/nontagged/AqlCleanJSONWithSchemaPrinterFactoryProvider.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/AListPrinter.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/ARecordPrinter.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/json/clean/schema/AListPrinterSchema.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/json/clean/schema/APrintVisitor.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/pointables/printer/json/clean/schema/ARecordPrinterSchema.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/schema/builder/HeterogeneousTypeComputerLoader.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/schema/builder/SchemaBuilder.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/formats/NonTaggedDataFormat.java
A asterixdb/asterix-schema-spark/pom.xml
A 
asterixdb/asterix-schema-spark/src/main/java/org/apache/asterix/schema/spark/SparkHeterogeneousTypeComputer.java
A asterixdb/asterix-schema/pom.xml
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/message/SchemaMessage.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/message/SchemaRegisterMessage.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/node/AbstractNestedSchemaNode.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/node/AbstractSchemaNode.java
A 
asterixdb/asterix-schema/src/main/java/org/apache/asterix/schema/node/FlatSchemaNode.java
A 

[jira] [Commented] (ASTERIXDB-1571) Subquery in from clause throws NullPointerException

[Yingyi Bu (JIRA)]

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

Yingyi Bu commented on ASTERIXDB-1571:
--

This is an error message issue.  The NPE is generated from creating an error 
message.

A subquery in a from clause has to be aliased, e.g.:

select *
from
(
  select * from samptable
) t;

> Subquery in from clause throws NullPointerException
> ---
>
> Key: ASTERIXDB-1571
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1571
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Master branch with latest commit id: 
> bd3a535b171a5cc7486175
>Reporter: Vignesh Raghunathan
>Assignee: Yingyi Bu
>
> To reproduce the issue, run the following sqlpp statements
> {code}
> drop dataverse sampdb if exists;
> create dataverse sampdb;
> use sampdb;
> drop dataset samptable if exists;
> drop type samptabletype if exists;
> create type samptabletype as closed {
>   id: int8
> };
> create dataset samptable(samptabletype) primary key id;
> insert into samptable ({'id' : 0});
> insert into samptable ({'id' : 1});
> select *
> from
> (
> select * from samptable
> );
> {code}
> Result:
> {code}
> Aug 12, 2016 1:57:08 PM org.apache.asterix.api.http.servlet.APIServlet doPost
> SEVERE: null
> java.lang.NullPointerException
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:130)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.Projection.accept(Projection.java:45)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:193)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectRegular.accept(SelectRegular.java:40)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:169)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectClause.accept(SelectClause.java:42)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:140)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.struct.SetOperationInput.accept(SetOperationInput.java:56)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:201)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectSetOperation.accept(SelectSetOperation.java:47)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:223)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.expression.SelectExpression.accept(SelectExpression.java:55)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:90)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:76)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedIdentifier(ExpressionToVariableUtil.java:48)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedVariable(ExpressionToVariableUtil.java:91)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromTerm(SQLPPParser.java:3767)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromClause(SQLPPParser.java:3674)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectBlock(SQLPPParser.java:3342)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectSetOperation(SQLPPParser.java:3255)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectExpression(SQLPPParser.java:3226)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.Query(SQLPPParser.java:2176)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SingleStatement(SQLPPParser.java:414)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.Statement(SQLPPParser.java:325)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.parse(SQLPPParser.java:260)
>   at 
> org.apache.asterix.api.http.servlet.APIServlet.doPost(APIServlet.java:119)
>   at javax.servlet.http.HttpServlet.service(HttpServlet.java:707)
>  

[jira] [Commented] (ASTERIXDB-1451) Upsert: Open Index test fails with duplicate rows in result when VBC page size is reduced

[Murtadha Hubail (JIRA)]

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

Murtadha Hubail commented on ASTERIXDB-1451:




I have been investigating this issue for some time. It is not related to the 
VBC page size nor upsert. When any record, with an enforced index of a type 
other than INT64, is written to a disk component, a delete operation with an 
antimatter tuple for that same record will not hide the record written in the 
disk component of the enforced index. Therefore, duplicate records will be 
returned in any query using the enforced index.

I haven't found the root cause yet, but I believe it is related to the variable 
type propagation during query compilation. I will discuss it with [~buyingyi] 
and [~alamoudi].

The issue can be easily reproduced by the following:

{code}
drop dataverse test if exists;
create dataverse test;
use dataverse test;

create type OrderOpenType as open {
  o_orderkey: int64
}

create dataset OrdersOpen(OrderOpenType)
primary key o_orderkey;

insert into dataset OrdersOpen (
  {"o_orderkey": 1,
  "o_custkey": 1,}
)

create index idx_Orders_Custkey on OrdersOpen(o_custkey:int32) enforced;

for $o in dataset('OrdersOpen')
where
 $o.o_custkey >=-1
return {
  "o_orderkey": $o.o_orderkey
}
{code}

> Upsert: Open Index test fails with duplicate rows in result when VBC page 
> size is reduced
> -
>
> Key: ASTERIXDB-1451
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1451
> Project: Apache AsterixDB
>  Issue Type: Bug
>Reporter: Michael Blow
>Assignee: Murtadha Hubail
>
> To repro:
> - configure storage.memorycomponent.pagesize to 8k, and increase 
> storage.memorycomponent.numpages to 24
> - run asterix-app runtime tests
> - observe failure in open-index test with duplicated rows as shown below.
> $ diff -du 
> src/test/resources/runtimets/results/upsert/open-index/open-index.1.adm 
> rttest/results/upsert/open-index.adm
> --- src/test/resources/runtimets/results/upsert/open-index/open-index.1.adm   
> 2016-04-27 20:40:58.0 -0700
> +++ rttest/results/upsert/open-index.adm  2016-05-16 19:08:57.0 
> -0700
> @@ -1099,11 +1099,15 @@
>  { "o_orderkey": 5927, "o_custkey": 116 }
>  { "o_orderkey": 5952, "o_custkey": 148 }
>  { "o_orderkey": 5955, "o_custkey": 94 }
> +{ "o_orderkey": 5955, "o_custkey": 94 }
> +{ "o_orderkey": 5957, "o_custkey": 89 }
>  { "o_orderkey": 5957, "o_custkey": 89 }
>  { "o_orderkey": 5958, "o_custkey": 115 }
>  { "o_orderkey": 5984, "o_custkey": 70 }
>  { "o_orderkey": 5985, "o_custkey": 143 }
>  { "o_orderkey": 5986, "o_custkey": 115 }
> +{ "o_orderkey": 5986, "o_custkey": 115 }
> +{ "o_orderkey": 5987, "o_custkey": 64 }
>  { "o_orderkey": 5987, "o_custkey": 64 }
>  { "o_orderkey": 10986, "o_custkey": 115 }
>  { "o_orderkey": 10987, "o_custkey": 64 }



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

Change in asterixdb[master]: introduce ConstantExpressionUtil

[Till Westmann (Code Review)]

Till Westmann has posted comments on this change.

Change subject: introduce ConstantExpressionUtil
..


Patch Set 2:

Addressed comments (except for the potential other uses - those should be 
next). Not much left :)

If you +2 this, feel free to merge.

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

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

Change in asterixdb[master]: introduce ConstantExpressionUtil

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: introduce ConstantExpressionUtil
..


Patch Set 2:

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

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

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

Change in asterixdb[master]: introduce ConstantExpressionUtil

[Till Westmann (Code Review)]

Hello Jenkins,

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

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

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

Change subject: introduce ConstantExpressionUtil
..

introduce ConstantExpressionUtil

Change-Id: Ic0034bd6ba7f449a03eda48070a8de614608c2dd
---
A 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/util/ConstantExpressionUtil.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/UnnestToDataScanRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
3 files changed, 68 insertions(+), 20 deletions(-)


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

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

Change in asterixdb[master]: Fuzzyjoin rules enabled with top-level CommonOperators and t...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Fuzzyjoin rules enabled with top-level CommonOperators and the 
testCases covering multiway fuzzyJoin and joinSelect cases.
..


Patch Set 49: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia829133808deeaba3c847ac383aa8ae76fa5d87e
Gerrit-PatchSet: 49
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Chen Li 
Gerrit-Reviewer: Inci Cetindil 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

[jira] [Created] (ASTERIXDB-1575) TwitterParser doesn't support non-ascii strings.

[Wail Alkowaileet (JIRA)]

Wail Alkowaileet created ASTERIXDB-1575:
---

 Summary: TwitterParser doesn't support non-ascii strings.
 Key: ASTERIXDB-1575
 URL: https://issues.apache.org/jira/browse/ASTERIXDB-1575
 Project: Apache AsterixDB
  Issue Type: Bug
Reporter: Wail Alkowaileet


Hi,
When I tried to run the TwitterFeed to collect Arabic tweets, I got malformed 
strings. 

it seems that JObjectUtil.getNormalizedString() discard all Arabic letters in 
UTF-8.



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

Change in asterixdb[master]: small refactoring

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: small refactoring
..


Patch Set 2:

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

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

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

Change in asterixdb[master]: small refactoring

[Till Westmann (Code Review)]

Hello Jenkins,

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

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

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

Change subject: small refactoring
..

small refactoring

Change-Id: I37eab1645416e3aad6119bba527c5e3b4b98fddc
---
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/dataflow/FeedRecordDataFlowController.java
1 file changed, 21 insertions(+), 15 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I37eab1645416e3aad6119bba527c5e3b4b98fddc
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: abdullah alamoudi 

Change in asterixdb[master]: Add Asterix Extension Manager

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add Asterix Extension Manager
..


Patch Set 12:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I280268495cc3aad00f898cba21f7299f7120ce5c
Gerrit-PatchSet: 12
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Steven Jacobs 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Add Asterix Extension Manager

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add Asterix Extension Manager
..


Patch Set 12:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I280268495cc3aad00f898cba21f7299f7120ce5c
Gerrit-PatchSet: 12
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Steven Jacobs 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Add Asterix Extension Manager

[abdullah alamoudi (Code Review)]

Hello Jenkins,

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

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

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

Change subject: Add Asterix Extension Manager
..

Add Asterix Extension Manager

More extension support is added. A user can now provide implementations
for the IExtension interface which will give them more control over
the behavior of the system and give them the ability to add custom
features.

Initial customizations include:
1. Metadata Tuple Translators
2. Metadata Datasets
3. Query Translators
4. Statement Handlers
5. Lang Compilation Provider

Change-Id: I280268495cc3aad00f898cba21f7299f7120ce5c
---
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveEvent.java
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveJobNotificationHandler.java
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveManager.java
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveRuntimeId.java
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveRuntimeManager.java
D 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveRuntimeRegistry.java
A 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveSourceOperatorNodePushable.java
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/IActiveRuntime.java
D 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/IActiveRuntimeRegistry.java
R 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/message/ActiveManagerMessage.java
C 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/message/ActivePartitionMessage.java
A 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExtension.java
A 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/ExtensionFunctionIdentifier.java
A 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IAlgebraExtensionManager.java
A 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/FuzzyUtils.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/AddEquivalenceClassForRecordConstructorRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByHandleFieldAccessRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ByNameToByIndexFieldAccessRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/CancelUnnestWithNestedListifyRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/FeedScanCollectionToUnnest.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/FuzzyEqRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/InjectTypeCastForSwitchCaseRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/InjectTypeCastForUnionRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceAutogenerateIDRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceDynamicTypeCastRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceUnnestForCollectionToSequenceRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/MetaFunctionToMetaVariableRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/PullPositionalVariableFromUnnestRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/RemoveRedundantListifyRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ReplaceSinkOpWithCommitOpRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ResolveVariableRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/SetClosedRecordConstructorsRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/SimilarityCheckRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/UnnestToDataScanRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AccessMethodUtils.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java
M 

Change in asterixdb[master]: Add Asterix Extension Manager

[abdullah alamoudi (Code Review)]

abdullah alamoudi has posted comments on this change.

Change subject: Add Asterix Extension Manager
..


Patch Set 7:

(25 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveRuntime.java
File 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveRuntime.java:

Line 36: 
> Why not move start() and stop() cannot stay in IActiveRuntime
Class is gone :)


https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-active/src/main/java/org/apache/asterix/active/runtime/StopActiveRuntimeOperatorDescriptor.java
File 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/runtime/StopActiveRuntimeOperatorDescriptor.java:

Line 40: public 
StopActiveRuntimeOperatorDescriptor(IOperatorDescriptorRegistry spec, 
ActiveRuntimeId runtimeId) {
> This class is never used. So I'm wondering if there's something we can do t
Class is gone. As I pointed out before that this mechanism of communicating 
with the active manager is bad.


https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-algebra/pom.xml
File asterixdb/asterix-algebra/pom.xml:

Line 163: 
> It seems you don't need to add this dependency explicitly here as it is a d
Done


https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExtension.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExtension.java:

Line 44: ILangCompilationProvider getAqlLangCompilationProvider();
> It seems that the extension mechanism shouldn't hard-code the AQL/SQL++ dua
Done


Line 44: ILangCompilationProvider getAqlLangCompilationProvider();
> Agree with Till.
Done


Line 60:  */
> The presence of this method here seems odd to me, as this interface is supp
Done


Line 62: UnnestOperator unnestOp, ILogicalExpression unnestExpr, 
AbstractFunctionCallExpression functionCallExpr) throws AlgebricksException;
> MAJOR SonarQube violation:
Done


https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IAlgebraExtensionManager.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IAlgebraExtensionManager.java:

Line 32: boolean unnestToDataScan(Mutable opRef, 
IOptimizationContext context, UnnestOperator unnestOp,
> Again, the presence of this method seems odd to me. I think the right way t
I totally agree but think that we should refactor it in another change :)


https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java:

Line 26: public interface IExtensionStatement extends Statement {
> Document this public interface.
Done


Line 34:  * Called when the {@code IQueryTranslator} encounters an 
extension statement
> The annotation could be sth. like "An implementation class should implement
Done


Line 35:  * 
> MAJOR SonarQube violation:
Done


Line 42: void handle(IStatementExecutor queryTranslator, 
AqlMetadataProvider metadataProvider, IHyracksClientConnection hcc)
> queryTranslator -> statementExecutor
Done


Line 43: throws Exception;
> CRITICAL SonarQube violation:
Done


https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java:

Line 154: IAlgebraExtensionManager algebraExtensionManager) {
> refactoring needed.
Done


Line 176: normalization.add(new 
UnnestToDataScanRule(algebraExtensionManager));
> refactoring needed.  Not necessary in this change.  File an issue if it is 
Done


https://asterix-gerrit.ics.uci.edu/#/c/1017/7/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java:

Line 34: 
> Add a brief comment for the interface.
Done


Line 40: ASYNC_DEFERRED
> Add brief comments for each enum value.
Done


Line 50:  * @param resultDelivery
> The comment seems out-dated -- resultDelivery is not a boolean value.
Done


Line 52:  * @return A List containing a QueryResult instance 
corresponding to each submitted query.
> The documentation of @return seems not right?
Done


Line 56: throws Exception;
> CRITICAL SonarQube violation:
Done


Line 62: ICompiledDmlStatement clfrqs) throws AsterixException, 
RemoteException, AlgebricksException, JSONException, ACIDException;
> meaningful variable name -- what does clfrqs mean?
Done


Line 63: 
> The relationship of 

[jira] [Assigned] (ASTERIXDB-1574) Result of subquery in where clause could not be cast to a scalar value

[Yingyi Bu (JIRA)]

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

Yingyi Bu reassigned ASTERIXDB-1574:


Assignee: Yingyi Bu

> Result of subquery in where clause could not be cast to a scalar value
> --
>
> Key: ASTERIXDB-1574
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1574
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Master branch with latest commit id: 
> bd3a535b171a5cc7486175
>Reporter: Vignesh Raghunathan
>Assignee: Yingyi Bu
>
> To reproduce the issue, run the following sqlpp statements:
> {code}
> drop dataverse sampdb if exists;
> create dataverse sampdb;
> use sampdb;
> drop dataset samptable if exists;
> drop type samptabletype if exists;
> create type samptabletype as closed {
>   id: int64
> };
> create dataset samptable(samptabletype) primary key id;
> insert into samptable ({'id' : 1});
> insert into samptable ({'id' : 2});
> insert into samptable ({'id' : 3});
> insert into samptable ({'id' : 4});
> insert into samptable ({'id' : 5});
> select *
> from samptable 
> where id > (select value avg(id) from samptable)[0];
> {code}
> Result:
> {code}
> org.apache.hyracks.algebricks.common.exceptions.AlgebricksException: The 
> input type for input argument 0(ARecord) is not expected.
>   at 
> org.apache.asterix.om.typecomputer.impl.CollectionMemberResultType.checkArgType(CollectionMemberResultType.java:40)
>   at 
> org.apache.asterix.om.typecomputer.base.AbstractResultTypeComputer.lambda$computeType$2(AbstractResultTypeComputer.java:39)
>   at 
> org.apache.asterix.om.typecomputer.base.AbstractResultTypeComputer$$Lambda$18/1226615280.checkArgTypes(Unknown
>  Source)
>   at 
> org.apache.asterix.om.typecomputer.impl.TypeComputeUtils.resolveResultType(TypeComputeUtils.java:92)
>   at 
> org.apache.asterix.om.typecomputer.base.AbstractResultTypeComputer.computeType(AbstractResultTypeComputer.java:39)
>   at 
> org.apache.asterix.dataflow.data.common.AqlExpressionTypeComputer.getTypeForFunction(AqlExpressionTypeComputer.java:80)
>   at 
> org.apache.asterix.dataflow.data.common.AqlExpressionTypeComputer.getType(AqlExpressionTypeComputer.java:53)
>   at 
> org.apache.hyracks.algebricks.core.algebra.typing.AbstractTypeEnvironment.getType(AbstractTypeEnvironment.java:45)
>   at 
> org.apache.hyracks.algebricks.core.algebra.operators.logical.UnnestOperator.computeOutputTypeEnvironment(UnnestOperator.java:51)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AlgebricksOptimizationContext.computeAndSetTypeEnvironmentForOperator(AlgebricksOptimizationContext.java:298)
>   at 
> org.apache.hyracks.algebricks.rewriter.rules.InferTypesRule.rewritePost(InferTypesRule.java:42)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:126)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:112)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.AbstractRuleController.rewriteOperatorRef(AbstractRuleController.java:100)
>   at 
> org.apache.hyracks.algebricks.compiler.rewriter.rulecontrollers.SequentialOnceRuleController.rewriteWithRuleCollection(SequentialOnceRuleController.java:44)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.HeuristicOptimizer.runOptimizationSets(HeuristicOptimizer.java:102)
>   at 
> org.apache.hyracks.algebricks.core.rewriter.base.HeuristicOptimizer.optimize(HeuristicOptimizer.java:82)
>   at 
> 

Change in asterixdb[master]: introduce AlgebraUtils and AlgebraUtils

[Yingyi Bu (Code Review)]

Yingyi Bu has posted comments on this change.

Change subject: introduce AlgebraUtils and AlgebraUtils
..


Patch Set 1:

Here are the names I like:
1.  AlgebraUtils -> ConstentExpressionUtil (no "s")
2.  FunctionUtil stay as it is.

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

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

Change in asterixdb[master]: Add upsert option for feed

[Till Westmann (Code Review)]

Till Westmann has posted comments on this change.

Change subject: Add upsert option for feed
..


Patch Set 3:

(1 comment)

Looks good now. I've got one tiny stylistic proposal, but that's not important. 
However, it would be really nice if you could add a regression test that checks 
that the syntax work and that the corresponding feed indeed has upsert 
semantics.

I'm sorry about the trickle review - should have spotted the missing test at 
first glance ...

https://asterix-gerrit.ics.uci.edu/#/c/1068/3/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 518: }
As the difference between the branches is quite small, how about something like 
this:

final Kind kind = isUpsertFeed ? Kind.UPSERT : Kind.INSERT;
feedModificationOp = new InsertDeleteUpsertOperator(targetDatasource, 
varRef, varRefsForLoading, metaExpSingletonList, kind, false);

with the corresponding import of the Kind enum?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5133e7c6941fea4110cc9983f99502f364dc810
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: Yes

[jira] [Assigned] (ASTERIXDB-1571) Subquery in from clause throws NullPointerException

[Yingyi Bu (JIRA)]

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

Yingyi Bu reassigned ASTERIXDB-1571:


Assignee: Yingyi Bu

> Subquery in from clause throws NullPointerException
> ---
>
> Key: ASTERIXDB-1571
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1571
> Project: Apache AsterixDB
>  Issue Type: Bug
> Environment: Master branch with latest commit id: 
> bd3a535b171a5cc7486175
>Reporter: Vignesh Raghunathan
>Assignee: Yingyi Bu
>
> To reproduce the issue, run the following sqlpp statements
> {code}
> drop dataverse sampdb if exists;
> create dataverse sampdb;
> use sampdb;
> drop dataset samptable if exists;
> drop type samptabletype if exists;
> create type samptabletype as closed {
>   id: int8
> };
> create dataset samptable(samptabletype) primary key id;
> insert into samptable ({'id' : 0});
> insert into samptable ({'id' : 1});
> select *
> from
> (
> select * from samptable
> );
> {code}
> Result:
> {code}
> Aug 12, 2016 1:57:08 PM org.apache.asterix.api.http.servlet.APIServlet doPost
> SEVERE: null
> java.lang.NullPointerException
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:130)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.Projection.accept(Projection.java:45)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:193)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectRegular.accept(SelectRegular.java:40)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:169)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectClause.accept(SelectClause.java:42)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:140)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.struct.SetOperationInput.accept(SetOperationInput.java:56)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:201)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.clause.SelectSetOperation.accept(SelectSetOperation.java:47)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:223)
>   at 
> org.apache.asterix.lang.sqlpp.visitor.SqlppFormatPrintVisitor.visit(SqlppFormatPrintVisitor.java:1)
>   at 
> org.apache.asterix.lang.sqlpp.expression.SelectExpression.accept(SelectExpression.java:55)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:90)
>   at 
> org.apache.asterix.lang.sqlpp.util.SqlppFormatPrintUtil.toString(SqlppFormatPrintUtil.java:76)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedIdentifier(ExpressionToVariableUtil.java:48)
>   at 
> org.apache.asterix.lang.sqlpp.util.ExpressionToVariableUtil.getGeneratedVariable(ExpressionToVariableUtil.java:91)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromTerm(SQLPPParser.java:3767)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.FromClause(SQLPPParser.java:3674)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectBlock(SQLPPParser.java:3342)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectSetOperation(SQLPPParser.java:3255)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SelectExpression(SQLPPParser.java:3226)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.Query(SQLPPParser.java:2176)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.SingleStatement(SQLPPParser.java:414)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.Statement(SQLPPParser.java:325)
>   at 
> org.apache.asterix.lang.sqlpp.parser.SQLPPParser.parse(SQLPPParser.java:260)
>   at 
> org.apache.asterix.api.http.servlet.APIServlet.doPost(APIServlet.java:119)
>   at javax.servlet.http.HttpServlet.service(HttpServlet.java:707)
>   at javax.servlet.http.HttpServlet.service(HttpServlet.java:790)
>   at 
> org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:845)
>   at 
> 

Change in asterixdb[master]: Add a testCases for issue-1478 The following commits from yo...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Add a testCases for issue-1478 The following commits from your 
working branch will be included:
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I90c122e48aa330753512a334883c85dbf9ef6cb6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Add a testCases for issue-1478 The following commits from yo...

[Wenhai Li (Code Review)]

Wenhai Li has uploaded a new change for review.

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

Change subject: Add a testCases for issue-1478 The following commits from your 
working branch will be included:
..

Add a testCases for issue-1478
The following commits from your working branch will be included:

commit 59f33057a840cb67d2cfef5a8e730a515dca7322
Merge: a253268 bd3a535
Author: Michael 
Date:   Fri Aug 12 07:46:47 2016 -0700

Merge branch 'master' into chinese

commit a253268e9ded282a8508c4d977b5dab6351408f2
Author: Michael 
Date:   Fri Aug 12 07:43:45 2016 -0700

Add the utf8 testCases.

Change-Id: I90c122e48aa330753512a334883c85dbf9ef6cb6
---
A asterixdb/asterix-app/data/adm-load/utf8.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/load/utf8/utf8.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/load/utf8/utf8.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/load/utf8/utf8.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/load/utf8/utf8.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite.xml
6 files changed, 189 insertions(+), 0 deletions(-)


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

diff --git a/asterixdb/asterix-app/data/adm-load/utf8.adm 
b/asterixdb/asterix-app/data/adm-load/utf8.adm
new file mode 100644
index 000..2621bc4
--- /dev/null
+++ b/asterixdb/asterix-app/data/adm-load/utf8.adm
@@ -0,0 +1,100 @@
+{"id":"1","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"2","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"3","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"4","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"5","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"6","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"7","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"8","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"9","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"10","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"11","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"12","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"13","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"14","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"15","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"16","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"17","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"18","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"19","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"20","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"21","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"22","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"23","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"24","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"25","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"26","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"27","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"28","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"29","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"30","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"31","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"32","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"33","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"34","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"35","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"36","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"37","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"38","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"39","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"40","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"41","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"42","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"43","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"44","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"45","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"46","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"47","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"48","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"49","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"50","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"51","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"52","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"53","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"54","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}
+{"id":"55","description":"随着人们信用活动的繁荣、社会对信用服务需求的激增，构建一个完整"}

Change in asterixdb[master]: TweetParser Extension

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: TweetParser Extension
..


Patch Set 10: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7021e7b779de05b9ec999a8d5f8464fb0ab413c0
Gerrit-PatchSet: 10
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Xikui Wang 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Fuzzyjoin rules enabled with top-level CommonOperators and t...

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Fuzzyjoin rules enabled with top-level CommonOperators and the 
testCases covering multiway fuzzyJoin and joinSelect cases.
..


Patch Set 49:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia829133808deeaba3c847ac383aa8ae76fa5d87e
Gerrit-PatchSet: 49
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li 
Gerrit-Reviewer: Chen Li 
Gerrit-Reviewer: Inci Cetindil 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No