maytasm commented on a change in pull request #11761:
URL: https://github.com/apache/druid/pull/11761#discussion_r720449998



##########
File path: docs/querying/sql.md
##########
@@ -877,7 +877,7 @@ Submit your query as the value of a "query" field in the 
JSON object within the
 |--------|----|-----------|
 |`query`|SQL query string.| none (required)|
 |`resultFormat`|Format of query results. See [Responses](#responses) for 
details.|`"object"`|
-|`header`|Whether or not to include a header. See [Responses](#responses) for 
details.|`false`|
+|`header`|Whether or not to include a header for the query response. See 
[Responses](#responses) for details.|`false`|

Review comment:
       I think this is confusing. The `header` here is talking about the column 
names of each column in the result. This is different from the header of the 
HTTP response. I think we should make it clear here.




-- 
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