[GitHub] [incubator-druid] clintropolis commented on a change in pull request #8196: GroupBy array-based result rows.

[GitBox <gi...@apache.org>]

clintropolis commented on a change in pull request #8196: GroupBy array-based result rows.
URL: https://github.com/apache/incubator-druid/pull/8196#discussion_r309098348
 
 

 ##########
 File path: server/src/main/java/org/apache/druid/server/QueryResource.java
 ##########
 @@ -163,7 +164,9 @@ public Response cancelQuery(@PathParam("id") String queryId, @Context final Http
   public Response doPost(
       final InputStream in,
       @QueryParam("pretty") final String pretty,
-      @Context final HttpServletRequest req // used to get request content-type,Accept header, remote address and auth-related headers
+
+      // used to get request content-type,Accept header, remote address and auth-related headers
 
 Review comment:
   nit: `.. request content-type,Accept header, ..` -> `.. request content-type, Accept header, ..` 
   
   (i know it was already like that, so imo only fix if you have something else to change).

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org