Jennifer88huang commented on a change in pull request #11406: URL: https://github.com/apache/pulsar/pull/11406#discussion_r673726397
########## File path: site2/docs/helm-overview.md ########## @@ -53,7 +53,7 @@ It includes support for: To get up and run with these charts as fast as possible, in a **non-production** use case, we provide a [quick start guide](getting-started-helm.md) for Proof of Concept (PoC) deployments. -This guide walks the user through deploying these charts with default values and features, but *does not* meet production ready requirements. To deploy these charts into production under sustained load, follow the complete [Installation Guide](helm-install.md). +This guide walks the user through deploying these charts with default values and features, but does not meet production ready requirements. To deploy these charts into production under sustained load, follow the complete [Installation Guide](helm-install.md). Review comment: ```suggestion This guide walks you through deploying these charts with default values and features; the default settings do not meet the production ready requirements. To deploy these charts into production under sustained load, follow the [installation guide](helm-install.md). ``` ########## File path: site2/docs/helm-overview.md ########## @@ -90,10 +90,10 @@ helm delete <pulsar-release-name> ``` For the purposes of continuity, these charts have some Kubernetes objects that cannot be removed when performing `helm delete`. -It is recommended to *conciously* remove these items, as they affect re-deployment. +It is recommended to remove these items, as they affect re-deployment. Review comment: ```suggestion It is recommended to remove these items, as they affect re-deployment. ``` ########## File path: site2/docs/helm-overview.md ########## @@ -53,7 +53,7 @@ It includes support for: To get up and run with these charts as fast as possible, in a **non-production** use case, we provide a [quick start guide](getting-started-helm.md) for Proof of Concept (PoC) deployments. -This guide walks the user through deploying these charts with default values and features, but *does not* meet production ready requirements. To deploy these charts into production under sustained load, follow the complete [Installation Guide](helm-install.md). +This guide walks the user through deploying these charts with default values and features, but does not meet production ready requirements. To deploy these charts into production under sustained load, follow the complete [Installation Guide](helm-install.md). Review comment: Try to use second person, for details, see https://developers.google.com/style/person -- 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: commits-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org