[GitHub] [druid] gianm commented on pull request #10233: Add "offset" parameter to the Scan query.

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

gianm commented on pull request #10233:
URL: https://github.com/apache/druid/pull/10233#issuecomment-668170209


   > @gianm, this is a great addition! Do you think it also makes sense to add `OFFSET` to other query types, namely timeseries/groupBy/topN queries?
   > 
   > I could see timeseries queries returning large result sets, so I think it'd be useful to throw away stuff that is not needed by a client, thereby reducing the # of bytes transferred over the wire. What do you think?
   
   I was thinking of starting with Scan and GroupBy. Timeseries could make sense too. TopN, I'm not as sure about, because the fact that offset needs to bump up limit means that it will start to increase memory use pretty quickly as you go to the later pages.


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



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