Repository: asterixdb
Updated Branches:
  refs/heads/master 107c2059d -> 31d03f078


[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 <jenk...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Reviewed-by: Michael Blow <mb...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/asterixdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/asterixdb/commit/31d03f07
Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/31d03f07
Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/31d03f07

Branch: refs/heads/master
Commit: 31d03f078fb5afe6bfb52e2c2788a89d7f333579
Parents: 107c205
Author: Murtadha Hubail <mhub...@apache.org>
Authored: Fri Aug 10 07:33:24 2018 -0700
Committer: Murtadha Hubail <mhub...@apache.org>
Committed: Fri Aug 10 08:26:03 2018 -0700

----------------------------------------------------------------------
 .../org/apache/asterix/api/http/server/QueryServiceServlet.java  | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/31d03f07/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
----------------------------------------------------------------------
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 @@ public class QueryServiceServlet extends 
AbstractQueryApiServlet {
                     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);

Reply via email to