You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/09/22 08:48:19 UTC
[GitHub] [pulsar-helm-chart] naveen1100 opened a new pull request #63: Make forceSync by default as "yes"
naveen1100 opened a new pull request #63:
URL: https://github.com/apache/pulsar-helm-chart/pull/63
### Motivation
* It's not recommended to run a production zookkeeper cluster with forceSync as "no". This is also mentioned in the forceSync section in https://pulsar.apache.org/docs/en/next/reference-configuration/#zookeeper
### Modifications
* Set ```-Dzookeeper.forceSync=yes``` as default in ```values.yaml```
### Verifying this change
- [ ] Make sure that the change passes the CI checks.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] sijie merged pull request #63: Make forceSync by default as "yes"
Posted by GitBox <gi...@apache.org>.
sijie merged pull request #63:
URL: https://github.com/apache/pulsar-helm-chart/pull/63
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] sijie merged pull request #63: Make forceSync by default as "yes"
Posted by GitBox <gi...@apache.org>.
sijie merged pull request #63:
URL: https://github.com/apache/pulsar-helm-chart/pull/63
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] naveen1100 commented on pull request #63: Make forceSync by default as "yes"
Posted by GitBox <gi...@apache.org>.
naveen1100 commented on pull request #63:
URL: https://github.com/apache/pulsar-helm-chart/pull/63#issuecomment-696599408
> you can simply drop the -Dzookeeper.forceSync line
@EladDolev Done
----------------------------------------------------------------
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:
users@infra.apache.org