[GitHub] merlimat commented on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
merlimat commented on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434186341 run integration tests This is an automated message from the Apache Git Service. To resp

[GitHub] merlimat closed pull request #2886: Upgrade commons-collections for Security Vulnerabilities[CVE-2015-7501]

2018-10-29 Thread GitBox
merlimat closed pull request #2886: Upgrade commons-collections for Security Vulnerabilities[CVE-2015-7501] URL: https://github.com/apache/pulsar/pull/2886 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 prove

[pulsar] branch master updated: Upgrade commons-collections for Security Vulnerabilities[CVE-2015-7501] (#2886)

2018-10-29 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 c7e2197 Upgrade commons-collections for Securit

[GitHub] merlimat commented on issue #2886: Upgrade commons-collections for Security Vulnerabilities[CVE-2015-7501]

2018-10-29 Thread GitBox
merlimat commented on issue #2886: Upgrade commons-collections for Security Vulnerabilities[CVE-2015-7501] URL: https://github.com/apache/pulsar/pull/2886#issuecomment-434167952 run integration tests This is an automated mess

[GitHub] merlimat commented on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
merlimat commented on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434167870 run java8 tests run integration tests This is an automated message from the Apache G

[GitHub] nanguawu commented on issue #2879: Failed to write entry (11, 0): Bookie handle is not available

2018-10-29 Thread GitBox
nanguawu commented on issue #2879: Failed to write entry (11, 0): Bookie handle is not available URL: https://github.com/apache/pulsar/issues/2879#issuecomment-434161385 The zookeeper that comes with pulsar is normal, always suggesting that bookie does not exist. cluster node: server057

[GitHub] jiazhai edited a comment on issue #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
jiazhai edited a comment on issue #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885#issuecomment-434147960 rerun java8 tests for org.apache.pulsar.functions.worker.PulsarWorkerAssignmentTest.testFunct

[GitHub] jiazhai commented on issue #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
jiazhai commented on issue #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885#issuecomment-434147960 rerun java8 tests This is an automated me

[GitHub] rdhabalia removed a comment on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia removed a comment on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434133447 rerun tests please This is an automated message from the Apache Git Service. T

[GitHub] rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434133594 test this please This is an automated message from the Apache Git Service. To respond

[GitHub] rdhabalia removed a comment on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia removed a comment on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434133594 test this please This is an automated message from the Apache Git Service. To

[GitHub] rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434133447 rerun tests please This is an automated message from the Apache Git Service. To respon

[GitHub] rdhabalia removed a comment on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia removed a comment on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434133380 rerun tests This is an automated message from the Apache Git Service. To respo

[GitHub] rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-434133380 rerun tests This is an automated message from the Apache Git Service. To respond to th

[GitHub] rdhabalia opened a new pull request #2886: Upgrade commons-collections for Security Vulnerabilities[CVE-2015-7501]

2018-10-29 Thread GitBox
rdhabalia opened a new pull request #2886: Upgrade commons-collections for Security Vulnerabilities[CVE-2015-7501] URL: https://github.com/apache/pulsar/pull/2886 ### Motivation Apache Commons Collections Security Vulnerabilities is fixed with commons-collection::3.2.2 https

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229085858 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229083715 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229078299 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229077361 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229077361 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] sijie commented on issue #2878: Fix helm lint error for zookeeper-metadata.yaml

2018-10-29 Thread GitBox
sijie commented on issue #2878: Fix helm lint error for zookeeper-metadata.yaml URL: https://github.com/apache/pulsar/pull/2878#issuecomment-434057291 @benjaminhuo thank you for your contribution. This is an automated message

[pulsar] branch master updated (f7a91a1 -> aad7300)

2018-10-29 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from f7a91a1 Fixed python websocket client sample code (#2881) add aad7300 Fix helm lint error for zookeeper-metadat

[GitHub] sijie closed pull request #2878: Fix helm lint error for zookeeper-metadata.yaml

2018-10-29 Thread GitBox
sijie closed pull request #2878: Fix helm lint error for zookeeper-metadata.yaml URL: https://github.com/apache/pulsar/pull/2878 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

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229075915 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] sijie commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229075685 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/ConnectionTimeoutTest.java ##

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229075098 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229074238 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229073688 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229072973 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/ConnectionTimeoutTest.java ##

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229072513 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] sijie commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229071864 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/ConnectionTimeoutTest.java ##

[GitHub] sijie commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229071695 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java ###

[GitHub] ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229066110 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java

[GitHub] ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229064736 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/ConnectionTimeoutTest.java ##

[GitHub] ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229064177 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/api/ClientBuilder.java ## @@

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229063131 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229062201 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229061948 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229061810 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] sijie commented on issue #2879: Failed to write entry (11, 0): Bookie handle is not available

2018-10-29 Thread GitBox
sijie commented on issue #2879: Failed to write entry (11, 0): Bookie handle is not available URL: https://github.com/apache/pulsar/issues/2879#issuecomment-434031630 @nanguawu : where did you run `bookkeeper sanity`? was it in the bookie machine? can you run `bookkeeper she

[GitHub] sijie commented on issue #2733: Extra logging for offloaders

2018-10-29 Thread GitBox
sijie commented on issue #2733: Extra logging for offloaders URL: https://github.com/apache/pulsar/pull/2733#issuecomment-434029265 run integration tests This is an automated message from the Apache Git Service. To respond to

[GitHub] sijie commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229048795 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java ###

[GitHub] sijie commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229047887 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/api/ClientBuilder.java ## @@ -309

[GitHub] sijie commented on a change in pull request #2852: Add connection timeout client configuration option

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#discussion_r229048269 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/ConnectionTimeoutTest.java ##

[GitHub] sijie commented on issue #2867: Use unique test names in PartitionedProducerConsumerTest to avoid individual tests inter-dependencies

2018-10-29 Thread GitBox
sijie commented on issue #2867: Use unique test names in PartitionedProducerConsumerTest to avoid individual tests inter-dependencies URL: https://github.com/apache/pulsar/pull/2867#issuecomment-434026343 run java8 tests run integration tests

[GitHub] sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files URL: https://github.com/apache/pulsar/pull/2869#discussion_r229043478 ## File path: pulsar-io/file/src/main/java/org/apache/pulsar/io/file/FileConsumerThread.java ## @@ -0,0 +1,11

[GitHub] sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files URL: https://github.com/apache/pulsar/pull/2869#discussion_r229044481 ## File path: pulsar-io/file/src/main/java/org/apache/pulsar/io/file/FileSource.java ## @@ -0,0 +1,70 @@ +/**

[GitHub] sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files URL: https://github.com/apache/pulsar/pull/2869#discussion_r229045789 ## File path: pulsar-io/file/src/test/java/org/apache/pulsar/io/file/AbstractFileTests.java ## @@ -0,0 +1,127

[GitHub] sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files URL: https://github.com/apache/pulsar/pull/2869#discussion_r229041427 ## File path: pulsar-io/file/src/test/resources/sinkConfig.yaml ## @@ -0,0 +1,33 @@ +# +# Licensed to the Apac

[GitHub] sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2869: Added Pulsar IO connector for local files URL: https://github.com/apache/pulsar/pull/2869#discussion_r229040944 ## File path: pulsar-io/file/pom.xml ## @@ -0,0 +1,71 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="ht

[GitHub] sijie commented on issue #2878: Fix helm lint error for zookeeper-metadata.yaml

2018-10-29 Thread GitBox
sijie commented on issue #2878: Fix helm lint error for zookeeper-metadata.yaml URL: https://github.com/apache/pulsar/pull/2878#issuecomment-434019287 run integration tests This is an automated message from the Apache Git Serv

[GitHub] sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885#discussion_r229038224 ## File path: pulsar-client-schema/src/main/java/org/apache/pulsar/client/api/Sche

[GitHub] sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885#discussion_r229040413 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/schema/KeyValue

[GitHub] sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885#discussion_r229038641 ## File path: pulsar-client-schema/src/main/java/org/apache/pulsar/client/api/Sche

[GitHub] sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885#discussion_r22903 ## File path: pulsar-client-schema/src/main/java/org/apache/pulsar/client/impl/sch

[GitHub] sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885#discussion_r229038838 ## File path: pulsar-client-schema/src/main/java/org/apache/pulsar/client/impl/sch

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

2018-10-29 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-434014699 rerun integration tests This is an automated messa

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229034923 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229033708 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229034275 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] sijie commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
sijie commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229035729 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed to t

[GitHub] rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia commented on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-433997557 sure.. will do..it also seems I need to update `LICENSE/NOTICE`, will fix it as well.

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

2018-10-29 Thread GitBox
srkukarni commented on issue #2826: Secrets Interface URL: https://github.com/apache/pulsar/pull/2826#issuecomment-433997503 @merlimat will close it once #2875 and a follow up example/doc change goes in This is an automated me

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

2018-10-29 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_r229018053 ## File path: pulsar-functions/instance/src/main/python/python_instance_main.py #

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

2018-10-29 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_r229017952 ## File path: pulsar-functions/worker/src/test/java/org/apache/pulsar/functions/worker/Sc

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

2018-10-29 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_r229017854 ## 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-29 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_r229017995 ## File path: pulsar-functions/runtime/src/test/java/org/apache/pulsar/functions/runtime/

[GitHub] merlimat commented on issue #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
merlimat commented on issue #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#issuecomment-433981664 There's a compilation error: ``` 2018-10-29\T\15:20:27.181 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.7.0:test

[GitHub] merlimat commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
merlimat commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r229003487 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed t

[GitHub] merlimat commented on issue #2878: Fix helm lint error for zookeeper-metadata.yaml

2018-10-29 Thread GitBox
merlimat commented on issue #2878: Fix helm lint error for zookeeper-metadata.yaml URL: https://github.com/apache/pulsar/pull/2878#issuecomment-433979197 run java8 tests run integration tests This is an automated message f

[GitHub] ivankelly commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r22824 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed

[GitHub] merlimat closed pull request #2881: Fixed python websocket client sample code

2018-10-29 Thread GitBox
merlimat closed pull request #2881: Fixed python websocket client sample code URL: https://github.com/apache/pulsar/pull/2881 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 pu

[pulsar] branch master updated: Fixed python websocket client sample code (#2881)

2018-10-29 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 f7a91a1 Fixed python websocket client sample co

[GitHub] merlimat commented on issue #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
merlimat commented on issue #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884#issuecomment-433974088 @rdhabalia Can you also take a look at upgrading the other libraries reported in #2882 ---

[GitHub] merlimat commented on issue #2882: security vulnerabilties in 3rd party components

2018-10-29 Thread GitBox
merlimat commented on issue #2882: security vulnerabilties in 3rd party components URL: https://github.com/apache/pulsar/issues/2882#issuecomment-433973402 Thanks @javabeanz for reporting. This is being addressed in #2884 Thi

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

2018-10-29 Thread GitBox
ivankelly commented on issue #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#issuecomment-433973168 rerun integration tests This is an automated message from the Apach

[GitHub] merlimat commented on a change in pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
merlimat commented on a change in pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#discussion_r228994385 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarHealthcheck.java ## @@ -0,0 +1,204 @@ +/** + * Licensed t

[GitHub] jiazhai opened a new pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord

2018-10-29 Thread GitBox
jiazhai opened a new pull request #2885: Add KeyValueSchema to satisfy org.apache.kafka.connect.connector.ConnectRecord URL: https://github.com/apache/pulsar/pull/2885 Kafka org.apache.kafka.connect.connector.ConnectRecord has both key and value schema. Currently we need a KeyValueSchem

[GitHub] rdhabalia opened a new pull request #2884: Upgrade jackson version to 2.9.7

2018-10-29 Thread GitBox
rdhabalia opened a new pull request #2884: Upgrade jackson version to 2.9.7 URL: https://github.com/apache/pulsar/pull/2884 ### Motivation [CVE-2018-7489](https://www.cvedetails.com/cve/CVE-2018-7489/) FasterXML jackson-databind before 2.8.11.1 and 2.9.x before 2.9.5 allows unauthent

[GitHub] ivankelly commented on issue #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly commented on issue #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883#issuecomment-433950269 @cckellogg This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] ivankelly opened a new pull request #2883: Add healthcheck for broker

2018-10-29 Thread GitBox
ivankelly opened a new pull request #2883: Add healthcheck for broker URL: https://github.com/apache/pulsar/pull/2883 This patch adds a healthcheck command that checks the health of the local broker. It subscribes to a topic in the heartbeat namespace for the broker, publishes to that

[GitHub] javabeanz opened a new issue #2882: security vulnerabilties in 3rd party components

2018-10-29 Thread GitBox
javabeanz opened a new issue #2882: security vulnerabilties in 3rd party components URL: https://github.com/apache/pulsar/issues/2882 mvn com.redhat.victims.maven:security-versions:check reveals many vulns, especially for jakscon and commons libs : [ERROR] jline:jline is vulnerable to C

[GitHub] quiye opened a new pull request #2881: Fixed python websocket client sample code

2018-10-29 Thread GitBox
quiye opened a new pull request #2881: Fixed python websocket client sample code URL: https://github.com/apache/pulsar/pull/2881 ### Motivation Python websocket client sample code is invalid for execute producer. http://pulsar.apache.org/docs/en/client-libraries-websocket/#python-p

[GitHub] HaraldGustafsson opened a new issue #2880: Unauthorised when deleting namespace with superuser role

2018-10-29 Thread GitBox
HaraldGustafsson opened a new issue #2880: Unauthorised when deleting namespace with superuser role URL: https://github.com/apache/pulsar/issues/2880 Expected behavior All the following admin is done using a superuser role towards a pulsar broker over the REST API. * cre

[GitHub] nanguawu opened a new issue #2879: Failed to write entry (11, 0): Bookie handle is not available

2018-10-29 Thread GitBox
nanguawu opened a new issue #2879: Failed to write entry (11, 0): Bookie handle is not available URL: https://github.com/apache/pulsar/issues/2879 Hi: my pulsar version: 2.2.0 I am currently deploying a multi-cluster on bare metal, Running the bookiesanity command for:

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

2018-10-29 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 5a72a65 Updated site at revision d13fe5b 5a