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-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-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-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));
-
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
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5213/ (3/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Analytics Compatibility Tests Failed https://goo.gl/y3EMTS : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit 'refs/changes/92/3192/1'
Michael Blow has posted comments on this change. Change subject: Merge commit 'refs/changes/92/3192/1' .. Patch Set 1: Code-Review-2 already done in https://asterix-gerrit.ics.uci.edu/#/c/3212/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3193 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie0aeae95d529c8540093642985c5fa9540ee03fe Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/593/ (12/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Michael Blow has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Verified+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Michael Blow has submitted this change and it was merged.
Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
..
[NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
- exercise non-UTF8 Accept-Charset in TestExecutor
- remove double-buffering on http responses
- minor refactoring / cleanup
Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3191
Reviewed-by: Murtadha Hubail
Sonar-Qube: Jenkins
Tested-by: Michael Blow
---
M asterixdb/asterix-app/pom.xml
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java
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/app/result/ResultPrinterTest.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/common/TestDataUtil.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/runtime/ParseDurationTest.java
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/CancellationTestExecutor.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/sqlpp/ParserTestExecutor.java
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/Duration.java
M
asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java
M
asterixdb/asterix-common/src/test/java/org/apache/asterix/test/server/RSSFeedServlet.java
M asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.sh
M asterixdb/asterix-server/src/main/opt/local/bin/stop-sample-cluster.sh
M
asterixdb/asterix-server/src/test/java/org/apache/asterix/test/server/SampleLocalClusterIT.java
M asterixdb/pom.xml
M hyracks-fullstack/hyracks/hyracks-http/pom.xml
M
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java
M
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/FullResponse.java
M
hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/utils/HttpUtil.java
M
hyracks-fullstack/hyracks/hyracks-http/src/test/java/org/apache/hyracks/test/http/HttpAcceptCharsetTest.java
M
hyracks-fullstack/hyracks/hyracks-http/src/test/java/org/apache/hyracks/test/http/HttpServerEncodingTest.java
A
hyracks-fullstack/hyracks/hyracks-test-support/src/main/java/org/apache/hyracks/test/string/EncodingUtils.java
25 files changed, 430 insertions(+), 452 deletions(-)
Approvals:
Jenkins: No violations found
Michael Blow: Verified
Murtadha Hubail: Looks good to me, approved
diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml
index 8ef9c45..281e1d4 100644
--- a/asterixdb/asterix-app/pom.xml
+++ b/asterixdb/asterix-app/pom.xml
@@ -666,7 +666,6 @@
org.apache.hyracks
hyracks-storage-am-lsm-invertedindex
- ${hyracks.version}
diff --git
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java
index 8fdcc41..7f74c92 100644
---
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java
+++
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java
@@ -18,12 +18,14 @@
*/
package org.apache.asterix.api.http.server;
+import java.io.IOException;
import java.util.concurrent.ConcurrentMap;
import org.apache.asterix.app.result.ResultHandle;
import org.apache.asterix.app.result.ResultReader;
import org.apache.asterix.common.api.IApplicationContext;
import org.apache.asterix.translator.IStatementExecutor.Stats;
+import org.apache.asterix.translator.SessionConfig;
import org.apache.asterix.translator.SessionOutput;
import org.apache.hyracks.api.exceptions.ErrorCode;
import org.apache.hyracks.api.exceptions.HyracksDataException;
@@ -91,7 +93,7 @@
// way to send the same OutputFormat value here as was
// originally determined there. Need to save this value on
// some object that we can obtain here.
-SessionOutput sessionOutput = RestApiServlet.initResponse(request,
response);
+SessionOutput sessionOutput = initResponse(request, response);
ResultUtil.printResults(appCtx, resultReader, sessionOutput, new
Stats(), null);
} catch (HyracksDataException e) {
final int errorCode = e.getErrorCode();
@@ -112,4 +114,82 @@
}
}
+/**
+ * Initialize the Content-Type of the response, and
[jira] [Commented] (ASTERIXDB-2519) Compiler error caused by index selection when search expression is not constant
[
https://issues.apache.org/jira/browse/ASTERIXDB-2519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16769899#comment-16769899
]
ASF subversion and git services commented on ASTERIXDB-2519:
Commit 99d970d01ab41263756d5063ee7fb460a24b2dfb in asterixdb's branch
refs/heads/master from Dmitry Lychagin
[ https://gitbox.apache.org/repos/asf?p=asterixdb.git;h=99d970d ]
[ASTERIXDB-2519][COMP] Fix compiler error with range predicate
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- Fix compiler error (Could not infer type for variable ...)
when non-constant expressions are used in the range predicate
Change-Id: I8896e6c0743dcd5b608ebd5a5f25d3077047c789
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3187
Sonar-Qube: Jenkins
Tested-by: Jenkins
Integration-Tests: Jenkins
Reviewed-by: Ali Alsuliman
> Compiler error caused by index selection when search expression is not
> constant
> ---
>
> Key: ASTERIXDB-2519
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2519
> Project: Apache AsterixDB
> Issue Type: Bug
> Components: COMP - Compiler
>Reporter: Dmitry Lychagin
>Assignee: Dmitry Lychagin
>Priority: Major
>
> # Create LineItem dataset
> # Create index on l_suppkey
> {noformat}
> create index idx_LineItem_suppkey on LineItem (l_suppkey) type btree;
> {noformat}
> # Run the following query
> {noformat}
> select value count(*)
> from LineItem as c
> where
> ((c.l_suppkey < 100 + tobigint(random()/2)) and
> (c.l_suppkey > 5 + tobigint(random()/2)));
> {noformat}
> # Error:
>{noformat}
>java.lang.Exception: ASX1079: Compilation error: Could not infer type for
> variable '$$44'.
>{noformat}
> There's a range search on indexed field and search expressions are not a
> constants
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/8005/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] enable configuration of external library dir
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE] enable configuration of external library dir .. Patch Set 1: Analytics Compatibility Tests Failed https://goo.gl/1Jbr3e : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/3210 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I31963a79dbae572cad1db930b5d97ebbbd76ee1b Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Xikui Wang Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9172/ (7/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5217/ (11/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Michael Blow has submitted this change and it was merged.
Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master'
..
Merge commit 'd9850dc' from 'stabilization-f69489' into 'master'
Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c
---
M asterixdb/asterix-app/pom.xml
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/ControllerConfig.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NCLogConfigurationFactory.java
3 files changed, 4 insertions(+), 39 deletions(-)
Approvals:
Jenkins: No violations found
Michael Blow: Looks good to me, approved; Verified
diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml
index 15c7b40..f74fbf3 100644
--- a/asterixdb/asterix-app/pom.xml
+++ b/asterixdb/asterix-app/pom.xml
@@ -287,12 +287,8 @@
asterix-gerrit-asterix-app
-<<< HEAD (6b0e2f [NO ISSUE][OTH] Do not send cancellation on null
clientConte)
-
**/SqlppExecutionWithCancellationTest.java,**/DmlTest.java,**/RepeatedTest.java,**/SqlppExecutionTest.java,**/*Compression*Test.java,**/*Ssl*Test.java
+
**/SqlppExecutionWithCancellationTest.java,**/DmlTest.java,**/RepeatedTest.java,**/SqlppExecutionTest.java,**/AqlExecutionTest.java,**/*Compression*Test.java,**/*Ssl*Test.java
**/*.java
-===
-
**/SqlppExecutionWithCancellationTest.java,**/DmlTest.java,**/RepeatedTest.java,**/SqlppExecutionTest.java,**/AqlExecutionTest.java
->>> BRANCH (d9850d [NO ISSUE][TEST] Use unique directory for external
library t)
@@ -340,11 +336,7 @@
asterix-gerrit-verify-asterix-app
-<<< HEAD (6b0e2f [NO ISSUE][OTH] Do not send cancellation on null
clientConte)
-**/*.java
-===
**/AqlExecutionTest.java
->>> BRANCH (d9850d [NO ISSUE][TEST] Use unique directory for external
library t)
**/SqlppExecution*IT.java,**/RebalanceWithCancellationIT.java
false
diff --git
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/ControllerConfig.java
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/ControllerConfig.java
index 028419f..f7291cb 100644
---
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/ControllerConfig.java
+++
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/ControllerConfig.java
@@ -18,12 +18,9 @@
*/
package org.apache.hyracks.control.common.controllers;
-<<< HEAD (6b0e2f [NO ISSUE][OTH] Do not send cancellation on null
clientConte)
import static org.apache.hyracks.control.common.config.OptionTypes.BOOLEAN;
-===
import java.io.File;
->>> BRANCH (d9850d [NO ISSUE][TEST] Use unique directory for external
library t)
import java.io.Serializable;
import java.net.URL;
import java.util.function.Function;
diff --git
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NCLogConfigurationFactory.java
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NCLogConfigurationFactory.java
index a427d30..a657d6b 100644
---
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NCLogConfigurationFactory.java
+++
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NCLogConfigurationFactory.java
@@ -18,10 +18,10 @@
*/
package org.apache.hyracks.control.nc;
+import java.io.File;
import java.net.URI;
import org.apache.hyracks.control.common.controllers.NCConfig;
-import org.apache.hyracks.util.file.FileUtil;
import org.apache.logging.log4j.Level;
import org.apache.logging.log4j.core.LoggerContext;
import org.apache.logging.log4j.core.config.Configuration;
@@ -32,12 +32,6 @@
import org.apache.logging.log4j.core.config.builder.api.ConfigurationBuilder;
import org.apache.logging.log4j.core.config.builder.api.LayoutComponentBuilder;
import org.apache.logging.log4j.core.config.builder.impl.BuiltConfiguration;
-<<< HEAD (6b0e2f [NO ISSUE][OTH] Do not send cancellation on null
clientConte)
-===
-
-import java.io.File;
-import java.net.URI;
->>> BRANCH (d9850d [NO ISSUE][TEST] Use unique directory for external
library t)
public class NCLogConfigurationFactory extends ConfigurationFactory {
private NCConfig config;
@@ -57,40 +51,22 @@
ComponentBuilder triggeringPolicy = builder.newComponent("Policies")
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Michael Blow has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Code-Review+2 Verified+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c Gerrit-PatchSet: 2 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[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app-stabilization-f69489/60/ (13/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9170/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3230/ (4/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Murtadha Hubail has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5127/ (10/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5588/ (9/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5796/ (8/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Hello Anon. E. Moose #1000171, Jenkins, Murtadha Hubail, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/3191 to look at the new patch set (#24). Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet - exercise non-UTF8 Accept-Charset in TestExecutor - remove double-buffering on http responses - minor refactoring / cleanup Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 --- M asterixdb/asterix-app/pom.xml M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java 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/app/result/ResultPrinterTest.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/common/TestDataUtil.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/runtime/ParseDurationTest.java M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/CancellationTestExecutor.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/sqlpp/ParserTestExecutor.java M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/Duration.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/server/RSSFeedServlet.java M asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.sh M asterixdb/asterix-server/src/main/opt/local/bin/stop-sample-cluster.sh M asterixdb/asterix-server/src/test/java/org/apache/asterix/test/server/SampleLocalClusterIT.java M asterixdb/pom.xml M hyracks-fullstack/hyracks/hyracks-http/pom.xml M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/ChunkedResponse.java M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/FullResponse.java M hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/utils/HttpUtil.java M hyracks-fullstack/hyracks/hyracks-http/src/test/java/org/apache/hyracks/test/http/HttpAcceptCharsetTest.java M hyracks-fullstack/hyracks/hyracks-http/src/test/java/org/apache/hyracks/test/http/HttpServerEncodingTest.java A hyracks-fullstack/hyracks/hyracks-test-support/src/main/java/org/apache/hyracks/test/string/EncodingUtils.java 25 files changed, 430 insertions(+), 452 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/91/3191/24 -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Michael Blow has uploaded a new patch set (#2). Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c --- M asterixdb/asterix-app/pom.xml M hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/ControllerConfig.java M hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NCLogConfigurationFactory.java 3 files changed, 4 insertions(+), 39 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/12/3212/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/5440/ (14/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/385/ (5/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-spidersilk-tests/217/ (15/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5440/ (6/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: 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/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5219/ (3/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Analytics Compatibility Compilation Successful https://goo.gl/MwUkcf : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5181/ (2/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' 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/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5216/ (12/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/592/ (13/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Jenkins has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 24: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10701/ (1/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 24 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5218/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/529/ (7/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/384/ (7/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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
[jira] [Commented] (ASTERIXDB-2520) Intermittent failure in replication: flushed_component test
[
https://issues.apache.org/jira/browse/ASTERIXDB-2520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16769886#comment-16769886
]
ASF subversion and git services commented on ASTERIXDB-2520:
Commit 527f141358ed36493b51d74a0516870131e077d8 in asterixdb's branch
refs/heads/master from Murtadha Makki Al Hubail
[ https://gitbox.apache.org/repos/asf?p=asterixdb.git;h=527f141 ]
[ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- Ensure thread-safety of metadata node takeover.
- To account for multiple failures of metadata node, make
dataset memory reservation idempotent.
Change-Id: I360226187e176ce3a0ccdcd7a1b611a01d906394
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3207
Sonar-Qube: Jenkins
Reviewed-by: Michael Blow
Integration-Tests: Jenkins
Tested-by: Jenkins
> Intermittent failure in replication: flushed_component test
> ---
>
> Key: ASTERIXDB-2520
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-2520
> Project: Apache AsterixDB
> Issue Type: Bug
> Components: REPL - Replication
>Affects Versions: 0.9.4
>Reporter: Murtadha Hubail
>Assignee: Murtadha Hubail
>Priority: Major
> Fix For: 0.9.5
>
> Attachments: info.log.gz
>
>
> The test is intermittently failing with:
> {noformat}
> 16:35:01.065 [Executor-10:asterix_nc1] ERROR
> org.apache.asterix.app.replication.message.MetadataNodeRequestMessage -
> Failed taking over metadata
> org.apache.asterix.common.exceptions.MetadataException: Memory is already
> reserved for dataset: 1 at
> org.apache.asterix.metadata.bootstrap.MetadataBootstrap.startUniverse(MetadataBootstrap.java:182)
> ~[asterix-metadata-0.9.5-SNAPSHOT.jar:0.9.5-SNAPSHOT] at
> org.apache.asterix.app.nc.NCAppRuntimeContext.initializeMetadata(NCAppRuntimeContext.java:432)
> ~[classes/:?] at
> org.apache.asterix.app.replication.message.MetadataNodeRequestMessage.handle(MetadataNodeRequestMessage.java:51)
> [classes/:?] at
> org.apache.asterix.messaging.NCMessageBroker.lambda$receivedMessage$0(NCMessageBroker.java:100)
> [classes/:?]
> {noformat}
> Logs attached.
> Â
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' 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/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent
Murtadha Hubail has submitted this change and it was merged.
Change subject: [ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent
..
[ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- Ensure thread-safety of metadata node takeover.
- To account for multiple failures of metadata node, make
dataset memory reservation idempotent.
Change-Id: I360226187e176ce3a0ccdcd7a1b611a01d906394
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3207
Sonar-Qube: Jenkins
Reviewed-by: Michael Blow
Integration-Tests: Jenkins
Tested-by: Jenkins
---
M
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/NcLifecycleCoordinator.java
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/DatasetMemoryManager.java
M
asterixdb/asterix-common/src/test/java/org/apache/asterix/test/context/DatasetMemoryManagerTest.java
3 files changed, 5 insertions(+), 12 deletions(-)
Approvals:
Jenkins: Verified; No violations found; Verified
Michael Blow: Looks good to me, approved
Objections:
Anon. E. Moose #1000171: Violations found
Jenkins: Violations found
diff --git
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/NcLifecycleCoordinator.java
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/NcLifecycleCoordinator.java
index 961593f..19ae171 100644
---
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/NcLifecycleCoordinator.java
+++
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/replication/NcLifecycleCoordinator.java
@@ -60,7 +60,7 @@
private static final Logger LOGGER = LogManager.getLogger();
protected IClusterStateManager clusterManager;
-protected String metadataNodeId;
+protected volatile String metadataNodeId;
protected Set pendingStartupCompletionNodes = new HashSet<>();
protected final ICCMessageBroker messageBroker;
private final boolean replicationEnabled;
@@ -157,7 +157,7 @@
}
@Override
-public void notifyMetadataNodeChange(String node) throws
HyracksDataException {
+public synchronized void notifyMetadataNodeChange(String node) throws
HyracksDataException {
if (metadataNodeId.equals(node)) {
return;
}
@@ -203,7 +203,7 @@
return tasks;
}
-private void process(MetadataNodeResponseMessage response) throws
HyracksDataException {
+private synchronized void process(MetadataNodeResponseMessage response)
throws HyracksDataException {
// rebind metadata node since it might be changing
MetadataManager.INSTANCE.rebindMetadataNode();
clusterManager.updateMetadataNode(response.getNodeId(),
response.isExported());
diff --git
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/DatasetMemoryManager.java
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/DatasetMemoryManager.java
index 1b4b48e..8aec240 100644
---
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/DatasetMemoryManager.java
+++
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/DatasetMemoryManager.java
@@ -82,7 +82,8 @@
@Override
public synchronized boolean reserve(int datasetId) {
if (reservedMap.containsKey(datasetId)) {
-throw new IllegalStateException("Memory is already reserved for
dataset: " + datasetId);
+LOGGER.info("Memory is already reserved for dataset: {}", () ->
datasetId);
+return true;
}
final long required = getTotalSize(datasetId);
if (!isAllocatable(required) && !allocatedMap.containsKey(datasetId)) {
diff --git
a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/context/DatasetMemoryManagerTest.java
b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/context/DatasetMemoryManagerTest.java
index b8e3604..2a26f08 100644
---
a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/context/DatasetMemoryManagerTest.java
+++
b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/context/DatasetMemoryManagerTest.java
@@ -83,15 +83,7 @@
Assert.assertEquals(memoryManager.getAvailable(), expectedBudget);
// double reserve
-boolean thrown = false;
Assert.assertTrue(memoryManager.reserve(2));
-try {
-memoryManager.reserve(2);
-} catch (IllegalStateException e) {
-Assert.assertTrue(e.getMessage().contains("already reserved"));
-thrown = true;
-}
-Assert.assertTrue(thrown);
// cancel reserved
memoryManager.cancelReserved(2);
--
To view, visit https://asterix-gerrit.ics.uci.edu/3207
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5126/ (11/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Analytics Compatibility Compilation Successful https://goo.gl/KwKaQP : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5795/ (6/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5180/ (9/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-stabilization-f69489-compat/530/ (8/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3229/ (5/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10700/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9171/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Analytics Compatibility Compilation Successful https://goo.gl/RBoZXC : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5439/ (10/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5587/ (3/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/5215/ (12/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/5125/ (11/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent .. Patch Set 4: BAD Compatibility Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3985/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/3207 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I360226187e176ce3a0ccdcd7a1b611a01d906394 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Michael Blow has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/3212
Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master'
..
Merge commit 'd9850dc' from 'stabilization-f69489' into 'master'
Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c
---
M asterixdb/NOTICE
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java
M asterixdb/asterix-app/pom.xml
M
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
M asterixdb/src/main/licenses/templates/asterix-notice.ftl
M hyracks-fullstack/NOTICE
M
hyracks-fullstack/hyracks-fullstack-license/src/main/licenses/templates/hyracks-notice.ftl
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/ControllerConfig.java
M
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/NCLogConfigurationFactory.java
M hyracks-fullstack/hyracks/hyracks-server/pom.xml
M
hyracks-fullstack/hyracks/hyracks-server/src/test/java/org/apache/hyracks/test/server/NCServiceIT.java
M
hyracks-fullstack/hyracks/hyracks-server/src/test/java/org/apache/hyracks/test/server/process/HyracksCCProcess.java
M
hyracks-fullstack/hyracks/hyracks-server/src/test/java/org/apache/hyracks/test/server/process/HyracksNCServiceProcess.java
M
hyracks-fullstack/hyracks/hyracks-server/src/test/java/org/apache/hyracks/test/server/process/HyracksServerProcess.java
M
hyracks-fullstack/hyracks/hyracks-server/src/test/java/org/apache/hyracks/test/server/process/HyracksVirtualCluster.java
M
hyracks-fullstack/hyracks/hyracks-server/src/test/resources/NCServiceIT/cc.conf
M
hyracks-fullstack/hyracks/hyracks-server/src/test/resources/NCServiceIT/nc-blue.conf
M
hyracks-fullstack/hyracks/hyracks-server/src/test/resources/NCServiceIT/nc-red.conf
M hyracks-fullstack/pom.xml
19 files changed, 156 insertions(+), 54 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/12/3212/1
diff --git a/asterixdb/NOTICE b/asterixdb/NOTICE
index 3f18a61..7615782 100644
--- a/asterixdb/NOTICE
+++ b/asterixdb/NOTICE
@@ -1,5 +1,5 @@
Apache AsterixDB
-Copyright 2015-2018 The Apache Software Foundation
+Copyright 2015-2019 The Apache Software Foundation
This product includes software developed at
The Apache Software Foundation (http://www.apache.org/).
diff --git
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java
index 8d6f04d..c5ff5d1 100644
---
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java
+++
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java
@@ -330,8 +330,10 @@
LimitType[] highKeyLimits = new LimitType[numSecondaryKeys];
boolean[] lowKeyInclusive = new boolean[numSecondaryKeys];
boolean[] highKeyInclusive = new boolean[numSecondaryKeys];
-ILogicalExpression[] constantAtRuntimeExpressions = new
ILogicalExpression[numSecondaryKeys];
-LogicalVariable[] constAtRuntimeExprVars = new
LogicalVariable[numSecondaryKeys];
+ILogicalExpression[] lowKeyConstAtRuntimeExpressions = new
ILogicalExpression[numSecondaryKeys];
+ILogicalExpression[] highKeyConstantAtRuntimeExpressions = new
ILogicalExpression[numSecondaryKeys];
+LogicalVariable[] lowKeyConstAtRuntimeExprVars = new
LogicalVariable[numSecondaryKeys];
+LogicalVariable[] highKeyConstAtRuntimeExprVars = new
LogicalVariable[numSecondaryKeys];
/* TODO: For now we don't do any sophisticated analysis of the func
exprs to come up with "the best" range
* predicate. If we can't figure out how to integrate a certain
funcExpr into the current predicate,
@@ -368,13 +370,6 @@
ILogicalExpression searchKeyExpr = returnedSearchKeyExpr.first;
ILogicalExpression searchKeyEQExpr = null;
boolean realTypeConvertedToIntegerType =
returnedSearchKeyExpr.third;
-if (searchKeyExpr.getExpressionTag() ==
LogicalExpressionTag.FUNCTION_CALL) {
-constantAtRuntimeExpressions[keyPos] = searchKeyExpr;
-constAtRuntimeExprVars[keyPos] = context.newVar();
-VariableReferenceExpression varRef = new
VariableReferenceExpression(constAtRuntimeExprVars[keyPos]);
-varRef.setSourceLocation(searchKeyExpr.getSourceLocation());
-searchKeyExpr = varRef;
-}
LimitType limit = getLimitType(optFuncExpr, probeSubTree);
if (limit == null) {
@@ -394,6 +389,24 @@
} else if (limit == LimitType.LOW_EXCLUSIVE) {
Change in asterixdb[master]: Merge commit 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-txnlog/383/ (8/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10699/ (2/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5179/ (9/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/5794/ (6/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/5586/ (5/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5438/ (10/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/3228/ (4/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c 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 'd9850dc' from 'stabilization-f69489' into 'mas...
Jenkins has posted comments on this change. Change subject: Merge commit 'd9850dc' from 'stabilization-f69489' into 'master' .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5217/ (1/16) -- To view, visit https://asterix-gerrit.ics.uci.edu/3212 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia35c75e8c35720dfd22436f4d08f6986e3aa1b3c Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Analytics Compatibility Compilation Successful https://goo.gl/LMTnbD : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app-stabilization-f69489/59/ (11/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Michael Blow has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Code-Review+2 Verified+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2520][RT] Make Dataset Memory Reservation Idempotent .. Patch Set 4: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3985/ -- To view, visit https://asterix-gerrit.ics.uci.edu/3207 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I360226187e176ce3a0ccdcd7a1b611a01d906394 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-openjdk11/591/ (10/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE] enable configuration of external library dir
Jenkins has posted comments on this change. Change subject: [NO ISSUE] enable configuration of external library dir .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/8003/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/3210 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I31963a79dbae572cad1db930b5d97ebbbd76ee1b Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Xikui Wang Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet
Murtadha Hubail has posted comments on this change. Change subject: [NO ISSUE] Handle Accept-Charset in QueryResultApiServlet .. Patch Set 23: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/3191 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f37eb684bf2457e5ff451bf5c8fbca742d531f2 Gerrit-PatchSet: 23 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/9169/ (7/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/10698/ (1/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/5437/ (9/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Hello Jenkins, Murtadha Hubail, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/3211 to look at the new patch set (#4). Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. [NO ISSUE][TEST] Use unique directory for external library tests Enables external library tests on hosts w/ a shared home directory Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 --- M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java 1 file changed, 11 insertions(+), 0 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/11/3211/4 -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/5216/ (2/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[stabilization-f69489]: [NO ISSUE][TEST] Use unique directory for external library t...
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TEST] Use unique directory for external library tests .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/5178/ (8/13) -- To view, visit https://asterix-gerrit.ics.uci.edu/3211 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I575cfc79f4df831783ce6551ae691d73410ca878 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Michael Blow Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Ian Maxon Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
