writer-jill commented on code in PR #16725:
URL: https://github.com/apache/druid/pull/16725#discussion_r1696701607


##########
docs/querying/lookups-cached-global.md:
##########
@@ -22,21 +26,22 @@ title: "Globally Cached Lookups"
   ~ under the License.
   -->
 
+Globally cached lookups all draw from the same cache pool, allowing each Druid 
process to have a fixed cache pool that can be used by cached lookups.
+
 To use this Apache Druid extension, 
[include](../configuration/extensions.md#loading-extensions) 
`druid-lookups-cached-global` in the extensions load list.
 
 ## Configuration
-:::info
- Static configuration is no longer supported. Lookups can be configured through
- [dynamic configuration](./lookups.md#configuration).
-:::
 
 Globally cached lookups are appropriate for lookups which are not possible to 
pass at query time due to their size,
 or are not desired to be passed at query time because the data is to reside in 
and be handled by the Druid servers,
 and are small enough to reasonably populate in-memory. This usually means tens 
to tens of thousands of entries per lookup.
 
-Globally cached lookups all draw from the same cache pool, allowing each 
process to have a fixed cache pool that can be used by cached lookups.
+:::info
+ Druid no longer supports static lookup configuration. You can configure 
lookups through
+ [dynamic configuration](./lookups.md#configuration).
+:::

Review Comment:
   yes, updated



-- 
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: [email protected]

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