gianm commented on a change in pull request #6916: Add master/data/query server 
concepts to docs/packaging
URL: https://github.com/apache/incubator-druid/pull/6916#discussion_r251402735
 
 

 ##########
 File path: docs/content/configuration/index.md
 ##########
 @@ -1071,11 +1088,15 @@ Amazon's EC2 is currently the only supported 
autoscaler.
 |`nodeData`|A JSON object that describes how to launch new nodes.|none; 
required|
 |`userData`|A JSON object that describes how to configure new nodes. If you 
have set druid.indexer.autoscale.workerVersion, this must have a 
versionReplacementString. Otherwise, a versionReplacementString is not 
necessary.|none; optional|
 
-## MiddleManager and Peons
+## Data Server
+
+This section contains the configuration options for the processes that reside 
on data servers (middle managers/peons and historicals).
 
 Review comment:
   Some suggested style adjustments and clarifications:
   
   > This section contains the configuration options for the processes that 
reside on Data servers (MiddleManagers and Historicals) in the suggested 
`[three-server configuration](../design/index.html#server-types)`.
   
   I'd suggest similar changes for the Master and Query sections in this doc.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to