You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/01/22 00:05:14 UTC
[GitHub] [incubator-pinot] kishoreg commented on issue #4999: Make PQL use
LIMIT N if TOP N is not set
kishoreg commented on issue #4999: Make PQL use LIMIT N if TOP N is not set
URL: https://github.com/apache/incubator-pinot/pull/4999#issuecomment-576945190
> I think this will break certain LinkedIn use cases where people blindly put LIMIT 0 for group by queries. @mcvsubbu @mayankshriv Do you remember seeing such use cases?
limit 0 without TOP? if yes, maybe check for recordLimit > 0
----------------------------------------------------------------
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@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org