[GitHub] jerrypeng commented on a change in pull request #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster

2018-06-11 Thread GitBox
jerrypeng commented on a change in pull request #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster URL: https://github.com/apache/incubator-pulsar/pull/1950#discussion_r194587655 ## File path:

[GitHub] massakam commented on a change in pull request #1921: Simplified authentication wiht Python client lib

2018-06-11 Thread GitBox
massakam commented on a change in pull request #1921: Simplified authentication wiht Python client lib URL: https://github.com/apache/incubator-pulsar/pull/1921#discussion_r194593897 ## File path: pulsar-client-cpp/python/pulsar_test.py ## @@ -114,6 +117,31 @@ def

[GitHub] massakam commented on issue #1921: Simplified authentication wiht Python client lib

2018-06-11 Thread GitBox
massakam commented on issue #1921: Simplified authentication wiht Python client lib URL: https://github.com/apache/incubator-pulsar/pull/1921#issuecomment-396437848 All the tests succeeded in my local environment by modifying `pulsar_test.py` as follows:

[GitHub] sijie commented on issue #1845: Functions schema integration

2018-06-11 Thread GitBox
sijie commented on issue #1845: Functions schema integration URL: https://github.com/apache/incubator-pulsar/pull/1845#issuecomment-396444551 retest this please This is an automated message from the Apache Git Service. To

[GitHub] jerrypeng commented on a change in pull request #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster

2018-06-11 Thread GitBox
jerrypeng commented on a change in pull request #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster URL: https://github.com/apache/incubator-pulsar/pull/1950#discussion_r194577854 ## File path:

[GitHub] sijie commented on issue #1942: Fix topic name in getting started documentation

2018-06-11 Thread GitBox
sijie commented on issue #1942: Fix topic name in getting started documentation URL: https://github.com/apache/incubator-pulsar/pull/1942#issuecomment-396443880 @merlimat what is your take here? shall we merge this first to avoid user confusions? then we can consider a better solution?

[GitHub] sijie commented on issue #1941: Tiered Storage documentation

2018-06-11 Thread GitBox
sijie commented on issue #1941: Tiered Storage documentation URL: https://github.com/apache/incubator-pulsar/pull/1941#issuecomment-396443975 @srkukarni @merlimat please review the latest PR This is an automated message from

[GitHub] Horaddrim commented on issue #1090: Allow to configure all options when using Kafka wrapper

2018-06-11 Thread GitBox
Horaddrim commented on issue #1090: Allow to configure all options when using Kafka wrapper URL: https://github.com/apache/incubator-pulsar/issues/1090#issuecomment-396446054 I'll try to do it :D Any firstcommer help? This

[GitHub] srkukarni commented on issue #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster

2018-06-11 Thread GitBox
srkukarni commented on issue #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster URL: https://github.com/apache/incubator-pulsar/pull/1950#issuecomment-396411794 retest this please This is an automated

[GitHub] sijie commented on issue #1944: Add CLI commands for schema registry

2018-06-11 Thread GitBox
sijie commented on issue #1944: Add CLI commands for schema registry URL: https://github.com/apache/incubator-pulsar/pull/1944#issuecomment-396444050 retest this please This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1944: Add CLI commands for schema registry

2018-06-11 Thread GitBox
sijie commented on issue #1944: Add CLI commands for schema registry URL: https://github.com/apache/incubator-pulsar/pull/1944#issuecomment-396444097 @srkukarni @merlimat can you review this PR again? This is an automated

[GitHub] sijie commented on issue #1921: Simplified authentication wiht Python client lib

2018-06-11 Thread GitBox
sijie commented on issue #1921: Simplified authentication wiht Python client lib URL: https://github.com/apache/incubator-pulsar/pull/1921#issuecomment-396444153 retest this please This is an automated message from the Apache

[GitHub] sijie commented on issue #1871: Compaction documentation follow-up

2018-06-11 Thread GitBox
sijie commented on issue #1871: Compaction documentation follow-up URL: https://github.com/apache/incubator-pulsar/pull/1871#issuecomment-396444300 @ivankelly let's merge this and can you fix the comment. so we can move it forward with 2.1 release.

[GitHub] massakam commented on a change in pull request #1921: Simplified authentication wiht Python client lib

2018-06-11 Thread GitBox
massakam commented on a change in pull request #1921: Simplified authentication wiht Python client lib URL: https://github.com/apache/incubator-pulsar/pull/1921#discussion_r194611751 ## File path: pulsar-client-cpp/python/src/authentication.cc ## @@ -18,14 +18,42 @@ */

[GitHub] merlimat closed pull request #1858: Cpp client: add readCompacted in consumer config

2018-06-11 Thread GitBox
merlimat closed pull request #1858: Cpp client: add readCompacted in consumer config URL: https://github.com/apache/incubator-pulsar/pull/1858 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[incubator-pulsar] branch master updated: Cpp client: add readCompacted in consumer config (#1858)

2018-06-11 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 8ab6c34 Cpp client: add

[incubator-pulsar] branch master updated: fix: jackson conflict dependency version (#1947)

2018-06-11 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 6289cbc fix: jackson conflict

[GitHub] merlimat commented on issue #1921: Simplified authentication wiht Python client lib

2018-06-11 Thread GitBox
merlimat commented on issue #1921: Simplified authentication wiht Python client lib URL: https://github.com/apache/incubator-pulsar/pull/1921#issuecomment-396316966 Thanks @massakam. Applied your suggestion This is an

[GitHub] mgodave commented on a change in pull request #1944: Add CLI commands for schema registry

2018-06-11 Thread GitBox
mgodave commented on a change in pull request #1944: Add CLI commands for schema registry URL: https://github.com/apache/incubator-pulsar/pull/1944#discussion_r194467804 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdSchemas.java ## @@

[GitHub] merlimat closed pull request #1947: fix: jackson conflict dependency version

2018-06-11 Thread GitBox
merlimat closed pull request #1947: fix: jackson conflict dependency version URL: https://github.com/apache/incubator-pulsar/pull/1947 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] sijie closed pull request #1948: Documentation for threshold based compaction

2018-06-11 Thread GitBox
sijie closed pull request #1948: Documentation for threshold based compaction URL: https://github.com/apache/incubator-pulsar/pull/1948 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[incubator-pulsar] branch master updated: adding avro schema (#1917)

2018-06-11 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 2dae33d adding avro schema (#1917)

[GitHub] merlimat closed pull request #1917: adding avro schema

2018-06-11 Thread GitBox
merlimat closed pull request #1917: adding avro schema URL: https://github.com/apache/incubator-pulsar/pull/1917 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[incubator-pulsar] branch master updated: Documentation for threshold based compaction (#1948)

2018-06-11 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 673db8b Documentation for threshold

[GitHub] srkukarni opened a new pull request #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster

2018-06-11 Thread GitBox
srkukarni opened a new pull request #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster URL: https://github.com/apache/incubator-pulsar/pull/1950 ### Motivation Provide a simple mechanism to run pulsar functions in Kubernetes. ### Modifications

[GitHub] srkukarni commented on issue #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster

2018-06-11 Thread GitBox
srkukarni commented on issue #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster URL: https://github.com/apache/incubator-pulsar/pull/1950#issuecomment-396395396 @cckellogg This is an automated message

[GitHub] dsambandam opened a new issue #1949: Zookeeper won't start on IPV6 machines - UnknownHostException

2018-06-11 Thread GitBox
dsambandam opened a new issue #1949: Zookeeper won't start on IPV6 machines - UnknownHostException URL: https://github.com/apache/incubator-pulsar/issues/1949 Expected behavior Successfully start zookeeprs on a IPV6 configured machine Actual behavior Fails to

[GitHub] sijie commented on issue #1949: Zookeeper won't start on IPV6 machines - UnknownHostException

2018-06-11 Thread GitBox
sijie commented on issue #1949: Zookeeper won't start on IPV6 machines - UnknownHostException URL: https://github.com/apache/incubator-pulsar/issues/1949#issuecomment-396396469 add "-Djava.net.preferIPv4Stack=true" to the java command opts might help.

[GitHub] srkukarni commented on issue #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster

2018-06-11 Thread GitBox
srkukarni commented on issue #1950: Enable Pulsar Functions to be deployed on a kubernetes cluster URL: https://github.com/apache/incubator-pulsar/pull/1950#issuecomment-396402554 retest this please This is an automated

[incubator-pulsar] branch asf-site updated: Updated site at revision 3db0643

2018-06-11 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/asf-site by this push: new 3055177 Updated site at revision

[GitHub] massakam commented on issue #1921: Simplified authentication wiht Python client lib

2018-06-11 Thread GitBox
massakam commented on issue #1921: Simplified authentication wiht Python client lib URL: https://github.com/apache/incubator-pulsar/pull/1921#issuecomment-396188648 > TypeError: No registered converter was able to produce a C++ rvalue of type AuthenticationWrapper from this Python object

[GitHub] ivankelly commented on issue #1942: Fix topic name in getting started documentation

2018-06-11 Thread GitBox
ivankelly commented on issue #1942: Fix topic name in getting started documentation URL: https://github.com/apache/incubator-pulsar/pull/1942#issuecomment-396217011 @sijie @merlimat Another user hit same issue today. Python module is broken for 2.0, and when they roll back to 1.22 the

[GitHub] ivankelly commented on issue #1941: Tiered Storage documentation

2018-06-11 Thread GitBox
ivankelly commented on issue #1941: Tiered Storage documentation URL: https://github.com/apache/incubator-pulsar/pull/1941#issuecomment-396231982 @lucperkins had missed that. Will pull some of it in. This is an automated

[GitHub] ivankelly commented on a change in pull request #1941: Tiered Storage documentation

2018-06-11 Thread GitBox
ivankelly commented on a change in pull request #1941: Tiered Storage documentation URL: https://github.com/apache/incubator-pulsar/pull/1941#discussion_r194394917 ## File path: site/docs/latest/cookbooks/tiered-storage.md ## @@ -0,0 +1,119 @@ +--- +title: Tiered Storage

[GitHub] ivankelly commented on a change in pull request #1941: Tiered Storage documentation

2018-06-11 Thread GitBox
ivankelly commented on a change in pull request #1941: Tiered Storage documentation URL: https://github.com/apache/incubator-pulsar/pull/1941#discussion_r194395414 ## File path: site/docs/latest/cookbooks/tiered-storage.md ## @@ -0,0 +1,119 @@ +--- +title: Tiered Storage

[GitHub] ivankelly commented on a change in pull request #1941: Tiered Storage documentation

2018-06-11 Thread GitBox
ivankelly commented on a change in pull request #1941: Tiered Storage documentation URL: https://github.com/apache/incubator-pulsar/pull/1941#discussion_r194395567 ## File path: site/docs/latest/cookbooks/tiered-storage.md ## @@ -0,0 +1,119 @@ +--- +title: Tiered Storage

[GitHub] ivankelly opened a new pull request #1948: Documentation for threshold based compaction

2018-06-11 Thread GitBox
ivankelly opened a new pull request #1948: Documentation for threshold based compaction URL: https://github.com/apache/incubator-pulsar/pull/1948 Updates for reference and cookbook documentation for compaction to show users how to set up automatic compaction based on a size threshold.