suneet-s commented on a change in pull request #10363:
URL: https://github.com/apache/druid/pull/10363#discussion_r485190171



##########
File path: docs/configuration/index.md
##########
@@ -1379,7 +1379,7 @@ These Historical configurations can be defined in the 
`historical/runtime.proper
 |Property|Description|Default|
 |--------|-----------|-------|
 |`druid.segmentCache.locations`|Segments assigned to a Historical process are 
first stored on the local file system (in a disk cache) and then served by the 
Historical process. These locations define where that local cache resides. This 
value cannot be NULL or EMPTY. Here is an example 
`druid.segmentCache.locations=[{"path": "/mnt/druidSegments", "maxSize": "10k", 
"freeSpacePercent": 1.0}]`. "freeSpacePercent" is optional, if provided then 
enforces that much of free disk partition space while storing segments. But, it 
depends on File.getTotalSpace() and File.getFreeSpace() methods, so enable if 
only if they work for your File System.| none |
-|`druid.segmentCache.locationSelectorStrategy`|The strategy used to select a 
location from the configured `druid.segmentCache.locations` for segment 
distribution. Possible values are `leastBytesUsed`, `roundRobin`, `random`, or 
`mostAvailableSize`. |leastBytesUsed|
+|`druid.segmentCache.locationSelectorStrategy.type`|The strategy used to 
select a location from the configured `druid.segmentCache.locations` for 
segment distribution. Possible values are `leastBytesUsed`, `roundRobin`, 
`random`, or `mostAvailableSize`. |leastBytesUsed|

Review comment:
       Is this a doc fix? I don't see an associated change in the code. Am I 
missing something?




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