Re: [I] [multistage] attach default limit 10 to queries that doesn't have a limit [pinot]

["guzzijones (via GitHub)" <gi...@apache.org>]

guzzijones commented on issue #12304:
URL: https://github.com/apache/pinot/issues/12304#issuecomment-2603286496

   idea of limiting a query to 10 records by default is flawed.  
   
   1.  A user who runs a query without a limit may not know there are more records.
   2.  A user who runs a query setting the limit may also not know there are more records.  
   
   In the end users bypass this by:
   1. setting some unusually high arbitrary limit and hope that the record count never exceeds it
   2. run 2 queries for a query.  
       1. first count the records 
       2. run the query originally requested with `limit [record count]` set
   
   the second option of course gets more complicated with window functions, group by , etc.


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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