Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/219/ (16/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Michael Blow has submitted this change and it was merged.
Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master'
..
Merge commit '5543132' from 'stabilization-f69489' into 'master'
Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21
---
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
D
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RestApiServlet.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/ResultExtractor.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
M
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/utils/HttpUtil.java
7 files changed, 65 insertions(+), 409 deletions(-)
Approvals:
Jenkins: No violations found
Michael Blow: Looks good to me, approved; Verified
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 5327d62..ffbb614 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
@@ -349,33 +349,29 @@
pw.print("\t}\n");
}
-<<< HEAD (093e84 Merge "Merge commit 'd9850dc' from
'stabilization-f69489' in)
-protected String getOptText(JsonNode node, String fieldName) {
-===
-private String getOptText(JsonNode node, Parameter parameter) {
+protected String getOptText(JsonNode node, Parameter parameter) {
return getOptText(node, parameter.str());
}
-private String getOptText(JsonNode node, String fieldName) {
->>> BRANCH (554313 [NO ISSUE] Handle Accept-Charset in
QueryResultApiServlet)
+protected String getOptText(JsonNode node, String fieldName) {
final JsonNode value = node.get(fieldName);
return value != null ? value.asText() : null;
}
-private boolean getOptBoolean(JsonNode node, Parameter parameter, boolean
defaultValue) {
+protected boolean getOptBoolean(JsonNode node, Parameter parameter,
boolean defaultValue) {
return getOptBoolean(node, parameter.str(), defaultValue);
}
-private boolean getOptBoolean(JsonNode node, String fieldName, boolean
defaultValue) {
+protected boolean getOptBoolean(JsonNode node, String fieldName, boolean
defaultValue) {
final JsonNode value = node.get(fieldName);
return value != null ? value.asBoolean() : defaultValue;
}
-private String getParameter(IServletRequest request, Parameter parameter) {
+protected String getParameter(IServletRequest request, Parameter
parameter) {
return request.getParameter(parameter.str());
}
-private boolean getOptBoolean(IServletRequest request, Parameter
parameter, boolean defaultValue) {
+protected boolean getOptBoolean(IServletRequest request, Parameter
parameter, boolean defaultValue) {
String value = request.getParameter(parameter.str());
return value == null ? defaultValue : Boolean.parseBoolean(value);
}
@@ -422,83 +418,37 @@
String contentType = HttpUtil.getContentTypeOnly(request);
if (HttpUtil.ContentType.APPLICATION_JSON.equals(contentType)) {
try {
-<<< HEAD (093e84 Merge "Merge commit 'd9850dc' from
'stabilization-f69489' in)
setParamFromJSON(request, param, optionalParams);
-===
-JsonNode jsonRequest =
OBJECT_MAPPER.readTree(HttpUtil.getRequestBody(request));
-param.setStatement(getOptText(jsonRequest,
Parameter.STATEMENT));
-param.setFormat(toLower(getOptText(jsonRequest,
Parameter.FORMAT)));
-param.setPretty(getOptBoolean(jsonRequest, Parameter.PRETTY,
false));
-param.setMode(toLower(getOptText(jsonRequest,
Parameter.MODE)));
-param.setClientContextID(getOptText(jsonRequest,
Parameter.CLIENT_ID));
-param.setTimeout(getOptText(jsonRequest, Parameter.TIMEOUT));
-param.setMaxResultReads(getOptText(jsonRequest,
Parameter.MAX_RESULT_READS));
-param.setPlanFormat(getOptText(jsonRequest,
Parameter.PLAN_FORMAT));
-param.setExpressionTree(getOptBoolean(jsonRequest,
Parameter.EXPRESSION_TREE, false));
-param.setRewrittenExpressionTree(
-getOptBoolean(jsonRequest,
Parameter.REWRITTEN_EXPRESSION_TREE, false));
-param.setLogicalPlan(getOptBoolean(jsonReque
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Michael Blow has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Code-Review+2 Verified+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ssl-compression/51/ (15/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/8008/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/389/ (8/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Analytics Compatibility Compilation Successful https://goo.gl/bAPUQZ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5221/ (13/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/5443/ (14/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. 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/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5131/ (12/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5592/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/597/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9176/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5800/ (6/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3234/ (5/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/534/ (7/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10705/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/3213 to look at the new patch set (#3). Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Merge commit '5543132' from 'stabilization-f69489' into 'master' Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java D asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RestApiServlet.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/ResultExtractor.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/utils/HttpUtil.java 7 files changed, 65 insertions(+), 409 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/13/3213/3 -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5223/ (10/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5444/ (9/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5185/ (11/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: -Contrib Analytics Compatibility Compilation Successful https://goo.gl/qCG4Me : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10704/ (8/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/596/ (13/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5220/ (12/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. 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/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5591/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5130/ (11/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Contrib-2 Analytics Compatibility Compilation Failed https://goo.gl/wHXvmv : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5222/ (5/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/533/ (10/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5799/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5443/ (6/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/3213 to look at the new patch set (#2). Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Merge commit '5543132' from 'stabilization-f69489' into 'master' Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java D asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RestApiServlet.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/ResultExtractor.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/utils/HttpUtil.java 5 files changed, 53 insertions(+), 400 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/13/3213/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/388/ (9/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9175/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3233/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5184/ (7/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3987/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Contrib-2 BAD Compatibility Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3987/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/8006/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Contrib-2 Analytics Compatibility Tests Failed https://goo.gl/qe7KTA : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/8006/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ssl-compression/49/ (16/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5218/ (15/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Analytics Compatibility Compilation Successful https://goo.gl/a4k2Me : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/531/ (12/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10702/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. 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/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5128/ (14/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5797/ (6/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5441/ (9/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9173/ (7/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5182/ (10/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/5441/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Michael Blow has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/3213
Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master'
..
Merge commit '5543132' from 'stabilization-f69489' into 'master'
Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21
---
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
D
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RestApiServlet.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/ResultExtractor.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
M
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/utils/HttpUtil.java
5 files changed, 38 insertions(+), 395 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/13/3213/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 5327d62..ffbb614 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
@@ -349,33 +349,29 @@
pw.print("\t}\n");
}
-<<< HEAD (093e84 Merge "Merge commit 'd9850dc' from
'stabilization-f69489' in)
-protected String getOptText(JsonNode node, String fieldName) {
-===
-private String getOptText(JsonNode node, Parameter parameter) {
+protected String getOptText(JsonNode node, Parameter parameter) {
return getOptText(node, parameter.str());
}
-private String getOptText(JsonNode node, String fieldName) {
->>> BRANCH (554313 [NO ISSUE] Handle Accept-Charset in
QueryResultApiServlet)
+protected String getOptText(JsonNode node, String fieldName) {
final JsonNode value = node.get(fieldName);
return value != null ? value.asText() : null;
}
-private boolean getOptBoolean(JsonNode node, Parameter parameter, boolean
defaultValue) {
+protected boolean getOptBoolean(JsonNode node, Parameter parameter,
boolean defaultValue) {
return getOptBoolean(node, parameter.str(), defaultValue);
}
-private boolean getOptBoolean(JsonNode node, String fieldName, boolean
defaultValue) {
+protected boolean getOptBoolean(JsonNode node, String fieldName, boolean
defaultValue) {
final JsonNode value = node.get(fieldName);
return value != null ? value.asBoolean() : defaultValue;
}
-private String getParameter(IServletRequest request, Parameter parameter) {
+protected String getParameter(IServletRequest request, Parameter
parameter) {
return request.getParameter(parameter.str());
}
-private boolean getOptBoolean(IServletRequest request, Parameter
parameter, boolean defaultValue) {
+protected boolean getOptBoolean(IServletRequest request, Parameter
parameter, boolean defaultValue) {
String value = request.getParameter(parameter.str());
return value == null ? defaultValue : Boolean.parseBoolean(value);
}
@@ -422,83 +418,37 @@
String contentType = HttpUtil.getContentTypeOnly(request);
if (HttpUtil.ContentType.APPLICATION_JSON.equals(contentType)) {
try {
-<<< HEAD (093e84 Merge "Merge commit 'd9850dc' from
'stabilization-f69489' in)
setParamFromJSON(request, param, optionalParams);
-===
-JsonNode jsonRequest =
OBJECT_MAPPER.readTree(HttpUtil.getRequestBody(request));
-param.setStatement(getOptText(jsonRequest,
Parameter.STATEMENT));
-param.setFormat(toLower(getOptText(jsonRequest,
Parameter.FORMAT)));
-param.setPretty(getOptBoolean(jsonRequest, Parameter.PRETTY,
false));
-param.setMode(toLower(getOptText(jsonRequest,
Parameter.MODE)));
-param.setClientContextID(getOptText(jsonRequest,
Parameter.CLIENT_ID));
-param.setTimeout(getOptText(jsonRequest, Parameter.TIMEOUT));
-param.setMaxResultReads(getOptText(jsonRequest,
Parameter.MAX_RESULT_READS));
-param.setPlanFormat(getOptText(jsonRequest,
Parameter.PLAN_FORMAT));
-param.setExpressionTree(getOptBoolean(jsonRequest,
Parameter.EXPRESSION_TREE, false));
-param.setRewrittenExpressionTree(
-getOptBoolean(jsonRequest,
Parameter.REWRITTEN_EXPRESSION_TREE, false));
-param.setLogicalPlan(getOptBoolean(jsonRequest,
Parameter.LOGICAL_PLAN, false));
-param.setOptimizedLogicalPlan(getOptBoolean(jsonRequest,
Parameter.OPTIMIZED_LOGICAL_PLAN, false));
-pa
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/594/ (5/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/218/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5589/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3231/ (8/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5220/ (11/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit '5543132' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit '5543132' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/386/ (13/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3213 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0e77df94ecea518e19b55edb90a2fe8157cac21 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
