[GitHub] [pulsar-manager] rvashishth commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
rvashishth commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-697154042 isn't MariaDB compatible with MySql implementation, it would be nice to use the MySQL with pulsar-manager without maintaining a cloned source?

[GitHub] [pulsar-manager] tuteng commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
tuteng commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-697066906 I'm sorry for the confusion. When we donated the project to apache, because of the conflict between the license of MySQL and the license of apache, we deleted the dependence

[GitHub] [pulsar-client-node] massakam merged pull request #121: Includes Support for ZSTD and SNAPPY Compression

2020-09-22 Thread GitBox
massakam merged pull request #121: URL: https://github.com/apache/pulsar-client-node/pull/121 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

[GitHub] [pulsar-manager] sannadi99 commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
sannadi99 commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-696713595 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [pulsar-helm-chart] naveen1100 commented on pull request #63: Make forceSync by default as "yes"

2020-09-22 Thread GitBox
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 messag

[GitHub] [pulsar-manager] eolivelli commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
eolivelli commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-696690803 @sannadi99 did you run the tests using MySQL ? I am not sure it works. Probably it will work because HerdDB is compatible with MySQL dialect. --

[GitHub] [pulsar-helm-chart] sijie merged pull request #63: Make forceSync by default as "yes"

2020-09-22 Thread GitBox
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

[GitHub] [pulsar-client-node] massakam merged pull request #121: Includes Support for ZSTD and SNAPPY Compression

2020-09-22 Thread GitBox
massakam merged pull request #121: URL: https://github.com/apache/pulsar-client-node/pull/121 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

[GitHub] [pulsar-manager] tuteng commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
tuteng commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-697066906 I'm sorry for the confusion. When we donated the project to apache, because of the conflict between the license of MySQL and the license of apache, we deleted the dependence

[GitHub] [pulsar-manager] sannadi99 commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
sannadi99 commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-696921416 @eolivelli If my approach is correct please help me with this PR approval or if it is not a correct way can you help me on using custom mysql database for pulsar manager?

[GitHub] [pulsar-helm-chart] naveen1100 opened a new pull request #64: changed publishNotReadyAddresses to service spec from annotation

2020-09-22 Thread GitBox
naveen1100 opened a new pull request #64: URL: https://github.com/apache/pulsar-helm-chart/pull/64 ### Motivation * ```publishNotReadyAddresses``` is a service spec and not a service annotation. This is mentioned in the K8s API docs at https://v1-17.docs.kubernetes.io/docs/reference

[GitHub] [pulsar-helm-chart] sijie merged pull request #63: Make forceSync by default as "yes"

2020-09-22 Thread GitBox
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

[GitHub] [pulsar-manager] sannadi99 commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
sannadi99 commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-696713595 @sannadi99 did you run the tests using MySQL ? I am not sure it works. @eolivelli Yes i did and it worked for me in my local (mysql server and pulsar manager run

[GitHub] [pulsar-manager] eolivelli commented on pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
eolivelli commented on pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338#issuecomment-696690803 @sannadi99 did you run the tests using MySQL ? I am not sure it works. Probably it will work because HerdDB is compatible with MySQL dialect. --

[GitHub] [pulsar-manager] sannadi99 opened a new pull request #338: Adding mysql connector dependencies

2020-09-22 Thread GitBox
sannadi99 opened a new pull request #338: URL: https://github.com/apache/pulsar-manager/pull/338 Fixes #Unable to find mysql jdbc driver dependency Master Issue: #Missing mysql jdbc connector dependency ### Motivation When i tried using mysql custom database for pulsar

Slack digest for #dev - 2020-09-22

2020-09-22 Thread Apache Pulsar Slack
2020-09-21 12:04:06 UTC - Vil: what is difference between auditorPeriodicBookieCheckInterval and auditorPeriodicCheckInterval ? 2020-09-21 12:05:10 UTC - Vil: and pulsar enables auto recovery in bookeeper yes or no by default? it is disabled in pulsar’s conf/broker.conf and in bookkeepers c

[GitHub] [pulsar-helm-chart] naveen1100 opened a new pull request #63: Make forceSync by default as "yes"

2020-09-22 Thread GitBox
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/