cryptoe commented on code in PR #15382:
URL: https://github.com/apache/druid/pull/15382#discussion_r1395279702


##########
docs/api-reference/sql-api.md:
##########
@@ -390,12 +390,10 @@ Generally, the `sql` and `sql/statements` endpoints 
support the same response bo
 
 Keep the following in mind when submitting queries to the `sql/statements` 
endpoint:
 
-- There are additional context parameters  for `sql/statements`:
+- Apart from the context parameters mentioned 
[here](../multi-stage-query/reference.md#context-parameters) there are 
additional context parameters for `sql/statements` specifically:
 
    - `executionMode`  determines how query results are fetched. Druid 
currently only supports `ASYNC`. You must manually retrieve your results after 
the query completes.
-   - `selectDestination` determines where final results get written. By 
default, results are written to task reports. Set this parameter to 
`durableStorage` to instruct Druid to write the results from SELECT queries to 
durable storage, which allows you to fetch larger result sets. Note that this 
requires you to have [durable storage for MSQ 
enabled](../operations/durable-storage.md).
-
-- The only supported value for `resultFormat` is JSON LINES.
+   - `selectDestination` determines where final results get written. By 
default, results are written to task reports. Set this parameter to 
`durableStorage` to instruct Druid to write the results from SELECT queries to 
durable storage, which allows you to fetch larger result sets. For result sets 
with more than 3000 rows, it is highly recommended to use `durableStorage`. 
Note that this requires you to have [durable storage for MSQ 
enabled](../operations/durable-storage.md).

Review Comment:
   The notion was using the same value we use for task report result chomping. 
   `Limits.MAX_SELECT_RESULT_ROWS` so that task reports donot ever explode. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to