[GitHub] [pulsar] rdhabalia commented on issue #5314: [pulsar-broker] fix ns-isolation api to fetch policy for specific broker

2019-10-07 Thread GitBox
rdhabalia commented on issue #5314: [pulsar-broker] fix ns-isolation api to fetch policy for specific broker URL: https://github.com/apache/pulsar/pull/5314#issuecomment-538872399 retest this please This is an automated

[GitHub] [pulsar] rdhabalia commented on issue #5313: [pulsar-cli] fix ns-isolation-policy api output

2019-10-07 Thread GitBox
rdhabalia commented on issue #5313: [pulsar-cli] fix ns-isolation-policy api output URL: https://github.com/apache/pulsar/pull/5313#issuecomment-538872218 rerun integration tests This is an automated message from the Apache

[GitHub] [pulsar] jiazhai commented on issue #5165: [PIP-43] Support producer to send msg with different schema

2019-10-07 Thread GitBox
jiazhai commented on issue #5165: [PIP-43] Support producer to send msg with different schema URL: https://github.com/apache/pulsar/pull/5165#issuecomment-539001984 run Integration Tests This is an automated message

[GitHub] [pulsar] Guchman commented on issue #5312: Function REST API documentation is insufficient for Create POST action

2019-10-07 Thread GitBox
Guchman commented on issue #5312: Function REST API documentation is insufficient for Create POST action URL: https://github.com/apache/pulsar/issues/5312#issuecomment-538965217 I think expected behaviour is to support application/json:)

[GitHub] [pulsar] jiazhai commented on issue #5276: Fixed race condition while triggering message redelivery after an ack-timeout event

2019-10-07 Thread GitBox
jiazhai commented on issue #5276: Fixed race condition while triggering message redelivery after an ack-timeout event URL: https://github.com/apache/pulsar/pull/5276#issuecomment-539006838 rerun java8 tests This is an

[GitHub] [pulsar] merlimat commented on issue #5279: Support for python native logging from python wrapper

2019-10-07 Thread GitBox
merlimat commented on issue #5279: Support for python native logging from python wrapper URL: https://github.com/apache/pulsar/pull/5279#issuecomment-539085590 run integration tests This is an automated message from

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-538224332 run java8 tests This is an

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-538710782 run Integration Tests This is an

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-538710748 run java8 tests This is an

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-537047021 run java8 tests This is an

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-538613945 run java8 tests This is an

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-538710795 run cpp tests This is an automated

[GitHub] [pulsar] merlimat merged pull request #5314: [pulsar-broker] fix ns-isolation api to fetch policy for specific broker

2019-10-07 Thread GitBox
merlimat merged pull request #5314: [pulsar-broker] fix ns-isolation api to fetch policy for specific broker URL: https://github.com/apache/pulsar/pull/5314 This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] merlimat merged pull request #5205: [doc] Fix broken links to the info about schemas and Schema Registry

2019-10-07 Thread GitBox
merlimat merged pull request #5205: [doc] Fix broken links to the info about schemas and Schema Registry URL: https://github.com/apache/pulsar/pull/5205 This is an automated message from the Apache Git Service. To respond

[pulsar] branch master updated: [doc] Fix broken links to the info about schemas and Schema Registry (#5205)

2019-10-07 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 7b06d3a [doc] Fix broken links to the info

[GitHub] [pulsar] merlimat merged pull request #5276: Fixed race condition while triggering message redelivery after an ack-timeout event

2019-10-07 Thread GitBox
merlimat merged pull request #5276: Fixed race condition while triggering message redelivery after an ack-timeout event URL: https://github.com/apache/pulsar/pull/5276 This is an automated message from the Apache Git

[pulsar] branch master updated (e5ded1b -> e840375)

2019-10-07 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from e5ded1b Add a source connector for MongoDB (#5316) add e840375 Fixed race condition while triggering message

[GitHub] [pulsar] merlimat merged pull request #5313: [pulsar-cli] fix ns-isolation-policy api output

2019-10-07 Thread GitBox
merlimat merged pull request #5313: [pulsar-cli] fix ns-isolation-policy api output URL: https://github.com/apache/pulsar/pull/5313 This is an automated message from the Apache Git Service. To respond to the message, please

[pulsar] branch master updated (e840375 -> de233d4)

2019-10-07 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from e840375 Fixed race condition while triggering message redelivery after an ack-timeout event (#5276) add

[pulsar] branch master updated (7b06d3a -> 21e2b52)

2019-10-07 Thread aahmed
This is an automated email from the ASF dual-hosted git repository. aahmed pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 7b06d3a [doc] Fix broken links to the info about schemas and Schema Registry (#5205) add 21e2b52 Fix Cmake

[GitHub] [pulsar] aahmed-se merged pull request #5263: Fix Cmake to build _pulsar for osx

2019-10-07 Thread GitBox
aahmed-se merged pull request #5263: Fix Cmake to build _pulsar for osx URL: https://github.com/apache/pulsar/pull/5263 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] rdhabalia commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
rdhabalia commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r332227446 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java ## @@

[GitHub] [pulsar] merlimat opened a new pull request #5331: Avoid rethrowing exception during test teardown

2019-10-07 Thread GitBox
merlimat opened a new pull request #5331: Avoid rethrowing exception during test teardown URL: https://github.com/apache/pulsar/pull/5331 ### Motivation In several cases, the test get stuck on the broker teardown. Do not re-throw the exception here, since the test logic itself was

[GitHub] [pulsar] TRManderson opened a new pull request #5332: [docs] Fix broken link to k8s YAML definitions

2019-10-07 Thread GitBox
TRManderson opened a new pull request #5332: [docs] Fix broken link to k8s YAML definitions URL: https://github.com/apache/pulsar/pull/5332 The file path to the relevant directory was moved under the deployment subdirectory, this fixes that.

[GitHub] [pulsar] jerrypeng commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
jerrypeng commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r332250591 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java ## @@

[GitHub] [pulsar] merlimat opened a new pull request #5328: Increase the retries count for PulsarFunctionStateTest

2019-10-07 Thread GitBox
merlimat opened a new pull request #5328: Increase the retries count for PulsarFunctionStateTest URL: https://github.com/apache/pulsar/pull/5328 ### Motivation The test has been intermittently failing: ``` 2019-10-07\T\20:09:37.235 [ERROR]

[GitHub] [pulsar] merlimat opened a new pull request #5329: Fix ConsumerRedeliveryTest by ensure clean state on each run

2019-10-07 Thread GitBox
merlimat opened a new pull request #5329: Fix ConsumerRedeliveryTest by ensure clean state on each run URL: https://github.com/apache/pulsar/pull/5329 ### Motivation The test retries are failing because the state for the prev execution lingers on. ```

[GitHub] [pulsar] jiazhai commented on issue #5045: [client] Introduce `batchingMaxBytes` setting in pulsar producer

2019-10-07 Thread GitBox
jiazhai commented on issue #5045: [client] Introduce `batchingMaxBytes` setting in pulsar producer URL: https://github.com/apache/pulsar/pull/5045#issuecomment-539170200 run java8 tests This is an automated message from the

[GitHub] [pulsar] jerrypeng commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
jerrypeng commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r332208797 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ## @@ -624,6

[GitHub] [pulsar] merlimat opened a new pull request #5330: Added basic metadata store API and initial ZK implementation

2019-10-07 Thread GitBox
merlimat opened a new pull request #5330: Added basic metadata store API and initial ZK implementation URL: https://github.com/apache/pulsar/pull/5330 ### Motivation This PR is the first in a series of changes. The ultimate goal is to have all the metadata accesses to ZooKeeper to

[GitHub] [pulsar] jiazhai commented on issue #5321: [pulsar-broker] Disable stickyRead by default

2019-10-07 Thread GitBox
jiazhai commented on issue #5321: [pulsar-broker] Disable stickyRead by default URL: https://github.com/apache/pulsar/pull/5321#issuecomment-539168802 run integration tests run java8 tests This is an automated message from

[GitHub] [pulsar] jiazhai commented on issue #5075: [pulsar-client] support set read-position based on timestamp

2019-10-07 Thread GitBox
jiazhai commented on issue #5075: [pulsar-client] support set read-position based on timestamp URL: https://github.com/apache/pulsar/pull/5075#issuecomment-539170020 @rdhabalia Would you please help rebase this PR? This is

[GitHub] [pulsar] chairmank commented on issue #5318: Pulsar 2.4.1 - Broker stops server Admin API at random

2019-10-07 Thread GitBox
chairmank commented on issue #5318: Pulsar 2.4.1 - Broker stops server Admin API at random URL: https://github.com/apache/pulsar/issues/5318#issuecomment-539181519 related to https://github.com/apache/pulsar/pull/5079 This

[GitHub] [pulsar] chairmank edited a comment on issue #5318: Pulsar 2.4.1 - Broker stops server Admin API at random

2019-10-07 Thread GitBox
chairmank edited a comment on issue #5318: Pulsar 2.4.1 - Broker stops server Admin API at random URL: https://github.com/apache/pulsar/issues/5318#issuecomment-539181519 related to https://github.com/apache/pulsar/pull/5079#issuecomment-531491624

[GitHub] [pulsar] rocketraman commented on issue #2328: Pulsar broker start failure when set functionsWorkerEnabled=true

2019-10-07 Thread GitBox
rocketraman commented on issue #2328: Pulsar broker start failure when set functionsWorkerEnabled=true URL: https://github.com/apache/pulsar/issues/2328#issuecomment-539105749 I have this same issue with running Pulsar on Kubernetes. I see that the script `gen-yml-from-env.py` is supposed

[GitHub] [pulsar] rocketraman edited a comment on issue #2328: Pulsar broker start failure when set functionsWorkerEnabled=true

2019-10-07 Thread GitBox
rocketraman edited a comment on issue #2328: Pulsar broker start failure when set functionsWorkerEnabled=true URL: https://github.com/apache/pulsar/issues/2328#issuecomment-539105749 I have this same issue with running Pulsar on Kubernetes. I see that the script `gen-yml-from-env.py` is

[GitHub] [pulsar] cdbartholomew commented on issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true

2019-10-07 Thread GitBox
cdbartholomew commented on issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true URL: https://github.com/apache/pulsar/issues/5325#issuecomment-539168930 @rocketraman expanding on @sijie's comment, I can confirm that if you set it as an environment

[GitHub] [pulsar] hanbo1990 opened a new issue #5326: pulsar released package missing configuraion for ranges from bookkie

2019-10-07 Thread GitBox
hanbo1990 opened a new issue #5326: pulsar released package missing configuraion for ranges from bookkie URL: https://github.com/apache/pulsar/issues/5326 **Describe the bug** Standalone pulsar's bookie creates data/ranges folder even bookkeeper's data path is changed to another

[GitHub] [pulsar] miton18 opened a new pull request #5327: typo: traffic

2019-10-07 Thread GitBox
miton18 opened a new pull request #5327: typo: traffic URL: https://github.com/apache/pulsar/pull/5327 typo This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [pulsar] merlimat commented on issue #5319: Collect broker publish latency in prometheus

2019-10-07 Thread GitBox
merlimat commented on issue #5319: Collect broker publish latency in prometheus URL: https://github.com/apache/pulsar/pull/5319#issuecomment-539158166 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] sijie commented on issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true

2019-10-07 Thread GitBox
sijie commented on issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true URL: https://github.com/apache/pulsar/issues/5325#issuecomment-539167556 @rocketraman INT_KEYS are used for the settings whose value is Integer. So I don't think pulsarFunctionCluster

[GitHub] [pulsar] rocketraman opened a new issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true

2019-10-07 Thread GitBox
rocketraman opened a new issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true URL: https://github.com/apache/pulsar/issues/5325 **Describe the bug** The Kubernetes deployment sets `PF_pulsarFunctionsCluster` but this doesn't appear to get applied to

[GitHub] [pulsar] merlimat commented on issue #5317: Allow to specify delivery delay in C++ client

2019-10-07 Thread GitBox
merlimat commented on issue #5317: Allow to specify delivery delay in C++ client URL: https://github.com/apache/pulsar/pull/5317#issuecomment-539127526 run java8 tests run integration tests This is an automated message

[GitHub] [pulsar] merlimat commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
merlimat commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r332197883 ## File path: managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerErrorsTest.java ## @@

[GitHub] [pulsar] merlimat commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
merlimat commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r332197410 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ## @@ -624,6

[GitHub] [pulsar] merlimat commented on issue #5317: Allow to specify delivery delay in C++ client

2019-10-07 Thread GitBox
merlimat commented on issue #5317: Allow to specify delivery delay in C++ client URL: https://github.com/apache/pulsar/pull/5317#issuecomment-539249390 run java8 tests run integration tests This is an automated message

[GitHub] [pulsar] aahmed-se opened a new pull request #5335: Build python 37 artifacts and use them for the docker image

2019-10-07 Thread GitBox
aahmed-se opened a new pull request #5335: Build python 37 artifacts and use them for the docker image URL: https://github.com/apache/pulsar/pull/5335 This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] aahmed-se commented on issue #5334: When building docker image, copy tarball at the end to use build cache

2019-10-07 Thread GitBox
aahmed-se commented on issue #5334: When building docker image, copy tarball at the end to use build cache URL: https://github.com/apache/pulsar/pull/5334#issuecomment-539293552 run java8 tests This is an automated message

[pulsar-client-go] branch master updated: (refactor)clean code (#70)

2019-10-07 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 702405f (refactor)clean code (#70)

[GitHub] [pulsar-client-go] wolfstudy merged pull request #70: [refactor] clean code

2019-10-07 Thread GitBox
wolfstudy merged pull request #70: [refactor] clean code URL: https://github.com/apache/pulsar-client-go/pull/70 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-539346131 run java8 tests This is an

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-539278587 run java8 tests This is an

[GitHub] [pulsar] congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 removed a comment on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-539278683 run Integration Tests This is an

[GitHub] [pulsar] grantwwu commented on issue #4780: Clarify how retention interacts with readers

2019-10-07 Thread GitBox
grantwwu commented on issue #4780: Clarify how retention interacts with readers URL: https://github.com/apache/pulsar/pull/4780#issuecomment-539332962 Er, this got merged with TODOs still in it... This is an automated message

[GitHub] [pulsar] congbobo184 commented on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 commented on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-539346131 run java8 tests This is an automated

[GitHub] [pulsar] merlimat commented on a change in pull request #5320: Expose metrics of Caffeine and Guava caches

2019-10-07 Thread GitBox
merlimat commented on a change in pull request #5320: Expose metrics of Caffeine and Guava caches URL: https://github.com/apache/pulsar/pull/5320#discussion_r332277957 ## File path: pulsar-zookeeper-utils/src/main/java/org/apache/pulsar/stats/CacheMetricsCollector.java ##

[GitHub] [pulsar] jerrypeng commented on issue #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
jerrypeng commented on issue #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#issuecomment-539261288 rerun java8 tests This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] aahmed-se commented on issue #5335: Build python 37 artifacts and use them for the docker image

2019-10-07 Thread GitBox
aahmed-se commented on issue #5335: Build python 37 artifacts and use them for the docker image URL: https://github.com/apache/pulsar/pull/5335#issuecomment-539272173 run cpp tests This is an automated message from the

[GitHub] [pulsar] merlimat commented on a change in pull request #5335: Build python 37 artifacts and use them for the docker image

2019-10-07 Thread GitBox
merlimat commented on a change in pull request #5335: Build python 37 artifacts and use them for the docker image URL: https://github.com/apache/pulsar/pull/5335#discussion_r332304347 ## File path: docker/pulsar/scripts/install-pulsar-client-37.sh ## @@ -20,5 +20,5 @@

[GitHub] [pulsar-client-go] wolfstudy commented on issue #70: [refactor] clean code

2019-10-07 Thread GitBox
wolfstudy commented on issue #70: [refactor] clean code URL: https://github.com/apache/pulsar-client-go/pull/70#issuecomment-539283480 run integration tests This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] merlimat commented on issue #5329: Fix ConsumerRedeliveryTest by ensure clean state on each run

2019-10-07 Thread GitBox
merlimat commented on issue #5329: Fix ConsumerRedeliveryTest by ensure clean state on each run URL: https://github.com/apache/pulsar/pull/5329#issuecomment-539246854 run cpp tests This is an automated message from the

[GitHub] [pulsar] merlimat commented on issue #5321: [pulsar-broker] Disable stickyRead by default

2019-10-07 Thread GitBox
merlimat commented on issue #5321: [pulsar-broker] Disable stickyRead by default URL: https://github.com/apache/pulsar/pull/5321#issuecomment-539246934 run java8 tests This is an automated message from the Apache Git Service.

[GitHub] [pulsar] merlimat commented on issue #5319: Collect broker publish latency in prometheus

2019-10-07 Thread GitBox
merlimat commented on issue #5319: Collect broker publish latency in prometheus URL: https://github.com/apache/pulsar/pull/5319#issuecomment-539247286 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] congbobo184 commented on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 commented on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-539278587 run java8 tests This is an automated

[GitHub] [pulsar] congbobo184 commented on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer

2019-10-07 Thread GitBox
congbobo184 commented on issue #5227: [PIP-44] Separate schema compatibility checker for producer and consumer URL: https://github.com/apache/pulsar/pull/5227#issuecomment-539278683 run Integration Tests This is an automated

[pulsar] branch master updated (21e2b52 -> d7012ae)

2019-10-07 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 21e2b52 Fix Cmake to build _pulsar for osx (#5263) add d7012ae Collect broker publish latency in prometheus

[GitHub] [pulsar] merlimat merged pull request #5319: Collect broker publish latency in prometheus

2019-10-07 Thread GitBox
merlimat merged pull request #5319: Collect broker publish latency in prometheus URL: https://github.com/apache/pulsar/pull/5319 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] aahmed-se commented on issue #5334: When building docker image, copy tarball at the end to use build cache

2019-10-07 Thread GitBox
aahmed-se commented on issue #5334: When building docker image, copy tarball at the end to use build cache URL: https://github.com/apache/pulsar/pull/5334#issuecomment-539299026 run integration tests This is an automated

[GitHub] [pulsar] rocketraman commented on issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true

2019-10-07 Thread GitBox
rocketraman commented on issue #5325: Pulsar on k8s: broker start failure when set functionsWorkerEnabled=true URL: https://github.com/apache/pulsar/issues/5325#issuecomment-539317646 Hmm, for some reason it did not work for me. I have exactly that (based on the sample in the repo) and

[GitHub] [pulsar] rocketraman opened a new issue #5309: [docs] Clarify wording around retention policies

2019-10-07 Thread GitBox
rocketraman opened a new issue #5309: [docs] Clarify wording around retention policies URL: https://github.com/apache/pulsar/issues/5309 In section https://pulsar.apache.org/docs/en/cookbooks-retention-expiry/#retention-policies, this text appears: > When you set a size limit of,

[GitHub] [pulsar] wolfstudy commented on issue #5305: [functions] Add log level in a property for log topic

2019-10-07 Thread GitBox
wolfstudy commented on issue #5305: [functions] Add log level in a property for log topic URL: https://github.com/apache/pulsar/pull/5305#issuecomment-539325826 run integration tests run java8 tests This is an automated

[GitHub] [pulsar-client-go] wolfstudy commented on issue #69: [ISSUE #68][feat]add Option for new producer (#68)

2019-10-07 Thread GitBox
wolfstudy commented on issue #69: [ISSUE #68][feat]add Option for new producer (#68) URL: https://github.com/apache/pulsar-client-go/pull/69#issuecomment-539327527 I think this is a good proposal to refactor the API interface. @xujianhai666 Can you fix this for `ClientOption` and

[GitHub] [pulsar-client-go] wolfstudy commented on issue #66: Fix interface conversion panic

2019-10-07 Thread GitBox
wolfstudy commented on issue #66: Fix interface conversion panic URL: https://github.com/apache/pulsar-client-go/pull/66#issuecomment-539327960 run integration tests This is an automated message from the Apache Git Service.

[GitHub] [pulsar] merlimat opened a new pull request #5333: Do not use timed receive in BrokerClientIntegrationTest.testUnsupportedBatchMessageConsumer

2019-10-07 Thread GitBox
merlimat opened a new pull request #5333: Do not use timed receive in BrokerClientIntegrationTest.testUnsupportedBatchMessageConsumer URL: https://github.com/apache/pulsar/pull/5333 ### Motivation The test is failing because the `receive(1, TimeUnit.SECONDS)` takes more than 1 sec

[GitHub] [pulsar] merlimat opened a new pull request #5334: When building docker image, copy tarball at the end to use build cache

2019-10-07 Thread GitBox
merlimat opened a new pull request #5334: When building docker image, copy tarball at the end to use build cache URL: https://github.com/apache/pulsar/pull/5334 ### Motivation Since the Pulsar tarball is the only part that keeps changing, we should copy it at the very end so that

[GitHub] [pulsar] aahmed-se commented on issue #5335: Build python 37 artifacts and use them for the docker image

2019-10-07 Thread GitBox
aahmed-se commented on issue #5335: Build python 37 artifacts and use them for the docker image URL: https://github.com/apache/pulsar/pull/5335#issuecomment-539272206 run java8 tests This is an automated message from the

[GitHub] [pulsar] aahmed-se commented on a change in pull request #5335: Build python 37 artifacts and use them for the docker image

2019-10-07 Thread GitBox
aahmed-se commented on a change in pull request #5335: Build python 37 artifacts and use them for the docker image URL: https://github.com/apache/pulsar/pull/5335#discussion_r332304726 ## File path: docker/pulsar/scripts/install-pulsar-client-37.sh ## @@ -20,5 +20,5 @@

[GitHub] [pulsar] bbonnin commented on issue #5305: [functions] Add log level in a property for log topic

2019-10-07 Thread GitBox
bbonnin commented on issue #5305: [functions] Add log level in a property for log topic URL: https://github.com/apache/pulsar/pull/5305#issuecomment-539333682 is there a way to restart tests that have failed in local ? This