[incubator-pulsar] branch master updated: Make client configuration protected in PulsarAdminTool (#1312)

2018-02-28 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 72aa7d9 Make client configuration

[GitHub] sijie closed pull request #1312: Make client configuration protected in PulsarAdminTool

2018-02-28 Thread GitBox
sijie closed pull request #1312: Make client configuration protected in PulsarAdminTool URL: https://github.com/apache/incubator-pulsar/pull/1312 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:

[GitHub] sijie closed pull request #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone

2018-02-28 Thread GitBox
sijie closed pull request #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone URL: https://github.com/apache/incubator-pulsar/pull/1309 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[incubator-pulsar] branch master updated: Precreate readonly path and increase disk usage thresholds for pulsar standalone (#1309)

2018-02-28 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 d0f8a40 Precreate readonly path and

[GitHub] sijie commented on issue #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone

2018-02-28 Thread GitBox
sijie commented on issue #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone URL: https://github.com/apache/incubator-pulsar/pull/1309#issuecomment-369469325 retest this please This is an

[GitHub] sijie opened a new pull request #1312: Make client configuration protected in PulsarAdminTool

2018-02-28 Thread GitBox
sijie opened a new pull request #1312: Make client configuration protected in PulsarAdminTool URL: https://github.com/apache/incubator-pulsar/pull/1312 This change allows pulsar-functions access client configuration in admin tool

[GitHub] merlimat opened a new pull request #1311: Converted main part of code to use builder APIs with typed interface

2018-02-28 Thread GitBox
merlimat opened a new pull request #1311: Converted main part of code to use builder APIs with typed interface URL: https://github.com/apache/incubator-pulsar/pull/1311 ### Motivation As part of PIP-12, in #1089 we have introduced a builder style interface to create

[GitHub] sijie commented on issue #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone

2018-02-28 Thread GitBox
sijie commented on issue #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone URL: https://github.com/apache/incubator-pulsar/pull/1309#issuecomment-369438346 retest this please This is an

[GitHub] jai1 commented on a change in pull request #1297: Add Configuration to set tlsClientAuth

2018-02-28 Thread GitBox
jai1 commented on a change in pull request #1297: Add Configuration to set tlsClientAuth URL: https://github.com/apache/incubator-pulsar/pull/1297#discussion_r171424385 ## File path:

[GitHub] merlimat commented on a change in pull request #1297: Add Configuration to set tlsClientAuth

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1297: Add Configuration to set tlsClientAuth URL: https://github.com/apache/incubator-pulsar/pull/1297#discussion_r171423209 ## File path:

[GitHub] merlimat commented on a change in pull request #1310: Utilities for arquillian integration tests

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1310: Utilities for arquillian integration tests URL: https://github.com/apache/incubator-pulsar/pull/1310#discussion_r171419235 ## File path: tests/integration-tests-utils/src/main/resources/log4j.properties ## @@ -0,0

[GitHub] merlimat commented on a change in pull request #1232: Schema registry (1/4)

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1232: Schema registry (1/4) URL: https://github.com/apache/incubator-pulsar/pull/1232#discussion_r171418965 ## File path: pulsar-common/src/main/proto/PulsarApi.proto ## @@ -197,6 +205,8 @@ message CommandSubscribe {

[GitHub] merlimat commented on a change in pull request #1232: Schema registry (1/4)

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1232: Schema registry (1/4) URL: https://github.com/apache/incubator-pulsar/pull/1232#discussion_r171411681 ## File path: pulsar-common/src/main/proto/PulsarApi.proto ## @@ -387,6 +400,7 @@ message CommandProducerSuccess {

[GitHub] merlimat commented on a change in pull request #1232: Schema registry (1/4)

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1232: Schema registry (1/4) URL: https://github.com/apache/incubator-pulsar/pull/1232#discussion_r171418590 ## File path: pulsar-common/src/main/proto/PulsarApi.proto ## @@ -22,6 +22,13 @@ package pulsar.proto; option

[GitHub] zhaijack commented on issue #1176: Documentation for `regex_pattern_subscribe`

2018-02-28 Thread GitBox
zhaijack commented on issue #1176: Documentation for `regex_pattern_subscribe` URL: https://github.com/apache/incubator-pulsar/issues/1176#issuecomment-369406971 related PRs #1103 #1279 #1298 This is an automated message

[GitHub] zhaijack commented on issue #1176: Documentation for `regex_pattern_subscribe`

2018-02-28 Thread GitBox
zhaijack commented on issue #1176: Documentation for `regex_pattern_subscribe` URL: https://github.com/apache/incubator-pulsar/issues/1176#issuecomment-369406321 the PRs has been merged. we could start this issue This is an

[GitHub] zhaijack commented on issue #1102: PIP-13-3: sub-regex: auto-discover topic addition/deletion

2018-02-28 Thread GitBox
zhaijack commented on issue #1102: PIP-13-3: sub-regex: auto-discover topic addition/deletion URL: https://github.com/apache/incubator-pulsar/issues/1102#issuecomment-369403218 https://github.com/apache/incubator-pulsar/pull/1298 merged it

[GitHub] zhaijack closed issue #1101: PIP-13-2: support regex based subscription

2018-02-28 Thread GitBox
zhaijack closed issue #1101: PIP-13-2: support regex based subscription URL: https://github.com/apache/incubator-pulsar/issues/1101 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] zhaijack commented on issue #1101: PIP-13-2: support regex based subscription

2018-02-28 Thread GitBox
zhaijack commented on issue #1101: PIP-13-2: support regex based subscription URL: https://github.com/apache/incubator-pulsar/issues/1101#issuecomment-369403560 merged by #1279 This is an automated message from the Apache

[GitHub] zhaijack commented on issue #1102: PIP-13-3: sub-regex: auto-discover topic addition/deletion

2018-02-28 Thread GitBox
zhaijack commented on issue #1102: PIP-13-3: sub-regex: auto-discover topic addition/deletion URL: https://github.com/apache/incubator-pulsar/issues/1102#issuecomment-369403218 https://github.com/apache/incubator-pulsar/pull/1175 merged it

[incubator-pulsar] branch master updated: Moved the bk-tests dependency to parent pom (#1308)

2018-02-28 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 3b9f3d8 Moved the bk-tests

[GitHub] merlimat closed pull request #1308: Moved the bk-tests dependency to parent pom

2018-02-28 Thread GitBox
merlimat closed pull request #1308: Moved the bk-tests dependency to parent pom URL: https://github.com/apache/incubator-pulsar/pull/1308 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

[GitHub] jai1 commented on issue #1297: Add Configuration to set tlsClientAuth

2018-02-28 Thread GitBox
jai1 commented on issue #1297: Add Configuration to set tlsClientAuth URL: https://github.com/apache/incubator-pulsar/pull/1297#issuecomment-369370856 retest this please This is an automated message from the Apache Git

[GitHub] sijie opened a new pull request #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone

2018-02-28 Thread GitBox
sijie opened a new pull request #1309: Precreate readonly path and increase disk usage thresholds for pulsar standalone URL: https://github.com/apache/incubator-pulsar/pull/1309 ### Motivation - precreate readonly path to avoid confused messages - use high disk usage

[GitHub] jai1 commented on issue #1297: Add Configuration to set tlsClientAuth

2018-02-28 Thread GitBox
jai1 commented on issue #1297: Add Configuration to set tlsClientAuth URL: https://github.com/apache/incubator-pulsar/pull/1297#issuecomment-369370600 @maskit @merlimat - Once the client is connected the authentication happens on handleConnect, if I am a mischief maker (not using

[GitHub] merlimat opened a new pull request #1308: Moved the bk-tests dependency to parent pom

2018-02-28 Thread GitBox
merlimat opened a new pull request #1308: Moved the bk-tests dependency to parent pom URL: https://github.com/apache/incubator-pulsar/pull/1308 ### Motivation Since we moved `MockBookKeeper` into BookKeeper repo, we needed to add multiple times the same dependencies to

[GitHub] merlimat commented on issue #1297: Add Configuration to set tlsClientAuth

2018-02-28 Thread GitBox
merlimat commented on issue #1297: Add Configuration to set tlsClientAuth URL: https://github.com/apache/incubator-pulsar/pull/1297#issuecomment-369353839 @jai1 What is the intention of this change? Is it to add an earlier level of protection so that you can configure Netty to directly

[GitHub] mgodave commented on a change in pull request #987: Add Type information to Pulsar client interface

2018-02-28 Thread GitBox
mgodave commented on a change in pull request #987: Add Type information to Pulsar client interface URL: https://github.com/apache/incubator-pulsar/pull/987#discussion_r171144982 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerBuilderImpl.java

[GitHub] mgodave commented on a change in pull request #987: Add Type information to Pulsar client interface

2018-02-28 Thread GitBox
mgodave commented on a change in pull request #987: Add Type information to Pulsar client interface URL: https://github.com/apache/incubator-pulsar/pull/987#discussion_r171323156 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/api/PulsarClient.java ##

[GitHub] merlimat commented on a change in pull request #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r171318004 ## File path:

[GitHub] merlimat commented on a change in pull request #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r171319116 ## File path: faq.md ## @@ -1,273

[GitHub] merlimat commented on a change in pull request #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-02-28 Thread GitBox
merlimat commented on a change in pull request #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r171318823 ## File path:

[incubator-pulsar] branch master updated: Docker image for integration testing (#1275)

2018-02-28 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 9c54e73 Docker image for integration

[incubator-pulsar] branch master updated: Cleanup old compacted topic ledgers when a new one is available (#1302)

2018-02-28 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 3a6b0fb Cleanup old compacted topic

[GitHub] merlimat closed pull request #1298: PIP-13-3/3: auto subscribe based on regex pattern topics changing

2018-02-28 Thread GitBox
merlimat closed pull request #1298: PIP-13-3/3: auto subscribe based on regex pattern topics changing URL: https://github.com/apache/incubator-pulsar/pull/1298 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] zhaijack commented on issue #1298: PIP-13-3/3: auto subscribe based on regex pattern topics changing

2018-02-28 Thread GitBox
zhaijack commented on issue #1298: PIP-13-3/3: auto subscribe based on regex pattern topics changing URL: https://github.com/apache/incubator-pulsar/pull/1298#issuecomment-369196117 Thanks for the instructions @merlimat , updated it.

[GitHub] ivankelly commented on issue #1302: Cleanup old compacted topic ledgers when a new one is available

2018-02-28 Thread GitBox
ivankelly commented on issue #1302: Cleanup old compacted topic ledgers when a new one is available URL: https://github.com/apache/incubator-pulsar/pull/1302#issuecomment-369160290 retest this please // seem to be flakes