adarshsanjeev commented on PR #14712:
URL: https://github.com/apache/druid/pull/14712#issuecomment-1659937763

   Tried this out locally and it works great! Way better than manually making 
posts to get results.
   
   I noticed something only while downloading the results, the format saved is 
as a json file, though according to the ResultFormat, ObjectLines is not valid 
json. (Firefox pointed this out too)
   
https://github.com/apache/druid/blob/master/sql/src/main/java/org/apache/druid/sql/http/ResultFormat.java#L96
   Should the file extension be changed here?
   
   I think part of the reason might be that the get results api is tagged as 
`@Produces(MediaType.APPLICATION_JSON)`. I think I missed this while changing 
the default format. I'm not sure if the console takes this into account while 
deciding the file extension. 


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