[GitHub] nkurihar commented on issue #1352: Delete inactive subscriptions automatically

2018-04-18 Thread GitBox
nkurihar commented on issue #1352: Delete inactive subscriptions automatically URL: https://github.com/apache/incubator-pulsar/pull/1352#issuecomment-382602294 @merlimat Any thoughts? This is an automated message from the

[GitHub] nkurihar commented on issue #1380: Refactor REST API for namespace policy

2018-04-18 Thread GitBox
nkurihar commented on issue #1380: Refactor REST API for namespace policy URL: https://github.com/apache/incubator-pulsar/pull/1380#issuecomment-382601997 @merlimat Thanks for comments. Let us close this PR for now and re-consider how to refactor.

[GitHub] nkurihar closed pull request #1380: Refactor REST API for namespace policy

2018-04-18 Thread GitBox
nkurihar closed pull request #1380: Refactor REST API for namespace policy URL: https://github.com/apache/incubator-pulsar/pull/1380 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] zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions

2018-04-18 Thread GitBox
zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions URL: https://github.com/apache/incubator-pulsar/issues/1616#issuecomment-382555310 Anyway creating 1000 subscriptions during even in ~7 seconds is still a slow performance.

[GitHub] zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions

2018-04-18 Thread GitBox
zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions URL: https://github.com/apache/incubator-pulsar/issues/1616#issuecomment-382554969 Anyway creating 1000 subscriptions during ~7 seconds is still a slow performance.

[GitHub] zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions

2018-04-18 Thread GitBox
zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions URL: https://github.com/apache/incubator-pulsar/issues/1616#issuecomment-382554969 Anyway creating 1000 subscriptions during ~7 seconds is still a slow performance.

[GitHub] zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions

2018-04-18 Thread GitBox
zubchenok commented on issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions URL: https://github.com/apache/incubator-pulsar/issues/1616#issuecomment-382554740 @sijie commented: > I think the different is admin.createSubscription is using http

[GitHub] zubchenok opened a new issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions

2018-04-18 Thread GitBox
zubchenok opened a new issue #1616: PulsarAdmin has a significantly slower performance for creating subscriptions URL: https://github.com/apache/incubator-pulsar/issues/1616 Expected behavior PulsarAdmin should have faster performance for subscription creation than creating

[GitHub] zubchenok commented on issue #1615: PulsarAdmin.createSubscription method does not work against a multi-node Pulsar cluster

2018-04-18 Thread GitBox
zubchenok commented on issue #1615: PulsarAdmin.createSubscription method does not work against a multi-node Pulsar cluster URL: https://github.com/apache/incubator-pulsar/issues/1615#issuecomment-382552461 Similar issue happens if you create a subscription via bin/pulsar-admin or HTTP

[GitHub] zubchenok opened a new issue #1615: PulsarAdmin.createSubscription method does not work against a multi-node Pulsar cluster

2018-04-18 Thread GitBox
zubchenok opened a new issue #1615: PulsarAdmin.createSubscription method does not work against a multi-node Pulsar cluster URL: https://github.com/apache/incubator-pulsar/issues/1615 Expected behavior Subscription create for a topic with PulsarAdmin.createSubscription method

[GitHub] merlimat closed pull request #1612: Fix pulsar broker handling WebApplicationException

2018-04-18 Thread GitBox
merlimat closed pull request #1612: Fix pulsar broker handling WebApplicationException URL: https://github.com/apache/incubator-pulsar/pull/1612 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: Fix pulsar broker handling WebApplicationException (#1612)

2018-04-18 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 f8c2d25 Fix pulsar broker handling

[GitHub] merlimat commented on issue #1607: Fixed initial default namespace setup with current cluster setup

2018-04-18 Thread GitBox
merlimat commented on issue #1607: Fixed initial default namespace setup with current cluster setup URL: https://github.com/apache/incubator-pulsar/pull/1607#issuecomment-382545082 retest this please This is an automated

[GitHub] merlimat opened a new issue #1614: Itermittent test timeout on PersistentFailoverE2ETest.testSimpleConsumerEventsWithoutPartition

2018-04-18 Thread GitBox
merlimat opened a new issue #1614: Itermittent test timeout on PersistentFailoverE2ETest.testSimpleConsumerEventsWithoutPartition URL: https://github.com/apache/incubator-pulsar/issues/1614 Expected behavior The time is timing out several times. eg:

[GitHub] merlimat closed pull request #1602: Proxy should close connections immediately after exception caught

2018-04-18 Thread GitBox
merlimat closed pull request #1602: Proxy should close connections immediately after exception caught URL: https://github.com/apache/incubator-pulsar/pull/1602 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 closed pull request #1606: Enhance Pulsar Connect api by attaching more information about the message

2018-04-18 Thread GitBox
sijie closed pull request #1606: Enhance Pulsar Connect api by attaching more information about the message URL: https://github.com/apache/incubator-pulsar/pull/1606 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[incubator-pulsar] branch master updated: Enhance Pulsar Connect api by attaching more information about the message (#1606)

2018-04-18 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 e6d1551 Enhance Pulsar Connect api by

[GitHub] sijie opened a new pull request #1613: Add a integration test for create subscription CLI command

2018-04-18 Thread GitBox
sijie opened a new pull request #1613: Add a integration test for create subscription CLI command URL: https://github.com/apache/incubator-pulsar/pull/1613 ### Motivation due to #1612 , create subscription will fail with `500` when connecting to a broker that doesn't own the topic.

[incubator-pulsar] branch master updated: On tenants cli tool, default tenant to all clusters if not specified (#1584)

2018-04-18 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 1334005 On tenants cli tool, default

[GitHub] merlimat closed pull request #1579: Added option to create namespace on CLI passing list of clusters

2018-04-18 Thread GitBox
merlimat closed pull request #1579: Added option to create namespace on CLI passing list of clusters URL: https://github.com/apache/incubator-pulsar/pull/1579 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] merlimat closed pull request #1584: On tenants cli tool, default tenant to all clusters if not specified

2018-04-18 Thread GitBox
merlimat closed pull request #1584: On tenants cli tool, default tenant to all clusters if not specified URL: https://github.com/apache/incubator-pulsar/pull/1584 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[incubator-pulsar] branch master updated: Use official bookkeeper release 4.7.0 (#1605)

2018-04-18 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 732105f Use official bookkeeper

[incubator-pulsar] branch master updated: Added option to create namespace on CLI passing list of clusters (#1579)

2018-04-18 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 488942e Added option to create

[GitHub] merlimat closed pull request #1605: Use official bookkeeper release 4.7.0

2018-04-18 Thread GitBox
merlimat closed pull request #1605: Use official bookkeeper release 4.7.0 URL: https://github.com/apache/incubator-pulsar/pull/1605 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 opened a new pull request #1612: Fix pulsar broker handling WebApplicationException

2018-04-18 Thread GitBox
sijie opened a new pull request #1612: Fix pulsar broker handling WebApplicationException URL: https://github.com/apache/incubator-pulsar/pull/1612 *Problem* Currently all WebApplicationExeptions are converted to 500 / InternalServerError by broker before responding to rest

[GitHub] merlimat commented on issue #1586: Added config to deploy pulsar in aws using ebs storage

2018-04-18 Thread GitBox
merlimat commented on issue #1586: Added config to deploy pulsar in aws using ebs storage URL: https://github.com/apache/incubator-pulsar/pull/1586#issuecomment-382520864 @saandrews You might need to rebase/merge with master to get the test fixes

[GitHub] sijie commented on issue #1586: Added config to deploy pulsar in aws using ebs storage

2018-04-18 Thread GitBox
sijie commented on issue #1586: Added config to deploy pulsar in aws using ebs storage URL: https://github.com/apache/incubator-pulsar/pull/1586#issuecomment-382520889 @saandrews you might need to rebase to latest master since cpp tests are flaky (due to #1611)

[GitHub] saandrews commented on issue #1586: Added config to deploy pulsar in aws using ebs storage

2018-04-18 Thread GitBox
saandrews commented on issue #1586: Added config to deploy pulsar in aws using ebs storage URL: https://github.com/apache/incubator-pulsar/pull/1586#issuecomment-382520004 retest this please This is an automated message from

[incubator-pulsar] branch master updated: cpp tests should wait for standalone up (#1611)

2018-04-18 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 e2a585a cpp tests should wait for

[GitHub] sijie closed pull request #1611: cpp tests should wait for standalone up

2018-04-18 Thread GitBox
sijie closed pull request #1611: cpp tests should wait for standalone up URL: https://github.com/apache/incubator-pulsar/pull/1611 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

[incubator-pulsar] branch master updated: Fixed intermittent test BrokerServiceThrottlingTest.testLookupThrottl… (#1598)

2018-04-18 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 974cb78 Fixed intermittent test

[GitHub] merlimat closed pull request #1598: Fixed intermittent test BrokerServiceThrottlingTest.testLookupThrottl…

2018-04-18 Thread GitBox
merlimat closed pull request #1598: Fixed intermittent test BrokerServiceThrottlingTest.testLookupThrottl… URL: https://github.com/apache/incubator-pulsar/pull/1598 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] sijie opened a new pull request #1611: cpp tests should wait for standalone up

2018-04-18 Thread GitBox
sijie opened a new pull request #1611: cpp tests should wait for standalone up URL: https://github.com/apache/incubator-pulsar/pull/1611 ### Motivation cpp tests are flaky and failing because tenants are not created. Tenants are not created because standalone is not up. ###

[GitHub] merlimat closed pull request #1609: Fix fail in BrokerServiceTest#testLookupThrottlingForClientByClient

2018-04-18 Thread GitBox
merlimat closed pull request #1609: Fix fail in BrokerServiceTest#testLookupThrottlingForClientByClient URL: https://github.com/apache/incubator-pulsar/pull/1609 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[incubator-pulsar] branch master updated: fix name space (#1609)

2018-04-18 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 828a8c0 fix name space (#1609)

[GitHub] zhaijack commented on issue #1609: Fix fail in BrokerServiceTest#testLookupThrottlingForClientByClient

2018-04-18 Thread GitBox
zhaijack commented on issue #1609: Fix fail in BrokerServiceTest#testLookupThrottlingForClientByClient URL: https://github.com/apache/incubator-pulsar/pull/1609#issuecomment-382465069 retest this please for org.apache.pulsar.broker.service.v1.V1_ReplicatorTest.testReplication

[GitHub] ivankelly commented on issue #1604: Compression support for compaction

2018-04-18 Thread GitBox
ivankelly commented on issue #1604: Compression support for compaction URL: https://github.com/apache/incubator-pulsar/pull/1604#issuecomment-382457687 retest this please // BrokerServiceTest.testLookupThrottlingForClientByClient & c++ BasicEndToEndTest

[GitHub] saandrews commented on issue #1586: Added config to deploy pulsar in aws using ebs storage

2018-04-18 Thread GitBox
saandrews commented on issue #1586: Added config to deploy pulsar in aws using ebs storage URL: https://github.com/apache/incubator-pulsar/pull/1586#issuecomment-382442708 retest this please This is an automated message from

[GitHub] zhaijack commented on issue #1593: PIP-17: the part of index block for offload.

2018-04-18 Thread GitBox
zhaijack commented on issue #1593: PIP-17: the part of index block for offload. URL: https://github.com/apache/incubator-pulsar/pull/1593#issuecomment-382442101 Thanks @ivankelly for the comments, will change it and add the logic for reading back.

[GitHub] ivankelly commented on issue #1604: Compression support for compaction

2018-04-18 Thread GitBox
ivankelly commented on issue #1604: Compression support for compaction URL: https://github.com/apache/incubator-pulsar/pull/1604#issuecomment-382428123 retest this please // BrokerServiceTest.testLookupThrottlingForClientByClient

[GitHub] ivankelly commented on issue #1549: offloadPrefix implementation for managed ledger

2018-04-18 Thread GitBox
ivankelly commented on issue #1549: offloadPrefix implementation for managed ledger URL: https://github.com/apache/incubator-pulsar/pull/1549#issuecomment-382427995 retest this please // BrokerServiceTest.testLookupThrottlingForClientByClient

[GitHub] srkukarni commented on issue #1609: Fix fail in BrokerServiceTest#testLookupThrottlingForClientByClient

2018-04-18 Thread GitBox
srkukarni commented on issue #1609: Fix fail in BrokerServiceTest#testLookupThrottlingForClientByClient URL: https://github.com/apache/incubator-pulsar/pull/1609#issuecomment-382400498 retest this please This is an automated

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

2018-04-18 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/incubator-pulsar.git The following commit(s) were added to refs/heads/asf-site by this push: new af04ba8 Updated site at revision