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

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 9:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie9c5400fd134ae75d43385255af7794e968b1c7e
Gerrit-PatchSet: 9
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

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

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

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


Patch Set 9:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie9c5400fd134ae75d43385255af7794e968b1c7e
Gerrit-PatchSet: 9
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

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

2016-12-22 Thread Ian Maxon (Code Review) 
Hello Jenkins,

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

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

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

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 0f252915cb31f068d6b57adfb05e4854d6c219b3
Author: Ian Maxon 
Date:   Mon Dec 19 18:19:52 2016 -0800

fix regressions

commit 9b04bbbce9c2058959d3ffbf79f9f63e36a7dfe1
Author: Ian Maxon 
Date:   Fri Dec 16 17:24:00 2016 -0800

tests pass!?

commit 08093c6341af55b1e2434d56087982acb6a57a93
Author: Ian Maxon 
Date:   Thu Dec 1 17:53:15 2016 -0800

fix cluster api tests

commit 1b0051c67bcebd36e7800726170c6fd1fe1c575b
Author: Ian Maxon 
Date:   Fri Nov 18 17:45:06 2016 -0800

it compiles.

Change-Id: Ie9c5400fd134ae75d43385255af7794e968b1c7e
---
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveEvent.java
M asterixdb/asterix-active/src/main/java/org/apache/asterix/active/Activity.java
M 
asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActivityState.java
M asterixdb/asterix-algebra/pom.xml
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/IStatementExecutor.java
M asterixdb/asterix-app/pom.xml
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/http/servlet/ClusterAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ClusterCCDetailsAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ClusterNodeDetailsAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ConnectorAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/DiagnosticsAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryStatusAPIServlet.java
M 
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/api/http/servlet/ShutdownAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/VersionAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultPrinter.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/messaging/MessagingChannelInterfaceFactory.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/messaging/MessagingChannelReadInterface.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/messaging/MessagingChannelWriteInterface.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/ConnectorAPIServletTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/bootstrap/TestNodeController.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/data/gen/ABooleanFieldValueGenerator.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/data/gen/ARecordValueGenerator.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/data/gen/AStringFieldValueGenerator.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestHelper.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestTupleReference.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/logging/CheckpointingTest.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/any-object/any-object.2.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_3/cluster_state_3.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_4/cluster_state_4.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_cc_1/cluster_state_cc_1.1.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_cc_stats_1/cluster_state_cc_stats_1.1.regexadm
M

Change in asterixdb[master]: Cleaning up typos and clarifying examples in string function...

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Cleaning up typos and clarifying examples in string function 
doc.
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I944d9dd1d1e23608b6a596190c3d89e19cfacead
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Carey 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Steven Jacobs 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1720 - Generate License Files
..


Patch Set 8: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1720 - Generate License Files
..


Patch Set 8: BAD+1

BAD Compatibility Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/110/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1720 - Generate License Files
..


Patch Set 8:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/110/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1720 - Generate License Files
..


Patch Set 8:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: Cleaning up typos and clarifying examples in string function...

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Cleaning up typos and clarifying examples in string function 
doc.
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I944d9dd1d1e23608b6a596190c3d89e19cfacead
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Carey 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Steven Jacobs 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1720 - Generate License Files
..


Patch Set 8:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Michael Blow (Code Review) 
Hello Jenkins,

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

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

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

Change subject: ASTERIXDB-1720 - Generate License Files
..

ASTERIXDB-1720 - Generate License Files

- Includes Maven plugin to analyze dependencies & assemble LICENSE files
using Apache FreeMarker templates, formatting to desired LICENSE output
format.
- LICENSE files for the 'asterix-server', 'asterix-installer',
and 'asterix-yarn' binary assemblies are generated by the build
- Automated NOTICE file generation is not addressed in this patch
- Automated LICENSE file generation for source release is not addressed
  by this patch
- Fixes ASTERIXDB-1311: Add Rome Apache 2.0 License in the LICENSE/NOTICE

Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
---
M asterixdb/asterix-installer/pom.xml
M asterixdb/asterix-installer/src/main/assembly/binary-assembly.xml
D asterixdb/asterix-installer/src/main/licenses/LICENSE
A asterixdb/asterix-maven-plugins/license-automation-plugin/pom.xml
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/DependencySet.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/DownloadLicensesMojo.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/ExtraLicenseFile.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/GenerateLicenseFileMojo.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/LicenseMojo.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/LicenseSpec.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/LicenseUtil.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/Override.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/SupplementalModelHelper.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/freemarker/IndentDirective.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/freemarker/LoadFileDirective.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/project/LicensedProjects.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/project/Project.java
M asterixdb/asterix-maven-plugins/pom.xml
M asterixdb/asterix-server/pom.xml
M asterixdb/asterix-server/src/main/assembly/binary-assembly.xml
D asterixdb/asterix-server/src/main/licenses/LICENSE
M asterixdb/asterix-yarn/pom.xml
M asterixdb/asterix-yarn/src/main/assembly/binary-assembly.xml
D asterixdb/asterix-yarn/src/main/licenses/LICENSE
M asterixdb/src/main/appended-resources/supplemental-models.xml
A asterixdb/src/main/licenses/content/asm.objectweb.org_license.html.txt
A 
asterixdb/src/main/licenses/content/glassfish.dev.java.net_public_CDDL_GPL_1_1.html.txt
A asterixdb/src/main/licenses/content/hamcrest_1.3_LICENSE.txt
A asterixdb/src/main/licenses/content/jdom--jdom--1.0_licenses_jdom.txt
A asterixdb/src/main/licenses/content/jline.sourceforge.net_license.html.txt
A 
asterixdb/src/main/licenses/content/opensource.org_licenses_bsd-license.php.txt
A 
asterixdb/src/main/licenses/content/opensource.org_licenses_mit-license.php.txt
A asterixdb/src/main/licenses/content/originals/asm.objectweb.org_license.html
A 
asterixdb/src/main/licenses/content/originals/glassfish.dev.java.net_public_CDDL_GPL_1_1.html
A 
asterixdb/src/main/licenses/content/originals/jline.sourceforge.net_license.html
A asterixdb/src/main/licenses/content/originals/www.antlr.org_license.html
A 
asterixdb/src/main/licenses/content/originals/www.eclipse.org_legal_epl-v10.html
A asterixdb/src/main/licenses/content/originals/www.json.org_license.html
A asterixdb/src/main/licenses/content/originals/www.slf4j.org_license.html
A asterixdb/src/main/licenses/content/originals/www.sun.com_cddl_cddl.html
A asterixdb/src/main/licenses/content/protobuf_2.5.0_LICENSE.txt
A 
asterixdb/src/main/licenses/content/raw.githubusercontent.com_codehaus_paranamer-git_master_LICENSE.txt
A 
asterixdb/src/main/licenses/content/raw.githubusercontent.com_mojohaus_appassembler_master_LICENSE.txt
A asterixdb/src/main/licenses/content/stax-utils_LICENSE.txt
A asterixdb/src/main/licenses/content/www.antlr.org_license.html.txt
A asterixdb/src/main/licenses/content/www.apache.org_licenses_LICENSE-2.0.txt
A asterixdb/src/main/licenses/content/www.eclipse.org_legal_epl-v10.html.txt
A asterixdb/src/main/licenses/content/www.jcraft.com_jsch_LICENSE.txt
A

Change in asterixdb[master]: Full-text implementation step 3

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Full-text implementation step 3
..


Patch Set 14: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1087854ac7cf5b6ef5094e27a1646f12f6a8653f
Gerrit-PatchSet: 14
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Fix async result delivery

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Fix async result delivery
..


Patch Set 8: Integration-Tests+1

Integration Tests Successful

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

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

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

Change in asterixdb[master]: Full-text implementation step 3

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Full-text implementation step 3
..


Patch Set 14: BAD+1

BAD Compatibility Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/109/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1087854ac7cf5b6ef5094e27a1646f12f6a8653f
Gerrit-PatchSet: 14
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Fix async result delivery

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Fix async result delivery
..


Patch Set 8: BAD-1

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/108/ : UNSTABLE

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

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

Change in asterixdb[master]: Full-text implementation step 3

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Full-text implementation step 3
..


Patch Set 14:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1087854ac7cf5b6ef5094e27a1646f12f6a8653f
Gerrit-PatchSet: 14
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Full-text implementation step 3

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Full-text implementation step 3
..


Patch Set 14:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/109/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1087854ac7cf5b6ef5094e27a1646f12f6a8653f
Gerrit-PatchSet: 14
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Fix async result delivery

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Fix async result delivery
..


Patch Set 8:

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

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

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

Change in asterixdb[master]: Cleaning up typos and clarifying examples in string function...

2016-12-22 Thread Steven Jacobs (Code Review) 
Steven Jacobs has posted comments on this change.

Change subject: Cleaning up typos and clarifying examples in string function 
doc.
..


Patch Set 1: Code-Review+1

I think the build failure is a fluke. It's definitely not caused by this change 
:)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I944d9dd1d1e23608b6a596190c3d89e19cfacead
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Carey 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Steven Jacobs 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No

Change in asterixdb[master]: Full-text implementation step 3

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Full-text implementation step 3
..


Patch Set 14:

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1087854ac7cf5b6ef5094e27a1646f12f6a8653f
Gerrit-PatchSet: 14
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Full-text implementation step 3

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Full-text implementation step 3
..


Patch Set 14:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1087854ac7cf5b6ef5094e27a1646f12f6a8653f
Gerrit-PatchSet: 14
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Taewoo Kim 
Gerrit-HasComments: No

Change in asterixdb[master]: Full-text implementation step 3

2016-12-22 Thread Taewoo Kim (Code Review) 
Hello Jenkins,

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

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

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

Change subject: Full-text implementation step 3
..

Full-text implementation step 3

 - Implemented index-based full-text contains() function.
 - Fixed a bug in the full-text code that can't process join cases well.
 - Introduced full-text type index syntax. The functionality was already
   in the codebase but it was not being used. So, using this syntax, we
   can utilize the given index type - Not the length partitioned keyword index.

Change-Id: I1087854ac7cf5b6ef5094e27a1646f12f6a8653f
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.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/InvertedIndexAccessMethod.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/InvertedIndexJobGenParams.java
A 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/inverted-index-basic/fulltext-contains.aql
A 
asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-basic/fulltext-contains.plan
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-05/fulltext-05.1.ddl.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-06/fulltext-06.1.ddl.aql
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-07/fulltext-07.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-01/fulltext-index-01.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-01/fulltext-index-01.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-01/fulltext-index-01.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-01/fulltext-index-01.4.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-01/fulltext-index-01.5.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-02/fulltext-index-02.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-02/fulltext-index-02.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-02/fulltext-index-02.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-02/fulltext-index-02.4.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-02/fulltext-index-02.5.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-02/fulltext-index-02.6.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.4.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.5.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.6.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.7.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-03/fulltext-index-03.8.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-04/fulltext-index-04.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-04/fulltext-index-04.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-04/fulltext-index-04.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-05/fulltext-index-05.1.ddl.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-05/fulltext-index-05.2.update.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-05/fulltext-index-05.3.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/fulltext/fulltext-index-06/fulltext-index-06.1.ddl.aql
A

Change in asterixdb[master]: Fix async result delivery

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Fix async result delivery
..


Patch Set 8:

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

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

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

Change in asterixdb[master]: Fix async result delivery

2016-12-22 Thread Till Westmann (Code Review) 
Hello Jenkins,

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

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

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

Change subject: Fix async result delivery
..

Fix async result delivery

- add support for result status to the test framework
- add sleep function

Change-Id: Iafba65d9c7bd8643c42e5126c8d89164ae328908
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryStatusAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CompilerExtensionManager.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/IStatementExecutorExtension.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/DefaultStatementExecutorFactory.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/flwor/at00/at00.5.async.sqlpp
M 
asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/ResultExtractor.java
M 
asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/SleepTypeComputer.java
A 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/SleepDescriptor.java
13 files changed, 299 insertions(+), 61 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/94/1394/8
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1394
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iafba65d9c7bd8643c42e5126c8d89164ae328908
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Jenkins

Change in asterixdb[master]: Cleaning up typos and clarifying examples in string function...

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Cleaning up typos and clarifying examples in string function 
doc.
..


Patch Set 1:

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/107/ : UNSTABLE

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

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

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1720 - Generate License Files
..


Patch Set 7:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
Gerrit-PatchSet: 7
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Michael Blow 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No

Change in asterixdb[master]: ASTERIXDB-1720 - Generate License Files

2016-12-22 Thread Michael Blow (Code Review) 
Hello Jenkins,

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

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

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

Change subject: ASTERIXDB-1720 - Generate License Files
..

ASTERIXDB-1720 - Generate License Files

- Includes Maven plugin to analyze dependencies & assemble LICENSE files
using Apache FreeMarker templates, formatting to desired LICENSE output
format.
- LICENSE files for the 'asterix-server', 'asterix-installer',
and 'asterix-yarn' binary assemblies are generated by the build
- Automated NOTICE file generation is not addressed in this patch
- Automated LICENSE file generation for source release is not addressed
  by this patch
- Fixes ASTERIXDB-1311: Add Rome Apache 2.0 License in the LICENSE/NOTICE

Change-Id: I0963a85cb2be47dbf6bfd8c7f6fec767ef32e7e2
---
M asterixdb/asterix-installer/pom.xml
M asterixdb/asterix-installer/src/main/assembly/binary-assembly.xml
D asterixdb/asterix-installer/src/main/licenses/LICENSE
A asterixdb/asterix-maven-plugins/license-automation-plugin/pom.xml
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/DependencySet.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/DownloadLicensesMojo.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/ExtraLicenseFile.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/GenerateLicenseFileMojo.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/LicenseMojo.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/LicenseSpec.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/LicenseUtil.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/Override.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/SupplementalModelHelper.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/freemarker/IndentDirective.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/freemarker/LoadFileDirective.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/project/LicensedProjects.java
A 
asterixdb/asterix-maven-plugins/license-automation-plugin/src/main/java/org/apache/asterix/license/project/Project.java
M asterixdb/asterix-maven-plugins/pom.xml
M asterixdb/asterix-server/pom.xml
M asterixdb/asterix-server/src/main/assembly/binary-assembly.xml
D asterixdb/asterix-server/src/main/licenses/LICENSE
M asterixdb/asterix-yarn/pom.xml
M asterixdb/asterix-yarn/src/main/assembly/binary-assembly.xml
D asterixdb/asterix-yarn/src/main/licenses/LICENSE
M asterixdb/src/main/appended-resources/supplemental-models.xml
A asterixdb/src/main/licenses/content/asm.objectweb.org_license.html.txt
A 
asterixdb/src/main/licenses/content/glassfish.dev.java.net_public_CDDL_GPL_1_1.html.txt
A asterixdb/src/main/licenses/content/hamcrest_1.3_LICENSE.txt
A asterixdb/src/main/licenses/content/jdom--jdom--1.0_licenses_jdom.txt
A asterixdb/src/main/licenses/content/jline.sourceforge.net_license.html.txt
A 
asterixdb/src/main/licenses/content/opensource.org_licenses_bsd-license.php.txt
A 
asterixdb/src/main/licenses/content/opensource.org_licenses_mit-license.php.txt
A asterixdb/src/main/licenses/content/originals/asm.objectweb.org_license.html
A 
asterixdb/src/main/licenses/content/originals/glassfish.dev.java.net_public_CDDL_GPL_1_1.html
A 
asterixdb/src/main/licenses/content/originals/jline.sourceforge.net_license.html
A asterixdb/src/main/licenses/content/originals/www.antlr.org_license.html
A 
asterixdb/src/main/licenses/content/originals/www.eclipse.org_legal_epl-v10.html
A asterixdb/src/main/licenses/content/originals/www.json.org_license.html
A asterixdb/src/main/licenses/content/originals/www.slf4j.org_license.html
A asterixdb/src/main/licenses/content/originals/www.sun.com_cddl_cddl.html
A asterixdb/src/main/licenses/content/protobuf_2.5.0_LICENSE.txt
A 
asterixdb/src/main/licenses/content/raw.githubusercontent.com_codehaus_paranamer-git_master_LICENSE.txt
A 
asterixdb/src/main/licenses/content/raw.githubusercontent.com_mojohaus_appassembler_master_LICENSE.txt
A asterixdb/src/main/licenses/content/stax-utils_LICENSE.txt
A asterixdb/src/main/licenses/content/www.antlr.org_license.html.txt
A asterixdb/src/main/licenses/content/www.apache.org_licenses_LICENSE-2.0.txt
A asterixdb/src/main/licenses/content/www.eclipse.org_legal_epl-v10.html.txt
A asterixdb/src/main/licenses/content/www.jcraft.com_jsch_LICENSE.txt
A

[jira] [Commented] (ASTERIXDB-1711) Clean up class names

2016-12-22 Thread ASF subversion and git services (JIRA) 

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

ASF subversion and git services commented on ASTERIXDB-1711:


Commit 0294b3119a9e7e33cc92ec0a2b7e2adaf0e2ce27 in asterixdb's branch 
refs/heads/master from [~tillw]
[ https://git-wip-us.apache.org/repos/asf?p=asterixdb.git;h=0294b31 ]

ASTERIXDB-1711: remove more Aql-prefixes

AqlMissingWriterFactory -> MissingWriterFactory
AqlPositionWriter -> PositionWriter
AqlMergeAggregationExpressionFactory -> MergeAggregationExpressionFactory
AqlMissableTypeComputer -> MissableTypeComputer
AqlPartialAggregationTypeComputer -> PartialAggregationTypeComputer
AqlOptimizationContextFactory -> OptimizationContextFactory

Change-Id: I72f3051ff5c76ccde54e8b3b33941eb936870194
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1386
Tested-by: Jenkins 
BAD: Jenkins 
Reviewed-by: Yingyi Bu 


> Clean up class names
> 
>
> Key: ASTERIXDB-1711
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1711
> Project: Apache AsterixDB
>  Issue Type: Task
>  Components: AsterixDB, Hyracks
>Reporter: Till
>Assignee: Till
>
> Some classes have names that include "AQL" (even though they are not language 
> specific) or "Asterix" (even tough that's part of package names).
> We should aim to name classes based on their functionality and clean up class 
> names that are not named that way.



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

Change in asterixdb[master]: ASTERIXDB-1711: remove more Aql-prefixes

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

Change subject: ASTERIXDB-1711: remove more Aql-prefixes
..


ASTERIXDB-1711: remove more Aql-prefixes

AqlMissingWriterFactory -> MissingWriterFactory
AqlPositionWriter -> PositionWriter
AqlMergeAggregationExpressionFactory -> MergeAggregationExpressionFactory
AqlMissableTypeComputer -> MissableTypeComputer
AqlPartialAggregationTypeComputer -> PartialAggregationTypeComputer
AqlOptimizationContextFactory -> OptimizationContextFactory

Change-Id: I72f3051ff5c76ccde54e8b3b33941eb936870194
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1386
Tested-by: Jenkins 
BAD: Jenkins 
Reviewed-by: Yingyi Bu 
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlPlusExpressionToPlanTranslator.java
R 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/PositionWriter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SqlppExpressionToPlanTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
R 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/common/MergeAggregationExpressionFactory.java
R 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/common/MissableTypeComputer.java
R 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/common/PartialAggregationTypeComputer.java
R 
asterixdb/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/MissingWriterFactory.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/formats/NonTaggedDataFormat.java
11 files changed, 30 insertions(+), 31 deletions(-)

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

Objections:
  Jenkins: Violations found



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 d2982e8..3ef5f2b 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
@@ -25,7 +25,7 @@
 
 import org.apache.asterix.common.config.GlobalConfig;
 import org.apache.asterix.dataflow.data.common.ExpressionTypeComputer;
-import org.apache.asterix.dataflow.data.nontagged.AqlMissingWriterFactory;
+import org.apache.asterix.dataflow.data.nontagged.MissingWriterFactory;
 import org.apache.asterix.formats.nontagged.ADMPrinterFactoryProvider;
 import org.apache.asterix.formats.nontagged.BinaryBooleanInspector;
 import org.apache.asterix.formats.nontagged.BinaryComparatorFactoryProvider;
@@ -125,7 +125,7 @@
 SerializerDeserializerProvider.INSTANCE, 
BinaryHashFunctionFactoryProvider.INSTANCE,
 BinaryHashFunctionFamilyProvider.INSTANCE, 
BinaryComparatorFactoryProvider.INSTANCE,
 TypeTraitProvider.INSTANCE, BinaryBooleanInspector.FACTORY, 
BinaryIntegerInspector.FACTORY,
-ADMPrinterFactoryProvider.INSTANCE, 
AqlMissingWriterFactory.INSTANCE, null,
+ADMPrinterFactoryProvider.INSTANCE, MissingWriterFactory.INSTANCE, 
null,
 new 
LogicalExpressionJobGenToExpressionRuntimeProviderAdapter(QueryLogicalExpressionJobGen.INSTANCE),
 ExpressionTypeComputer.INSTANCE, null, null, null, null, 
GlobalConfig.DEFAULT_FRAME_SIZE, null);
 
diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlExpressionToPlanTranslator.java
 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlExpressionToPlanTranslator.java
index 175ff96..de17448 100644
--- 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlExpressionToPlanTranslator.java
+++ 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlExpressionToPlanTranslator.java
@@ -83,7 +83,7 @@
 LogicalVariable pVar = context.newVar(fc.getPosVarExpr());
 // We set the positional variable type as INT64 type.
 returnedOp = new UnnestOperator(v, new 
MutableObject(makeUnnestExpression(eo.first)),
-pVar, BuiltinType.AINT64, new AqlPositionWriter());
+pVar, BuiltinType.AINT64, new PositionWriter());
 }
 returnedOp.getInputs().add(eo.second);
 return new Pair<>(returnedOp, v);
diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlPlusExpressionToPlanTranslator.java

Change in asterixdb[master]: ASTERIXDB-1711: remove more Aql-prefixes

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

Change subject: ASTERIXDB-1711: remove more Aql-prefixes
..


Patch Set 5: Code-Review+2

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

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

Change in asterixdb[master]: Cleaning up typos and clarifying examples in string function...

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

Change subject: Cleaning up typos and clarifying examples in string function 
doc.
..


Patch Set 1: Code-Review+2

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

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

Change in asterixdb[master]: Cleaning up typos and clarifying examples in string function...

2016-12-22 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Cleaning up typos and clarifying examples in string function 
doc.
..


Patch Set 1:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/107/

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

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