kfaraz commented on code in PR #18752: URL: https://github.com/apache/druid/pull/18752#discussion_r2540430094
########## processing/src/main/java/org/apache/druid/query/http/ClientSqlQuery.java: ########## @@ -39,6 +39,9 @@ public class ClientSqlQuery @JsonProperty private final String resultFormat; + /** + * Indicates whether the query result should include a header. + */ Review Comment: Alternative to this, we may also just rename the getter method to `includeHeader` similar to how it is done in class `SqlQuery` (which is the Broker-side equivalent of this POJO). (this change may be done in a separate PR). -- 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]
