You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Yingyi Bu (Code Review)" <do...@asterixdb.incubator.apache.org> on 2016/10/08 06:15:52 UTC
Change in asterixdb[master]: Make pretty as the default for JSON requests.
Yingyi Bu has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1264
Change subject: Make pretty as the default for JSON requests.
......................................................................
Make pretty as the default for JSON requests.
Change-Id: I09b2ffd3ce4702711542faf69ddfde3f3b0739e2
---
M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/64/1264/1
diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
index 427e177..a04fd07 100644
--- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
+++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
@@ -418,7 +418,7 @@
JSONObject jsonRequest = new JSONObject(getRequestBody(request));
param.statement = jsonRequest.getString(Parameter.STATEMENT.str());
param.format = toLower(jsonRequest.optString(Parameter.FORMAT.str()));
- param.pretty = jsonRequest.optBoolean(Parameter.PRETTY.str());
+ param.pretty = jsonRequest.optBoolean(Parameter.PRETTY.str(), true);
param.clientContextID = jsonRequest.optString(Parameter.CLIENT_ID.str());
} catch (JSONException e) {
// if the JSON parsing fails, the statement is empty and we get an empty statement error
--
To view, visit https://asterix-gerrit.ics.uci.edu/1264
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I09b2ffd3ce4702711542faf69ddfde3f3b0739e2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Change in asterixdb[master]: Make pretty as the default for JSON requests.
Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.
Change subject: Make pretty as the default for JSON requests.
......................................................................
Patch Set 1:
Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2960/
--
To view, visit https://asterix-gerrit.ics.uci.edu/1264
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I09b2ffd3ce4702711542faf69ddfde3f3b0739e2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No
Change in asterixdb[master]: Make pretty as the default for JSON requests.
Posted by "Yingyi Bu (Code Review)" <do...@asterixdb.incubator.apache.org>.
Yingyi Bu has abandoned this change.
Change subject: Make pretty as the default for JSON requests.
......................................................................
Abandoned
--
To view, visit https://asterix-gerrit.ics.uci.edu/1264
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: I09b2ffd3ce4702711542faf69ddfde3f3b0739e2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>