[GitHub] rdhabalia commented on issue #2916: Fix zk latency metrics collector

2018-11-02 Thread GitBox
rdhabalia commented on issue #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916#issuecomment-435547348 rerun test please This is an automated message from the Apache Git Service. To

[GitHub] rdhabalia commented on issue #2916: Fix zk latency metrics collector

2018-11-02 Thread GitBox
rdhabalia commented on issue #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916#issuecomment-435547365 rerun java8 tests This is an automated message from the Apache Git Service. To

[GitHub] rdhabalia removed a comment on issue #2916: Fix zk latency metrics collector

2018-11-02 Thread GitBox
rdhabalia removed a comment on issue #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916#issuecomment-435547365 rerun java8 tests This is an automated message from the Apache Git Service.

[GitHub] rdhabalia removed a comment on issue #2916: Fix zk latency metrics collector

2018-11-02 Thread GitBox
rdhabalia removed a comment on issue #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916#issuecomment-435547348 rerun test please This is an automated message from the Apache Git Service.

[GitHub] rdhabalia commented on issue #2916: Fix zk latency metrics collector

2018-11-02 Thread GitBox
rdhabalia commented on issue #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916#issuecomment-435547377 rerun integration tests This is an automated message from the Apache Git Service. To

[GitHub] rdhabalia removed a comment on issue #2916: Fix zk latency metrics collector

2018-11-02 Thread GitBox
rdhabalia removed a comment on issue #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916#issuecomment-435547377 rerun integration tests This is an automated message from the Apache Git

[GitHub] srkukarni commented on issue #2918: Prefer rest path over Function/Source/Sink Config values

2018-11-02 Thread GitBox
srkukarni commented on issue #2918: Prefer rest path over Function/Source/Sink Config values URL: https://github.com/apache/pulsar/pull/2918#issuecomment-435559342 rerun cpp tests This is an automated message from the Apache

[GitHub] srkukarni opened a new pull request #2917: Remove functions-util dependency from pulsar-client-admin

2018-11-02 Thread GitBox
srkukarni opened a new pull request #2917: Remove functions-util dependency from pulsar-client-admin URL: https://github.com/apache/pulsar/pull/2917 ### Motivation This pr removes functions-util dependency from pulsar-client-admin. With this change, the admin is only dependent on

[GitHub] sijie closed pull request #2913: Removed checked in examples jar and instead added mvn target to copy

2018-11-02 Thread GitBox
sijie closed pull request #2913: Removed checked in examples jar and instead added mvn target to copy URL: https://github.com/apache/pulsar/pull/2913 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

[pulsar] branch master updated: Removed checked in examples jar and instead added mvn target to copy (#2913)

2018-11-02 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 4686724 Removed checked in examples jar and

[pulsar] branch master updated: [state] Initialize bookkeeper table service metadata on initializing pulsar cluster metadata (#2706)

2018-11-02 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 4e9971d [state] Initialize bookkeeper table

[pulsar] branch master updated: Added ability to take python zip files that contain all their dependencies (#2915)

2018-11-02 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 35cd182 Added ability to take python zip

[GitHub] srkukarni commented on issue #2917: Remove functions-util dependency from pulsar-client-admin

2018-11-02 Thread GitBox
srkukarni commented on issue #2917: Remove functions-util dependency from pulsar-client-admin URL: https://github.com/apache/pulsar/pull/2917#issuecomment-435546210 rerun cpp tests This is an automated message from the

[GitHub] srkukarni opened a new pull request #2918: Prefer rest path over Function/Source/Sink Config values

2018-11-02 Thread GitBox
srkukarni opened a new pull request #2918: Prefer rest path over Function/Source/Sink Config values URL: https://github.com/apache/pulsar/pull/2918 ### Motivation When users create/update sink/source/function, they call the rest endpoint that has a tenant/namespace/name components

[GitHub] srkukarni commented on issue #2917: Remove functions-util dependency from pulsar-client-admin

2018-11-02 Thread GitBox
srkukarni commented on issue #2917: Remove functions-util dependency from pulsar-client-admin URL: https://github.com/apache/pulsar/pull/2917#issuecomment-435559620 rerun integration tests This is an automated message from

[GitHub] sijie closed pull request #2706: [state] Initialize bookkeeper table service metadata on initializing pulsar cluster metadata

2018-11-02 Thread GitBox
sijie closed pull request #2706: [state] Initialize bookkeeper table service metadata on initializing pulsar cluster metadata URL: https://github.com/apache/pulsar/pull/2706 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] srkukarni commented on issue #2917: Remove functions-util dependency from pulsar-client-admin

2018-11-02 Thread GitBox
srkukarni commented on issue #2917: Remove functions-util dependency from pulsar-client-admin URL: https://github.com/apache/pulsar/pull/2917#issuecomment-435550141 rerun java8 tests This is an automated message from the

[pulsar] branch master updated: Add state storage service url as part of internal configuration data (#2908)

2018-11-02 Thread zhaijia
This is an automated email from the ASF dual-hosted git repository. zhaijia 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 355ebc5 Add state storage service url as part

[GitHub] jiazhai closed pull request #2908: [config] Add state storage service url as part of internal configuration data

2018-11-02 Thread GitBox
jiazhai closed pull request #2908: [config] Add state storage service url as part of internal configuration data URL: https://github.com/apache/pulsar/pull/2908 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

[GitHub] codelipenghui commented on issue #2876: Limit the client reconnect behavior.

2018-11-02 Thread GitBox
codelipenghui commented on issue #2876: Limit the client reconnect behavior. URL: https://github.com/apache/pulsar/issues/2876#issuecomment-435292468 I think more secure and unified on broker side, broker already limit the maxConsumersPerTopic and maxConsumersPerSubscription, broker limit

[GitHub] codelipenghui edited a comment on issue #2876: Limit the client reconnect behavior.

2018-11-02 Thread GitBox
codelipenghui edited a comment on issue #2876: Limit the client reconnect behavior. URL: https://github.com/apache/pulsar/issues/2876#issuecomment-435292468 @sijie I think more secure and unified on broker side, broker already limit the maxConsumersPerTopic and

[pulsar] branch asf-site updated: Updated site at revision 08d8e25

2018-11-02 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/pulsar.git The following commit(s) were added to refs/heads/asf-site by this push: new 1ff54d6 Updated site at revision 08d8e25

[GitHub] srkukarni opened a new pull request #2915: Added ability to take python zip files that contain all their dependencies

2018-11-02 Thread GitBox
srkukarni opened a new pull request #2915: Added ability to take python zip files that contain all their dependencies URL: https://github.com/apache/pulsar/pull/2915 ### Motivation Explain here the context, and why you're making that change. What is the problem you're trying

[GitHub] srkukarni closed pull request #2897: [WIP] Add archive support to python instance

2018-11-02 Thread GitBox
srkukarni closed pull request #2897: [WIP] Add archive support to python instance URL: https://github.com/apache/pulsar/pull/2897 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] srkukarni commented on issue #2897: [WIP] Add archive support to python instance

2018-11-02 Thread GitBox
srkukarni commented on issue #2897: [WIP] Add archive support to python instance URL: https://github.com/apache/pulsar/pull/2897#issuecomment-435501289 Closing this in favor of #2915 This is an automated message from the

[GitHub] srkukarni opened a new pull request #2913: Removed checked in examples jar and instead added mvn target to copy

2018-11-02 Thread GitBox
srkukarni opened a new pull request #2913: Removed checked in examples jar and instead added mvn target to copy URL: https://github.com/apache/pulsar/pull/2913 ### Motivation Copy the required pulsar-examples jar by adding a maven rule rather than physically checking in. ###

[GitHub] oliverisaac commented on issue #2795: namespace grant-permission Returns 500 in 2.1.1-incubating

2018-11-02 Thread GitBox
oliverisaac commented on issue #2795: namespace grant-permission Returns 500 in 2.1.1-incubating URL: https://github.com/apache/pulsar/issues/2795#issuecomment-435439429 Yup, good to go! This is an automated message from the

[GitHub] oliverisaac closed issue #2795: namespace grant-permission Returns 500 in 2.1.1-incubating

2018-11-02 Thread GitBox
oliverisaac closed issue #2795: namespace grant-permission Returns 500 in 2.1.1-incubating URL: https://github.com/apache/pulsar/issues/2795 This is an automated message from the Apache Git Service. To respond to the

[GitHub] jerrypeng opened a new pull request #2914: Beginging to refactor function metrics to use prometheus

2018-11-02 Thread GitBox
jerrypeng opened a new pull request #2914: Beginging to refactor function metrics to use prometheus URL: https://github.com/apache/pulsar/pull/2914 ### Motivation Current function metrics system is convoluted, incorrect, confusing. Refactor metrics to use prometheus. I only

[GitHub] rdhabalia commented on issue #2911: Fix pulsar function-metrics artifacts deploy

2018-11-02 Thread GitBox
rdhabalia commented on issue #2911: Fix pulsar function-metrics artifacts deploy URL: https://github.com/apache/pulsar/pull/2911#issuecomment-435485509 rerun java8 tests This is an automated message from the Apache Git

[GitHub] merlimat commented on a change in pull request #2914: Beginging to refactor function metrics to use prometheus

2018-11-02 Thread GitBox
merlimat commented on a change in pull request #2914: Beginging to refactor function metrics to use prometheus URL: https://github.com/apache/pulsar/pull/2914#discussion_r230511407 ## File path: pulsar-functions/instance/pom.xml ## @@ -107,6 +107,25 @@ typetools

[GitHub] srkukarni commented on issue #2913: Removed checked in examples jar and instead added mvn target to copy

2018-11-02 Thread GitBox
srkukarni commented on issue #2913: Removed checked in examples jar and instead added mvn target to copy URL: https://github.com/apache/pulsar/pull/2913#issuecomment-435514562 rerun tests This is an automated message from

[GitHub] merlimat commented on a change in pull request #2915: Added ability to take python zip files that contain all their dependencies

2018-11-02 Thread GitBox
merlimat commented on a change in pull request #2915: Added ability to take python zip files that contain all their dependencies URL: https://github.com/apache/pulsar/pull/2915#discussion_r230514041 ## File path: pulsar-functions/instance/src/main/python/python_instance_main.py

[GitHub] srkukarni commented on a change in pull request #2915: Added ability to take python zip files that contain all their dependencies

2018-11-02 Thread GitBox
srkukarni commented on a change in pull request #2915: Added ability to take python zip files that contain all their dependencies URL: https://github.com/apache/pulsar/pull/2915#discussion_r230515294 ## File path: pulsar-functions/instance/src/main/python/python_instance_main.py

[GitHub] srkukarni closed pull request #2907: Move all validation/inferring missing args to serverside

2018-11-02 Thread GitBox
srkukarni closed pull request #2907: Move all validation/inferring missing args to serverside URL: https://github.com/apache/pulsar/pull/2907 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

[GitHub] rdhabalia opened a new pull request #2916: Fix zk latency metrics collector

2018-11-02 Thread GitBox
rdhabalia opened a new pull request #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916 ### Motivation After zk upgrade, broker-zk client latency metrics is broken. ``` 22:59:30.824 [pulsar-1-2] DEBUG

[GitHub] srkukarni commented on issue #2911: Fix pulsar function-metrics artifacts deploy

2018-11-02 Thread GitBox
srkukarni commented on issue #2911: Fix pulsar function-metrics artifacts deploy URL: https://github.com/apache/pulsar/pull/2911#issuecomment-435464797 We plan to deprecate PrometheusMetricsServer in favor of having a embedded server within a function instance itself. Thus all of this code