gianm commented on a change in pull request #8881: New `OBJECTARRAY`
implementation for `SqlQuery.resultFormat`
URL: https://github.com/apache/druid/pull/8881#discussion_r367155396
##########
File path: sql/src/main/java/org/apache/druid/sql/http/ResultFormat.java
##########
@@ -100,6 +100,20 @@ public Writer createFormatter(final OutputStream
outputStream, final ObjectMappe
{
return new ObjectLinesWriter(outputStream, jsonMapper);
}
+ },
+
+ OBJECTARRAY {
Review comment:
This should be added to the doc page `docs/querying/sql.md`. I don't think
the name is very clear though. Maybe "wrappedObject" would be clearer in
relation to the other ones?
- object: array of objects
- array: array of arrays
- wrappedObject: object containing an array of objects
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
