nishantmonu51 commented on pull request #10093:
URL: https://github.com/apache/druid/pull/10093#issuecomment-653539435


   @jihoonson : Yes, its an issue when limitSpec if pushed down, other thing to 
note, if you push down the limit once and then try to execute a query without 
pushing down the limit or remove the limit itself, then also it will be 
problematic as the cache will have the results with limit and the results will 
be served from cache. 
   
   Updated the PR description about the default value.  


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to