xvrl commented on code in PR #14827:
URL: https://github.com/apache/druid/pull/14827#discussion_r1303400380


##########
docs/configuration/index.md:
##########
@@ -2103,18 +2103,18 @@ In addition to the normal cache metrics, the caffeine 
cache implementation also
 
 Uses memcached as cache backend. This allows all processes to share the same 
cache.
 
-| Property                      | Description                                  
                                                                  | Default     
      |
-|-------------------------------|----------------------------------------------------------------------------------------------------------------|-------------------|
-| `druid.cache.expiration`      | Memcached [expiration 
time](https://code.google.com/p/memcached/wiki/NewCommands#Standard_Protocol).  
         | 2592000 (30 days) |
-| `druid.cache.timeout`         | Maximum time in milliseconds to wait for a 
response from Memcached.                                            | 500       
        |
-| `druid.cache.hosts`           | Comma separated list of Memcached hosts 
`<host:port>`.                                                         | none   
           |
-| `druid.cache.maxObjectSize`   | Maximum object size in bytes for a Memcached 
object.                                                           | 52428800 
(50 MiB) |
-| `druid.cache.memcachedPrefix` | Key prefix for all keys in Memcached.        
                                                                  | druid       
      |
-| `druid.cache.numConnections`  | Number of memcached connections to use.      
                                                                  | 1           
      |
-| `druid.cache.protocol`        | Memcached communication protocol. Can be 
binary or text.                                                       | binary  
          |
-| `druid.cache.locator`         | Memcached locator. Can be consistent or 
array_mod.                                                             | 
consistent        |
-| `druid.cache.enableTls`       | Use TLS based ssl connection for Memcached 
client.  Boolean                                                    | false     
        |
-| `druid.cache.clientMode`      | ClientMode. Dynamic mode uses AutoDiscovery 
feature of AWS Elasticache Memcache. String. "static" or "dynamic" | static     
       |
+| Property                      | Description                                  
                                                          | Default           |
+|-------------------------------|--------------------------------------------------------------------------------------------------------|-------------------|
+| `druid.cache.expiration`      | Memcached [expiration 
time](https://code.google.com/p/memcached/wiki/NewCommands#Standard_Protocol).  
 | 2592000 (30 days) |
+| `druid.cache.timeout`         | Maximum time in milliseconds to wait for a 
response from Memcached.                                    | 500               
|
+| `druid.cache.hosts`           | Comma separated list of Memcached hosts 
`<host:port>`.                                                 | none           
   |

Review Comment:
   can we update the documentation here to explain that list of hosts applies 
for `static` mode and how that changes when clientMode is set to `dynamic`?



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