jgutmann commented on a change in pull request #3975: ReadTheDocs documentation for Table Configs, Monitoring, and Deployment URL: https://github.com/apache/incubator-pinot/pull/3975#discussion_r268266909
########## File path: docs/in_production.rst ########## @@ -40,11 +40,14 @@ Recommended environment Deploying Pinot --------------- -Direct deployment of Pinot -~~~~~~~~~~~~~~~~~~~~~~~~~~ +In general, when deploying Pinot services, it is best to adhere to a specific ordering in which the various components should be deployed. This deployment order is recommended incase of the scenario that there might be protocol or other significant differences, the deployments go out in a predictable order in which failure due to these changes can be avoided. Review comment: I don't think that is the scope of this section. The way this was presented to me is that I need to describe how to deploy Pinot in terms of a version upgrade. Everything you are describing sounds like it should be part of an architecture section for how to implement Pinot ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
