[GitHub] [druid] sthetland commented on a change in pull request #11370: Clarify that Broker caching for groupBy v2 queries does not work

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

sthetland commented on a change in pull request #11370:
URL: https://github.com/apache/druid/pull/11370#discussion_r661795616



##########
File path: docs/configuration/index.md
##########
@@ -1773,6 +1773,9 @@ You can optionally only configure caching to be enabled on the Broker by setting
 
 See [cache configuration](#cache-configuration) for how to configure cache settings.
 
+> Note: Even if cache is enabled, Brokers never cache group-by queries when [groupBy v2 strategy](../querying/groupbyquery.md#strategies), which is the default, is configured.

Review comment:
       ```suggestion
   > Note: Even if cache is enabled, Brokers never cache groupBy queries when [groupBy v2 strategy](../querying/groupbyquery.md#strategies), which is the default, is configured.
   ```




-- 
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@druid.apache.org

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