317brian commented on code in PR #12459:
URL: https://github.com/apache/druid/pull/12459#discussion_r855557912


##########
docs/configuration/index.md:
##########
@@ -1369,7 +1369,7 @@ Middle managers pass their configurations down to their 
child peons. The MiddleM
 |`druid.indexer.runner.endPort`|Ending port used for peon processes, should be 
greater than or equal to `druid.indexer.runner.startPort` and less than 
65536.|65535|
 |`druid.indexer.runner.ports`|A JSON array of integers to specify ports that 
used for peon processes. If provided and non-empty, ports for peon processes 
will be chosen from these ports. And 
`druid.indexer.runner.startPort/druid.indexer.runner.endPort` will be 
completely ignored.|`[]`|
 |`druid.worker.ip`|The IP of the worker.|localhost|
-|`druid.worker.version`|Version identifier for the MiddleManager.|0|
+|`druid.worker.version`|Version identifier for the MiddleManager. Note that 
the version number is a string. This affects the expected behavior during 
certain operations like when it's compared against 
`druid.indexer.runner.minWorkerVersion`. Specifically, the version comparison 
follows dictionary order. Use ISO8601 date format for the version so that later 
versions are recognized as being more recent.|0|

Review Comment:
   ```suggestion
   |`druid.worker.version`|Version identifier for the MiddleManager. The 
version number is a string. This affects the expected behavior during certain 
operations like comparison against `druid.indexer.runner.minWorkerVersion`. 
Specifically, the version comparison follows dictionary order. Use ISO8601 date 
format for the version to accomodate data comparisons.|0|
   ```



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