Murtadha Hubail has submitted this change and it was merged. Change subject: [NO ISSUE][FAIL] Return Bad Request on No Statement ......................................................................
[NO ISSUE][FAIL] Return Bad Request on No Statement - user model changes: no - storage format changes: no - interface changes: no Change-Id: I7a1f0cb61f792d9a40efb0c23ac8679df659f474 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2873 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Reviewed-by: Michael Blow <[email protected]> --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins: Verified; No violations found Michael Blow: Looks good to me, approved 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 c05ce6f..a32f8be 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 @@ -19,6 +19,7 @@ package org.apache.asterix.api.http.server; import static org.apache.asterix.common.exceptions.ErrorCode.ASTERIX; +import static org.apache.asterix.common.exceptions.ErrorCode.NO_STATEMENT_PROVIDED; import static org.apache.asterix.common.exceptions.ErrorCode.REJECT_BAD_CLUSTER_STATE; import static org.apache.asterix.common.exceptions.ErrorCode.REJECT_NODE_UNREGISTERED; import static org.apache.asterix.common.exceptions.ErrorCode.REQUEST_TIMEOUT; @@ -626,6 +627,9 @@ LOGGER.warn("handleException: {}: {}", he.getMessage(), param); state.setStatus(ResultStatus.FATAL, HttpResponseStatus.SERVICE_UNAVAILABLE); break; + case ASTERIX + NO_STATEMENT_PROVIDED: + state.setStatus(ResultStatus.FATAL, HttpResponseStatus.BAD_REQUEST); + break; default: LOGGER.warn("handleException: unexpected exception {}: {}", he.getMessage(), param, he); state.setStatus(ResultStatus.FATAL, HttpResponseStatus.INTERNAL_SERVER_ERROR); -- To view, visit https://asterix-gerrit.ics.uci.edu/2873 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a1f0cb61f792d9a40efb0c23ac8679df659f474 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]>
