[GitHub] sijie commented on a change in pull request #2347: Fix: function with multi-topic not acking on effectively-once

2018-08-08 Thread GitBox
sijie commented on a change in pull request #2347: Fix: function with multi-topic not acking on effectively-once URL: https://github.com/apache/incubator-pulsar/pull/2347#discussion_r208791550 ## File path:

[GitHub] rdhabalia opened a new pull request #2347: Fix: function with multi-topic not acking on effectively-once

2018-08-08 Thread GitBox
rdhabalia opened a new pull request #2347: Fix: function with multi-topic not acking on effectively-once URL: https://github.com/apache/incubator-pulsar/pull/2347 ### Motivation `MultiTopicsConsumerImpl` doesn't support `acknowledgeCumulativeAsync` and therefore, function with

[GitHub] jiazhai opened a new pull request #2346: Issue #2330: change getTopicName in MultiTopicsConsumer

2018-08-08 Thread GitBox
jiazhai opened a new pull request #2346: Issue #2330: change getTopicName in MultiTopicsConsumer URL: https://github.com/apache/incubator-pulsar/pull/2346 ### Motivation `getTopicName` in `TopicMessageIdImpl` and `TopicMessageImpl` is some kind of confusing. Developer may

[incubator-pulsar] branch master updated: REST and CLI to get function metrics in json for monitoring (#2296)

2018-08-08 Thread rdhabalia
This is an automated email from the ASF dual-hosted git repository. rdhabalia pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 20b41de REST and CLI to get

[GitHub] rdhabalia closed pull request #2296: REST and CLI to get function metrics in json for monitoring

2018-08-08 Thread GitBox
rdhabalia closed pull request #2296: REST and CLI to get function metrics in json for monitoring URL: https://github.com/apache/incubator-pulsar/pull/2296 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] sijie commented on issue #2283: Improve error message if authorization is not enabled

2018-08-08 Thread GitBox
sijie commented on issue #2283: Improve error message if authorization is not enabled URL: https://github.com/apache/incubator-pulsar/issues/2283#issuecomment-411575999 also need to remove authorizationEnabled since there is already a authProvider setting

[GitHub] sijie commented on issue #2324: Use pulsar-daemon start zookeeper will use port of 8080

2018-08-08 Thread GitBox
sijie commented on issue #2324: Use pulsar-daemon start zookeeper will use port of 8080 URL: https://github.com/apache/incubator-pulsar/issues/2324#issuecomment-411561820 Close this one since it was answered. This is an

[GitHub] sijie closed pull request #2345: [documentation] Fix syntax error on tiered storage documentation

2018-08-08 Thread GitBox
sijie closed pull request #2345: [documentation] Fix syntax error on tiered storage documentation URL: https://github.com/apache/incubator-pulsar/pull/2345 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

[incubator-pulsar] branch master updated: [documentation] Fix syntax error on tiered storage documentation (#2345)

2018-08-08 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 3b3f541 [documentation] Fix syntax

[GitHub] sijie opened a new pull request #2345: [documentation] Fix syntax error on tiered storage documentation

2018-08-08 Thread GitBox
sijie opened a new pull request #2345: [documentation] Fix syntax error on tiered storage documentation URL: https://github.com/apache/incubator-pulsar/pull/2345 ### Motivation There is a syntax error in tiered storage doc introduced by #2152. ``` Liquid Exception:

[GitHub] srkukarni commented on issue #2296: REST and CLI to get function metrics in json for monitoring

2018-08-08 Thread GitBox
srkukarni commented on issue #2296: REST and CLI to get function metrics in json for monitoring URL: https://github.com/apache/incubator-pulsar/pull/2296#issuecomment-411502635 rerun integration tests This is an automated

[GitHub] sijie commented on issue #2331: [conf] add zookeeper admin server settings to the configuration files

2018-08-08 Thread GitBox
sijie commented on issue #2331: [conf] add zookeeper admin server settings to the configuration files URL: https://github.com/apache/incubator-pulsar/pull/2331#issuecomment-411501364 run java8 tests This is an automated

[GitHub] sijie opened a new pull request #2344: [documentation] Cherrypick #2341 to version 2.1 documentation

2018-08-08 Thread GitBox
sijie opened a new pull request #2344: [documentation] Cherrypick #2341 to version 2.1 documentation URL: https://github.com/apache/incubator-pulsar/pull/2344 Cherrypick documentation changes #2341 to 2.1 documentation

[GitHub] sijie closed issue #2252: documentation: incorrect flags for bin/pulsar

2018-08-08 Thread GitBox
sijie closed issue #2252: documentation: incorrect flags for bin/pulsar URL: https://github.com/apache/incubator-pulsar/issues/2252 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-pulsar] branch master updated: [documentation] Issue 2252: incorrect flags for bin/pulsar (#2341)

2018-08-08 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 0260df0 [documentation] Issue 2252:

[GitHub] sijie commented on issue #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N

2018-08-08 Thread GitBox
sijie commented on issue #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N URL: https://github.com/apache/incubator-pulsar/pull/2342#issuecomment-411499082 rerun java8 tests This is an

[GitHub] sijie opened a new issue #2343: Enable state in cluster mode

2018-08-08 Thread GitBox
sijie opened a new issue #2343: Enable state in cluster mode URL: https://github.com/apache/incubator-pulsar/issues/2343 A couple of things missing on enabling state in cluster mode when I am working on #2335. This is the master ticket for tracking the work items for enabling state

[GitHub] srkukarni edited a comment on issue #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N

2018-08-08 Thread GitBox
srkukarni edited a comment on issue #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N URL: https://github.com/apache/incubator-pulsar/pull/2342#issuecomment-411416988 rerun java8 tests This

[GitHub] srkukarni commented on issue #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N

2018-08-08 Thread GitBox
srkukarni commented on issue #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N URL: https://github.com/apache/incubator-pulsar/pull/2342#issuecomment-411416988 rerun java tests This is an

[GitHub] srkukarni commented on issue #2341: [documentation] Issue 2252: incorrect flags for bin/pulsar

2018-08-08 Thread GitBox
srkukarni commented on issue #2341: [documentation] Issue 2252: incorrect flags for bin/pulsar URL: https://github.com/apache/incubator-pulsar/pull/2341#issuecomment-411416401 rerun integration tests This is an automated

[GitHub] jiazhai commented on issue #2311: [cpp] receiver queue size config acorss partitions in multi-topics-consumer

2018-08-08 Thread GitBox
jiazhai commented on issue #2311: [cpp] receiver queue size config acorss partitions in multi-topics-consumer URL: https://github.com/apache/incubator-pulsar/pull/2311#issuecomment-411383832 @sijie added python config This

[GitHub] jiazhai commented on a change in pull request #2311: [cpp] receiver queue size config acorss partitions in multi-topics-consumer

2018-08-08 Thread GitBox
jiazhai commented on a change in pull request #2311: [cpp] receiver queue size config acorss partitions in multi-topics-consumer URL: https://github.com/apache/incubator-pulsar/pull/2311#discussion_r208520704 ## File path:

[GitHub] sijie opened a new pull request #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N

2018-08-08 Thread GitBox
sijie opened a new pull request #2342: Issue 1067: Problems with Partitioned Topics which name contains -partition-N URL: https://github.com/apache/incubator-pulsar/pull/2342 ### Motivation Fixes #1067. Someone accidentally created a partitioned topic in one of our

[GitHub] sijie opened a new pull request #2341: [documentation] Issue 2252: incorrect flags for bin/pulsar

2018-08-08 Thread GitBox
sijie opened a new pull request #2341: [documentation] Issue 2252: incorrect flags for bin/pulsar URL: https://github.com/apache/incubator-pulsar/pull/2341 ### Motivation Fixes #2252 The documentation lists -c and --conf as possible arguments of the bin/pulsar command.

[GitHub] sijie closed pull request #2152: GCS offload support(4): add documentations for GCS

2018-08-08 Thread GitBox
sijie closed pull request #2152: GCS offload support(4): add documentations for GCS URL: https://github.com/apache/incubator-pulsar/pull/2152 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

[incubator-pulsar] branch master updated: GCS offload support(4): add documentations for GCS (#2152)

2018-08-08 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 5e1ee37 GCS offload support(4): add

[incubator-pulsar] branch master updated: Fix flaky test PersistentTopicE2ETest.testMessageRedelivery (#2320)

2018-08-08 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new d81bf6d Fix flaky test

[GitHub] sijie closed pull request #2320: Fix flaky test PersistentTopicE2ETest.testMessageRedelivery

2018-08-08 Thread GitBox
sijie closed pull request #2320: Fix flaky test PersistentTopicE2ETest.testMessageRedelivery URL: https://github.com/apache/incubator-pulsar/pull/2320 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] sijie commented on issue #2331: [conf] add zookeeper admin server settings to the configuration files

2018-08-08 Thread GitBox
sijie commented on issue #2331: [conf] add zookeeper admin server settings to the configuration files URL: https://github.com/apache/incubator-pulsar/pull/2331#issuecomment-411329730 run java8 tests This is an automated

[incubator-pulsar] branch master updated: Introduce warmup period in performance producer (#2338)

2018-08-08 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new acc26c9 Introduce warmup period in

[GitHub] sijie closed pull request #2337: [website] fix broken links

2018-08-08 Thread GitBox
sijie closed pull request #2337: [website] fix broken links URL: https://github.com/apache/incubator-pulsar/pull/2337 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] sijie closed pull request #2338: Introduce warmup period in performance producer

2018-08-08 Thread GitBox
sijie closed pull request #2338: Introduce warmup period in performance producer URL: https://github.com/apache/incubator-pulsar/pull/2338 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

[incubator-pulsar] branch master updated: [website] fix broken links (#2337)

2018-08-08 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 8cc7498 [website] fix broken links

[incubator-pulsar] branch master updated: added FRench to website (#2339)

2018-08-08 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 20e652b added FRench to website

[GitHub] sijie commented on issue #2339: added FRench to website

2018-08-08 Thread GitBox
sijie commented on issue #2339: added FRench to website URL: https://github.com/apache/incubator-pulsar/pull/2339#issuecomment-411329483 @danielccyr thank you for your contribution. This is an automated message from the

[GitHub] sijie closed pull request #2339: added FRench to website

2018-08-08 Thread GitBox
sijie closed pull request #2339: added FRench to website URL: https://github.com/apache/incubator-pulsar/pull/2339 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

[GitHub] sijie commented on issue #2330: Function localrun quits when using partitioned topic.

2018-08-08 Thread GitBox
sijie commented on issue #2330: Function localrun quits when using partitioned topic. URL: https://github.com/apache/incubator-pulsar/issues/2330#issuecomment-411305563 I had a discussion with @codelipenghui . the original question was around stateful function. stateful function

[GitHub] codelipenghui edited a comment on issue #2330: Functions localrun quit by can't get the reason

2018-08-08 Thread GitBox
codelipenghui edited a comment on issue #2330: Functions localrun quit by can't get the reason URL: https://github.com/apache/incubator-pulsar/issues/2330#issuecomment-411298133 Topic of plat.correctness.verification is a partitioned topic with six partitions. When i set input to

[GitHub] codelipenghui edited a comment on issue #2330: Functions localrun quit by can't get the reason

2018-08-08 Thread GitBox
codelipenghui edited a comment on issue #2330: Functions localrun quit by can't get the reason URL: https://github.com/apache/incubator-pulsar/issues/2330#issuecomment-411298133 Topic of plat.correctness.verification is a partitioned topic with six partitions. When i set input to

[GitHub] codelipenghui commented on issue #2330: Functions localrun quit by can't get the reason

2018-08-08 Thread GitBox
codelipenghui commented on issue #2330: Functions localrun quit by can't get the reason URL: https://github.com/apache/incubator-pulsar/issues/2330#issuecomment-411298133 Topic of plat.correctness.verification is a partitioned topic with six partitions. When i set input to