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
