nishantmonu51 opened a new pull request #10093:
URL: https://github.com/apache/druid/pull/10093


   ### Description
   
   This PR Fixes a bug with groupBy query cache key. The current cache key for 
groupby queries does not contain limitSpec info. So when a user queries with a 
given limit lets say 10 and then increase the limit to 100, he would still get 
the cached results with older limit. 
   
   Note: This only shows up if you have enabled caching for groupby queries, 
which is disabled by default. 
   
   This PR has:
   - [*] been self-reviewed.
   - [*] added unit tests or modified existing tests to cover new code paths, 
ensuring the threshold for [code 
coverage](https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md)
 is met.
   - [*] been tested in a test Druid cluster.
   


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