Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-28 Thread Murtadha Hubail (Code Review)
Murtadha Hubail has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

(5 comments)

https://asterix-gerrit.ics.uci.edu/#/c/3038/1//COMMIT_MSG
Commit Message:

Line 7: [NO ISSUE][API][MTD] Refactor query service servlet
\n


https://asterix-gerrit.ics.uci.edu/#/c/3038/1/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
File 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java:

PS1, Line 426: setFromJSON
A better place for this might be QueryServiceRequestParameters.ofJson(JsonNode 
request). But you may just rename this now to setParamFromJSON and that can be 
done on a later stage.


PS1, Line 432: setFromRequest
A better place for this might be 
QueryServiceRequestParameters.of(IServletRequest request). But you may just 
rename this now to setParamFromRequest and that can be done on a later stage.


PS1, Line 661: printPre
rename to printOptionalResultFields


https://asterix-gerrit.ics.uci.edu/#/c/3038/1/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
File 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java:

PS1, Line 1800: "`" + dvName + "`"
You may introduce a SQL++ escape method in a util and use it here.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: Yes


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-26 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Analytics Compatibility Tests Failed
https://goo.gl/Le5QJq : UNSTABLE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Murtadha Hubail 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1: Contrib-2

Analytics Compatibility Tests Failed
https://goo.gl/rB45Q5 : UNSTABLE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1: Contrib+1

BAD Compatibility Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/191/
 (13/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5150/ 
(11/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10245/ (6/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Analytics Compatibility Compilation Successful
https://goo.gl/s9EESS : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/190/
 (12/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/4764/ 
(8/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/4675/ 
(9/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/4985/ 
(7/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/5104/
 (1/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Ali Alsuliman (Code Review)
Ali Alsuliman has uploaded a new change for review.

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

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..

[NO ISSUE][API][MTD] Refactor query service servlet
- user model changes: no
- storage format changes: no
- interface changes: no

Details:
Refactor query service servlet in addition to some code
clean-ups.

Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
---
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/bootstrap/MetadataPrimaryIndexes.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/MetadataConstants.java
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/ConstantMergePolicyFactory.java
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/NoMergePolicyFactory.java
M 
hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/PrefixMergePolicyFactory.java
7 files changed, 106 insertions(+), 76 deletions(-)


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

diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
index 765ba9c..7aa24e3 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
@@ -411,62 +411,72 @@
 return result;
 }
 
-private QueryServiceRequestParameters getRequestParameters(IServletRequest 
request) throws IOException {
-final String contentType = HttpUtil.getContentTypeOnly(request);
+private QueryServiceRequestParameters 
createRequestParameters(IServletRequest request) throws IOException {
 QueryServiceRequestParameters param = new 
QueryServiceRequestParameters();
 param.setHost(host(request));
 param.setPath(servletPath(request));
+setRequestParam(request, param);
+return param;
+}
+
+protected void setRequestParam(IServletRequest request, 
QueryServiceRequestParameters param) throws IOException {
+String contentType = HttpUtil.getContentTypeOnly(request);
 if (HttpUtil.ContentType.APPLICATION_JSON.equals(contentType)) {
 try {
-JsonNode jsonRequest = 
OBJECT_MAPPER.readTree(HttpUtil.getRequestBody(request));
-final String statementParam = Parameter.STATEMENT.str();
-if (jsonRequest.has(statementParam)) {
-
param.setStatement(jsonRequest.get(statementParam).asText());
-}
-param.setFormat(toLower(getOptText(jsonRequest, 
Parameter.FORMAT.str(;
-param.setPretty(getOptBoolean(jsonRequest, 
Parameter.PRETTY.str(), false));
-param.setMode(toLower(getOptText(jsonRequest, 
Parameter.MODE.str(;
-param.setClientContextID(getOptText(jsonRequest, 
Parameter.CLIENT_ID.str()));
-param.setTimeout(getOptText(jsonRequest, 
Parameter.TIMEOUT.str()));
-param.setMaxResultReads(getOptText(jsonRequest, 
Parameter.MAX_RESULT_READS.str()));
-param.setPlanFormat(getOptText(jsonRequest, 
Parameter.PLAN_FORMAT.str()));
-param.setExpressionTree(getOptBoolean(jsonRequest, 
Parameter.EXPRESSION_TREE.str(), false));
-param.setRewrittenExpressionTree(
-getOptBoolean(jsonRequest, 
Parameter.REWRITTEN_EXPRESSION_TREE.str(), false));
-param.setLogicalPlan(getOptBoolean(jsonRequest, 
Parameter.LOGICAL_PLAN.str(), false));
-param.setOptimizedLogicalPlan(
-getOptBoolean(jsonRequest, 
Parameter.OPTIMIZED_LOGICAL_PLAN.str(), false));
-param.setJob(getOptBoolean(jsonRequest, Parameter.JOB.str(), 
false));
-param.setSignature(getOptBoolean(jsonRequest, 
Parameter.SIGNATURE.str(), true));
-param.setStatementParams(
-getOptStatementParameters(jsonRequest, 
jsonRequest.fieldNames(), JsonNode::get, v -> v));
-param.setMultiStatement(getOptBoolean(jsonRequest, 
Parameter.MULTI_STATEMENT.str(), true));
+setFromJSON(request, param);
 } catch (JsonParseException | JsonMappingException e) {
 // if the 

Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/4730/ 
(5/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5334/ (2/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/4766/
 (10/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/8712/ (3/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][API][MTD] Refactor query service servlet - user m...

2018-11-20 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][API][MTD] Refactor query service servlet - user 
model changes: no - storage format changes: no - interface changes: no
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/2774/ 
(4/13)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idbc81219cbe2f337b3c4aaf4a242ef2ebeb165c5
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No