[GitHub] benjaminhuo commented on issue #2858: What official pulsar helm chart request 15GB memory for bookeeper, zookeeper and broker?

2018-10-28 Thread GitBox
benjaminhuo commented on issue #2858: What official pulsar helm chart request 15GB memory for bookeeper,zookeeper and broker? URL: https://github.com/apache/pulsar/issues/2858#issuecomment-433687197 @sijie Another irrelevant question regarding pulsar: Is the connection to broker per clie

[GitHub] sijie commented on issue #2858: What official pulsar helm chart request 15GB memory for bookeeper, zookeeper and broker?

2018-10-28 Thread GitBox
sijie commented on issue #2858: What official pulsar helm chart request 15GB memory for bookeeper,zookeeper and broker? URL: https://github.com/apache/pulsar/issues/2858#issuecomment-433706772 @benjaminhuo Pulsar is using netty, so the connections will be managed per client.

[GitHub] benjaminhuo commented on issue #2858: What official pulsar helm chart request 15GB memory for bookeeper, zookeeper and broker?

2018-10-28 Thread GitBox
benjaminhuo commented on issue #2858: What official pulsar helm chart request 15GB memory for bookeeper,zookeeper and broker? URL: https://github.com/apache/pulsar/issues/2858#issuecomment-433706925 Great! Thanks very much. Ben

[GitHub] sijie commented on issue #2821: [Request] Pulsar client libraries Support Nodejs

2018-10-28 Thread GitBox
sijie commented on issue #2821: [Request] Pulsar client libraries Support Nodejs URL: https://github.com/apache/pulsar/issues/2821#issuecomment-433706910 This is same as #1070 This is an automated message from the Apache Git

[pulsar] branch master updated: Removed metrics from getstatus (#2866)

2018-10-28 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 4c44a8c Removed metrics from getstatus (#286

[GitHub] srkukarni closed pull request #2866: Removed metrics from getstatus

2018-10-28 Thread GitBox
srkukarni closed pull request #2866: Removed metrics from getstatus URL: https://github.com/apache/pulsar/pull/2866 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

[pulsar] branch master updated: Secretprovider interfaces and some default implementations (#2855)

2018-10-28 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 1aff3c4 Secretprovider interfaces and some d

[GitHub] srkukarni closed pull request #2855: Secretprovider interfaces and some default implementations

2018-10-28 Thread GitBox
srkukarni closed pull request #2855: Secretprovider interfaces and some default implementations URL: https://github.com/apache/pulsar/pull/2855 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 t

[GitHub] srkukarni opened a new pull request #2873: Remove metrics from getstatus in python functions

2018-10-28 Thread GitBox
srkukarni opened a new pull request #2873: Remove metrics from getstatus in python functions URL: https://github.com/apache/pulsar/pull/2873 ### Motivation GetStatus should not return metrics information. Java already removed this in #2866. This change removes this from python

[GitHub] Audace opened a new issue #2874: 'ConsumerConfiguration' object has no attribute 'read_compacted'

2018-10-28 Thread GitBox
Audace opened a new issue #2874: 'ConsumerConfiguration' object has no attribute 'read_compacted' URL: https://github.com/apache/pulsar/issues/2874 Expected behavior I want to experiment with Pulsar functions. I'm running Pulsar using Docker in standalone mode using the image t

[GitHub] srkukarni opened a new pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
srkukarni opened a new pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875 ### Motivation This pr defines the secrets interface for java and python functions and hooks their implementations. #

[GitHub] srkukarni commented on issue #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
srkukarni commented on issue #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#issuecomment-433725928 @cckellogg This is an automated message from the

[GitHub] Audace commented on issue #2874: 'ConsumerConfiguration' object has no attribute 'read_compacted'

2018-10-28 Thread GitBox
Audace commented on issue #2874: 'ConsumerConfiguration' object has no attribute 'read_compacted' URL: https://github.com/apache/pulsar/issues/2874#issuecomment-433726858 The correct approach here was using `create` to create the function in cluster mode. However, the way the `docker-compo

[GitHub] Audace closed issue #2874: 'ConsumerConfiguration' object has no attribute 'read_compacted'

2018-10-28 Thread GitBox
Audace closed issue #2874: 'ConsumerConfiguration' object has no attribute 'read_compacted' URL: https://github.com/apache/pulsar/issues/2874 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228759374 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/

[GitHub] cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228759162 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/

[GitHub] srkukarni commented on issue #2873: Remove metrics from getstatus in python functions

2018-10-28 Thread GitBox
srkukarni commented on issue #2873: Remove metrics from getstatus in python functions URL: https://github.com/apache/pulsar/pull/2873#issuecomment-433730938 rerun java8 tests This is an automated message from the Apache Git S

[GitHub] jerrypeng commented on issue #2873: Remove metrics from getstatus in python functions

2018-10-28 Thread GitBox
jerrypeng commented on issue #2873: Remove metrics from getstatus in python functions URL: https://github.com/apache/pulsar/pull/2873#issuecomment-433731592 Do you need to regenerate the python protobuf now that you have marked metrics deprecated? -

[GitHub] jerrypeng commented on issue #2868: Fixing function latency metric arithmetic error

2018-10-28 Thread GitBox
jerrypeng commented on issue #2868: Fixing function latency metric arithmetic error URL: https://github.com/apache/pulsar/pull/2868#issuecomment-433731668 rerun integration tests This is an automated message from the Apache G

[GitHub] srkukarni commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
srkukarni commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228761197 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/

[GitHub] srkukarni commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
srkukarni commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228761228 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/

[GitHub] cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228764406 ## File path: pulsar-functions/worker/src/test/java/org/apache/pulsar/functions/worker/Sc

[GitHub] cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228764450 ## File path: pulsar-functions/runtime/src/test/java/org/apache/pulsar/functions/runtime/

[GitHub] cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228764708 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/

[GitHub] cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
cckellogg commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228764450 ## File path: pulsar-functions/runtime/src/test/java/org/apache/pulsar/functions/runtime/

[GitHub] jerrypeng commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations

2018-10-28 Thread GitBox
jerrypeng commented on a change in pull request #2875: Hooked up secrets function api with secret function implementations URL: https://github.com/apache/pulsar/pull/2875#discussion_r228766165 ## File path: pulsar-functions/instance/src/main/python/python_instance_main.py #

[pulsar] branch master updated: Remove metrics from getstatus in python functions (#2873)

2018-10-28 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 6a51109 Remove metrics from getstatus in pyt

[GitHub] srkukarni closed pull request #2873: Remove metrics from getstatus in python functions

2018-10-28 Thread GitBox
srkukarni closed pull request #2873: Remove metrics from getstatus in python functions URL: https://github.com/apache/pulsar/pull/2873 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] jerrypeng commented on issue #2854: Fixed ZLib decompression in C++ client

2018-10-28 Thread GitBox
jerrypeng commented on issue #2854: Fixed ZLib decompression in C++ client URL: https://github.com/apache/pulsar/pull/2854#issuecomment-433752026 run java8 tests This is an automated message from the Apache Git Service

[GitHub] codelipenghui opened a new issue #2876: Limit the client reconnect behavior.

2018-10-28 Thread GitBox
codelipenghui opened a new issue #2876: Limit the client reconnect behavior. URL: https://github.com/apache/pulsar/issues/2876 ## Actual behavior I got some un-normal metrics in broker stats. https://user-images.githubusercontent.com/12592133/47623764-115d6480-db50-11e8-90fa-c7

[GitHub] srkukarni commented on issue #2873: Remove metrics from getstatus in python functions

2018-10-28 Thread GitBox
srkukarni commented on issue #2873: Remove metrics from getstatus in python functions URL: https://github.com/apache/pulsar/pull/2873#issuecomment-433758307 marking a field Deprecated doesn’t have any affect on the .pb files except for some warning declarations. Nothing to worry about ---

[GitHub] merlimat closed pull request #2854: Fixed ZLib decompression in C++ client

2018-10-28 Thread GitBox
merlimat closed pull request #2854: Fixed ZLib decompression in C++ client URL: https://github.com/apache/pulsar/pull/2854 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

[pulsar] branch master updated: Fixed ZLib decompression in C++ client (#2854)

2018-10-28 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new b990e1a Fixed ZLib decompression in C++ client

[GitHub] merlimat commented on issue #2868: Fixing function latency metric arithmetic error

2018-10-28 Thread GitBox
merlimat commented on issue #2868: Fixing function latency metric arithmetic error URL: https://github.com/apache/pulsar/pull/2868#issuecomment-433759092 run integration tests This is an automated message from the Apache Git

[GitHub] merlimat commented on issue #2852: Add connection timeout client configuration option

2018-10-28 Thread GitBox
merlimat commented on issue #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#issuecomment-433759711 run integration tests This is an automated message from the Apac

[GitHub] merlimat commented on issue #2826: Secrets Interface

2018-10-28 Thread GitBox
merlimat commented on issue #2826: Secrets Interface URL: https://github.com/apache/pulsar/pull/2826#issuecomment-433760609 @srkukarni This can be closed? This is an automated message from the Apache Git Service. To respond t

[GitHub] jiazhai commented on a change in pull request #2865: Refactored JCloud Tiered Storage

2018-10-28 Thread GitBox
jiazhai commented on a change in pull request #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#discussion_r228781944 ## File path: managed-ledger/src/test/java/org/apache/bookkeeper/client/PulsarMockReadHandle.java ## @@ -30,12 +30,1

[GitHub] jiazhai commented on a change in pull request #2865: Refactored JCloud Tiered Storage

2018-10-28 Thread GitBox
jiazhai commented on a change in pull request #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#discussion_r228781976 ## File path: tiered-storage/jcloud/pom.xml ## @@ -41,7 +41,7 @@ org.apache.pulsar jclouds-shaded

[GitHub] jiazhai commented on issue #2865: Refactored JCloud Tiered Storage

2018-10-28 Thread GitBox
jiazhai commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-433770742 rerun integration tests This is an automated message from the Apache Git Service. To res

[GitHub] jiazhai opened a new issue #2877: Add integration test for KafkaConnectSource

2018-10-28 Thread GitBox
jiazhai opened a new issue #2877: Add integration test for KafkaConnectSource URL: https://github.com/apache/pulsar/issues/2877 In PR #2791, @sijie suggested to add an integration test for debezium. KafkaConnectSource is a little not compatible with current source test framework. In c

[GitHub] jiazhai commented on issue #2865: Refactored JCloud Tiered Storage

2018-10-28 Thread GitBox
jiazhai commented on issue #2865: Refactored JCloud Tiered Storage URL: https://github.com/apache/pulsar/pull/2865#issuecomment-433779158 @david-streamlio Seems the integration tests also need taking care of. https://builds.apache.org/job/pulsar_precommit_integrationtests/4510/ -

[GitHub] jiazhai commented on issue #2791: Debezium: integrate KafkaConnectSource with MySqlConnectorTask

2018-10-28 Thread GitBox
jiazhai commented on issue #2791: Debezium: integrate KafkaConnectSource with MySqlConnectorTask URL: https://github.com/apache/pulsar/pull/2791#issuecomment-433786118 @sijie opened new issue for integration tests. This is an

[GitHub] benjaminhuo opened a new pull request #2878: Fix helm lint error for zookeeper-metadata.yaml

2018-10-28 Thread GitBox
benjaminhuo opened a new pull request #2878: Fix helm lint error for zookeeper-metadata.yaml URL: https://github.com/apache/pulsar/pull/2878 ### Motivation The following errors occurs when running : helm lint pulsar/ ==> Linting pulsar/ [INFO] Chart.yaml: icon is recomm

[GitHub] jerrypeng closed pull request #2868: Fixing function latency metric arithmetic error

2018-10-28 Thread GitBox
jerrypeng closed pull request #2868: Fixing function latency metric arithmetic error URL: https://github.com/apache/pulsar/pull/2868 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 fo

[pulsar] branch master updated: Fixing function latency metric arithmetic error (#2868)

2018-10-28 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 5fa83b8 Fixing function latency metric arith

[pulsar] branch master updated: Debezium: integrate KafkaConnectSource with MySqlConnectorTask (#2791)

2018-10-28 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new d13fe5b Debezium: integrate KafkaConnectSource w

[GitHub] sijie closed pull request #2791: Debezium: integrate KafkaConnectSource with MySqlConnectorTask

2018-10-28 Thread GitBox
sijie closed pull request #2791: Debezium: integrate KafkaConnectSource with MySqlConnectorTask URL: https://github.com/apache/pulsar/pull/2791 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 t