[GitHub] jerrypeng commented on a change in pull request #2684: [function] Fix expected health check interval

2018-09-30 Thread GitBox
jerrypeng commented on a change in pull request #2684: [function] Fix expected health check interval URL: https://github.com/apache/pulsar/pull/2684#discussion_r221490085 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/RuntimeSpawner.java

[GitHub] codelipenghui commented on issue #2659: Add backlog quota configuration on topic

2018-09-30 Thread GitBox
codelipenghui commented on issue #2659: Add backlog quota configuration on topic URL: https://github.com/apache/pulsar/pull/2659#issuecomment-425769031 #2688 This is the master ticket to tracking this PR This is an automated

[GitHub] codelipenghui commented on issue #2689: Backlog Quota Configuration on topic

2018-09-30 Thread GitBox
codelipenghui commented on issue #2689: Backlog Quota Configuration on topic URL: https://github.com/apache/pulsar/issues/2689#issuecomment-425768867 pull request from [#2659](https://github.com/apache/pulsar/pull/2659) This

[GitHub] codelipenghui opened a new issue #2689: Backlog Quota Configuration on topic

2018-09-30 Thread GitBox
codelipenghui opened a new issue #2689: Backlog Quota Configuration on topic URL: https://github.com/apache/pulsar/issues/2689 This is a sub task for #2688 for adding policy of backlog quota on topic level, addressing comment: [#2659](https://github.com/apache/pulsar/pull/2659)(review)

[GitHub] codelipenghui opened a new issue #2688: Polices On Topic

2018-09-30 Thread GitBox
codelipenghui opened a new issue #2688: Polices On Topic URL: https://github.com/apache/pulsar/issues/2688 This is the master ticket for tracking all the tasks for polices on topic - [ ] 1. MessageTTL [Get, Set] - [ ] 2. Retention Policy [Get, Set] - [ ] 3. Message Dispatch

[GitHub] sfescape commented on issue #2687: Websocket API - Expose ReachedEndOfTopic in Reader/Consumer API

2018-09-30 Thread GitBox
sfescape commented on issue #2687: Websocket API - Expose ReachedEndOfTopic in Reader/Consumer API URL: https://github.com/apache/pulsar/issues/2687#issuecomment-425767843 In looking at the Websocket API documentation, I don't see anything in the endpoint responses that give me that

[GitHub] jerrypeng edited a comment on issue #2687: Websocket API - Expose ReachedEndOfTopic in Reader/Consumer API

2018-09-30 Thread GitBox
jerrypeng edited a comment on issue #2687: Websocket API - Expose ReachedEndOfTopic in Reader/Consumer API URL: https://github.com/apache/pulsar/issues/2687#issuecomment-425767389 @sfescape the Reader and Consumer API does have this functionality: reader.hasReachedEndOfTopic()

[GitHub] jerrypeng commented on issue #2687: Websocket API - Expose ReachedEndOfTopic in Reader/Consumer API

2018-09-30 Thread GitBox
jerrypeng commented on issue #2687: Websocket API - Expose ReachedEndOfTopic in Reader/Consumer API URL: https://github.com/apache/pulsar/issues/2687#issuecomment-425767389 @sfescape the Reader API does have this functionality: reader.hasReachedEndOfTopic()

[pulsar] branch master updated: Change access to public for method getProducers(). (#2663)

2018-09-30 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 e7930f3 Change access to public for method

[GitHub] jiazhai commented on issue #2615: Azure offloader

2018-09-30 Thread GitBox
jiazhai commented on issue #2615: Azure offloader URL: https://github.com/apache/pulsar/pull/2615#issuecomment-425764181 rerun java8 tests This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] jiazhai edited a comment on issue #2615: Azure offloader

2018-09-30 Thread GitBox
jiazhai edited a comment on issue #2615: Azure offloader URL: https://github.com/apache/pulsar/pull/2615#issuecomment-425764181 rerun java8 tests for org.apache.pulsar.broker.service.v1.V1_ReplicatorTest.testCloseReplicatorStartProducer

[GitHub] viirya commented on issue #2681: Fix ConsumerBuilder and PulsarClient document

2018-09-30 Thread GitBox
viirya commented on issue #2681: Fix ConsumerBuilder and PulsarClient document URL: https://github.com/apache/pulsar/pull/2681#issuecomment-425763865 Thank you! This is an automated message from the Apache Git Service. To

[GitHub] sijie opened a new pull request #2686: [functions][metrics] Update function metrics

2018-09-30 Thread GitBox
sijie opened a new pull request #2686: [functions][metrics] Update function metrics URL: https://github.com/apache/pulsar/pull/2686 some prometheus agent are picky at the prometheus output format. this change to make metric type is by the metric. The resulting output is showed as below

[GitHub] sijie commented on issue #2685: [schema] introduce AUTO_PRODUCE schema

2018-09-30 Thread GitBox
sijie commented on issue #2685: [schema] introduce AUTO_PRODUCE schema URL: https://github.com/apache/pulsar/pull/2685#issuecomment-425761238 this pr is based on #2684 This is an automated message from the Apache Git

[GitHub] sijie opened a new pull request #2685: [schema] introduce AUTO_PRODUCE schema

2018-09-30 Thread GitBox
sijie opened a new pull request #2685: [schema] introduce AUTO_PRODUCE schema URL: https://github.com/apache/pulsar/pull/2685 *Motivation* Currently trigger function is broken due to the schema enforcement we added recently: A producer without schema can't produce messages into

[GitHub] sijie opened a new pull request #2683: [tests] disabled cassandra connector test and python function test

2018-09-30 Thread GitBox
sijie opened a new pull request #2683: [tests] disabled cassandra connector test and python function test URL: https://github.com/apache/pulsar/pull/2683 These 3 tests are having some weird problems. Disabling them to bring integration tests back to normal. Will fix them in the

[GitHub] sijie commented on issue #2622: Add Maven Source Plugin.

2018-09-30 Thread GitBox
sijie commented on issue #2622: Add Maven Source Plugin. URL: https://github.com/apache/pulsar/pull/2622#issuecomment-425743488 run cpp tests This is an automated message from the Apache Git Service. To respond to the

[GitHub] sijie closed pull request #2681: Fix ConsumerBuilder and PulsarClient document

2018-09-30 Thread GitBox
sijie closed pull request #2681: Fix ConsumerBuilder and PulsarClient document URL: https://github.com/apache/pulsar/pull/2681 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

[pulsar] branch master updated: Add support for schema extraction from a jar (#2578)

2018-09-30 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 671c7bc Add support for schema extraction from

[GitHub] sijie commented on issue #2663: Change access to public for method getProducers().

2018-09-30 Thread GitBox
sijie commented on issue #2663: Change access to public for method getProducers(). URL: https://github.com/apache/pulsar/pull/2663#issuecomment-425741522 run java8 tests This is an automated message from the Apache Git

[pulsar] branch master updated: Clarify and add tests for schema change scenarios (#2669)

2018-09-30 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 ff6de4a Clarify and add tests for schema change

[GitHub] sijie closed pull request #2669: Clarify and add tests for schema change scenarios

2018-09-30 Thread GitBox
sijie closed pull request #2669: Clarify and add tests for schema change scenarios URL: https://github.com/apache/pulsar/pull/2669 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] sijie closed pull request #2679: Add types comment in Prometheus stats

2018-09-30 Thread GitBox
sijie closed pull request #2679: Add types comment in Prometheus stats URL: https://github.com/apache/pulsar/pull/2679 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

[GitHub] viirya opened a new pull request #2681: Fix ConsumerBuilder and PulsarClient document

2018-09-30 Thread GitBox
viirya opened a new pull request #2681: Fix ConsumerBuilder and PulsarClient document URL: https://github.com/apache/pulsar/pull/2681 ### Motivation There are few document error in code. We should fix them. ### Modifications Fixes document in `ConsumerBuilder` and

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

2018-09-30 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 9c0fc04 Updated site at revision 357c5d1

[GitHub] kuangye098 commented on issue #2664: Support for Transactional Message

2018-09-30 Thread GitBox
kuangye098 commented on issue #2664: Support for Transactional Message URL: https://github.com/apache/pulsar/issues/2664#issuecomment-425704917 @sijie OK,I will read it carefully and then write a PIP to describe the proposal of changes.

[GitHub] Carl-Hee commented on issue #2551: Client switch connection between different pulsar cluster.

2018-09-30 Thread GitBox
Carl-Hee commented on issue #2551: Client switch connection between different pulsar cluster. URL: https://github.com/apache/pulsar/issues/2551#issuecomment-425704243 @sijie Is dns necessary, or can we use IP address directly?

[GitHub] sijie commented on issue #2680: pulsar golang client install failed.

2018-09-30 Thread GitBox
sijie commented on issue #2680: pulsar golang client install failed. URL: https://github.com/apache/pulsar/issues/2680#issuecomment-425701842 @templarzq pulsar go client is a cgo wrapper over existing cpp client. currently we only provide cpp client for mac and linux. it doesn't support