[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132033540 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/AdminResource.java

[GitHub] merlimat commented on issue #660: Updated download links to Apache mirrors and prepared for multiple ve?

2017-08-08 Thread git
merlimat commented on issue #660: Updated download links to Apache mirrors and prepared for multiple ve? URL: https://github.com/apache/incubator-pulsar/pull/660#issuecomment-321120532 retest this please This is an

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132040863 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/AdminResource.java

[GitHub] merlimat opened a new pull request #660: Updated download links to Apache mirrors and prepared for multiple ve?

2017-08-08 Thread git
merlimat opened a new pull request #660: Updated download links to Apache mirrors and prepared for multiple ve? URL: https://github.com/apache/incubator-pulsar/pull/660 ?rsions docs ### Motivation With the 1.19.0-incubating release we will be storing the artifacts on the ASF

[GitHub] rdhabalia commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
rdhabalia commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132036537 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/AdminResource.java

[GitHub] merlimat closed pull request #632: Splitted Admin API reference into multiple pages

2017-08-08 Thread git
merlimat closed pull request #632: Splitted Admin API reference into multiple pages URL: https://github.com/apache/incubator-pulsar/pull/632 This is an automated message from the Apache Git Service. To respond to the

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132034288 ## File path:

[GitHub] cckellogg opened a new pull request #659: Pulsar connectors

2017-08-08 Thread git
cckellogg opened a new pull request #659: Pulsar connectors URL: https://github.com/apache/incubator-pulsar/pull/659 ### Motivation Start a framework work for building source and sink connectors. ### Modifications Added a base framework for building pulsar source and

[GitHub] rdhabalia commented on a change in pull request #660: Updated download links to Apache mirrors and prepared for multiple ve?

2017-08-08 Thread git
rdhabalia commented on a change in pull request #660: Updated download links to Apache mirrors and prepared for multiple ve? URL: https://github.com/apache/incubator-pulsar/pull/660#discussion_r132068359 ## File path: site/_config.yml ## @@ -25,9 +25,8 @@ pulsar_repo:

[GitHub] merlimat commented on a change in pull request #660: Updated download links to Apache mirrors and prepared for multiple ve?

2017-08-08 Thread git
merlimat commented on a change in pull request #660: Updated download links to Apache mirrors and prepared for multiple ve? URL: https://github.com/apache/incubator-pulsar/pull/660#discussion_r132069835 ## File path: site/_config.yml ## @@ -25,9 +25,8 @@ pulsar_repo:

[GitHub] merlimat commented on issue #658: Use $UID instead of $USER in docker build

2017-08-08 Thread git
merlimat commented on issue #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658#issuecomment-321043000 @rdhabalia Please take a look. I've verified the website now build on Jenkins with this fix.

[GitHub] merlimat commented on a change in pull request #632: Splitted Admin API reference into multiple pages

2017-08-08 Thread git
merlimat commented on a change in pull request #632: Splitted Admin API reference into multiple pages URL: https://github.com/apache/incubator-pulsar/pull/632#discussion_r132001068 ## File path: site/docs/latest/admin-api/partitioned-topics.md ## @@ -1,9 +1,15 @@ +---

[GitHub] merlimat commented on a change in pull request #632: Splitted Admin API reference into multiple pages

2017-08-08 Thread git
merlimat commented on a change in pull request #632: Splitted Admin API reference into multiple pages URL: https://github.com/apache/incubator-pulsar/pull/632#discussion_r132001213 ## File path: site/docs/latest/admin-api/permissions.md ## @@ -1,155 +1,6 @@ -Pulsar {%

[GitHub] merlimat commented on a change in pull request #632: Splitted Admin API reference into multiple pages

2017-08-08 Thread git
merlimat commented on a change in pull request #632: Splitted Admin API reference into multiple pages URL: https://github.com/apache/incubator-pulsar/pull/632#discussion_r132002230 ## File path: site/docs/latest/admin-api/permissions.md ## @@ -1,155 +1,6 @@ -Pulsar {%

[GitHub] merlimat commented on issue #658: Use $UID instead of $USER in docker build

2017-08-08 Thread git
merlimat commented on issue #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658#issuecomment-321050797 retest this please This is an automated message from the Apache Git

[GitHub] merlimat commented on issue #658: Use $UID instead of $USER in docker build

2017-08-08 Thread git
merlimat commented on issue #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658#issuecomment-321050797 retest this please This is an automated message from the Apache Git

[GitHub] merlimat opened a new pull request #658: Use $UID instead of $USER in docker build

2017-08-08 Thread git
merlimat opened a new pull request #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658 ### Motivation In #652, there was an attempt to fix the website build by using current user in Docker. It doesn't work in Jenkins and the

[GitHub] rdhabalia commented on a change in pull request #658: Use $UID instead of $USER in docker build

2017-08-08 Thread git
rdhabalia commented on a change in pull request #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658#discussion_r131995895 ## File path: site/scripts/docker-build.sh ## @@ -35,6 +35,9 @@ echo " Build Pulsar website

[GitHub] merlimat closed pull request #641: Added missing copyright notice for Circe library

2017-08-08 Thread git
merlimat closed pull request #641: Added missing copyright notice for Circe library URL: https://github.com/apache/incubator-pulsar/pull/641 This is an automated message from the Apache Git Service. To respond to the

[GitHub] merlimat opened a new issue #657: Intermittent C++ test failure BatchMessageTest.testProducerConfig

2017-08-08 Thread git
merlimat opened a new issue #657: Intermittent C++ test failure BatchMessageTest.testProducerConfig URL: https://github.com/apache/incubator-pulsar/issues/657 As seen in https://builds.apache.org/job/pulsar-master/22/console ``` [17/75] BatchMessageTest.testProducerConfig (30 ms)

[GitHub] merlimat closed pull request #654: Do not delete inactive topics if they have retention policy

2017-08-08 Thread git
merlimat closed pull request #654: Do not delete inactive topics if they have retention policy URL: https://github.com/apache/incubator-pulsar/pull/654 This is an automated message from the Apache Git Service. To respond

[GitHub] merlimat commented on issue #658: Use $UID instead of $USER in docker build

2017-08-08 Thread git
merlimat commented on issue #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658#issuecomment-321064613 retest this please This is an automated message from the Apache Git

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r131990002 ## File path: conf/broker.conf ## @@ -106,6 +106,14 @@ maxUnackedMessagesPerBroker=0 #

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132003259 ## File path:

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r131996619 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/Consumer.java ##

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r131992335 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/Namespaces.java ##

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r131991850 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/AdminResource.java

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132003921 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/DispatchRate.java

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132004308 ## File path:

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132003620 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/DispatchRate.java

[GitHub] merlimat commented on issue #632: Splitted Admin API reference into multiple pages

2017-08-08 Thread git
merlimat commented on issue #632: Splitted Admin API reference into multiple pages URL: https://github.com/apache/incubator-pulsar/pull/632#issuecomment-321056047 @rdhabalia updated This is an automated message from the

[GitHub] merlimat closed pull request #658: Use $UID instead of $USER in docker build

2017-08-08 Thread git
merlimat closed pull request #658: Use $UID instead of $USER in docker build URL: https://github.com/apache/incubator-pulsar/pull/658 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rdhabalia commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
rdhabalia commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132026790 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/RateLimiter.java

[GitHub] merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
merlimat commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132027792 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/RateLimiter.java ##

[GitHub] rdhabalia commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting

2017-08-08 Thread git
rdhabalia commented on a change in pull request #634: PIP-3 : Introduce message-dispatch rate limiting URL: https://github.com/apache/incubator-pulsar/pull/634#discussion_r132031203 ## File path: