[GitHub] asfgit commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group

2017-11-14 Thread GitBox
asfgit commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group URL: https://github.com/apache/incubator-pulsar/pull/896#issuecomment-344430368 FAILURE --none-- This is an automated message

[GitHub] rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group

2017-11-14 Thread GitBox
rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group URL: https://github.com/apache/incubator-pulsar/pull/896#issuecomment-344415329 retest this please This is an automated message

[GitHub] asfgit commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group

2017-11-14 Thread GitBox
asfgit commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group URL: https://github.com/apache/incubator-pulsar/pull/896#issuecomment-344375080 FAILURE --none-- This is an automated message

[GitHub] asfgit commented on issue #902: Set error-message into RestException

2017-11-14 Thread GitBox
asfgit commented on issue #902: Set error-message into RestException URL: https://github.com/apache/incubator-pulsar/pull/902#issuecomment-344389006 SUCCESS --none-- This is an automated message from the Apache Git

[GitHub] rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group

2017-11-14 Thread GitBox
rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group URL: https://github.com/apache/incubator-pulsar/pull/896#issuecomment-344415329 retest this please This is an automated message

[GitHub] rdhabalia closed pull request #902: Set error-message into RestException

2017-11-14 Thread GitBox
rdhabalia closed pull request #902: Set error-message into RestException URL: https://github.com/apache/incubator-pulsar/pull/902 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] asfgit commented on issue #901: Introduce configuration converter

2017-11-14 Thread GitBox
asfgit commented on issue #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#issuecomment-344383612 SUCCESS --none-- This is an automated message from the Apache Git

[GitHub] rdhabalia commented on a change in pull request #901: Introduce configuration converter

2017-11-14 Thread GitBox
rdhabalia commented on a change in pull request #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#discussion_r150950022 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ##

[GitHub] asfgit commented on issue #901: Introduce configuration converter

2017-11-14 Thread GitBox
asfgit commented on issue #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#issuecomment-344383612 SUCCESS --none-- This is an automated message from the Apache Git

[GitHub] rdhabalia commented on a change in pull request #901: Introduce configuration converter

2017-11-14 Thread GitBox
rdhabalia commented on a change in pull request #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#discussion_r150948924 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ##

[GitHub] yush1ga commented on a change in pull request #901: Introduce configuration converter

2017-11-14 Thread GitBox
yush1ga commented on a change in pull request #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#discussion_r150965286 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ## @@

[GitHub] yush1ga commented on a change in pull request #901: Introduce configuration converter

2017-11-14 Thread GitBox
yush1ga commented on a change in pull request #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#discussion_r150964865 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ## @@

[GitHub] asfgit commented on issue #901: Introduce configuration converter

2017-11-14 Thread GitBox
asfgit commented on issue #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#issuecomment-344417331 SUCCESS --none-- This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #903: PIP-8: Introduce peer cluster for global namespace redirection

2017-11-14 Thread GitBox
asfgit commented on issue #903: PIP-8: Introduce peer cluster for global namespace redirection URL: https://github.com/apache/incubator-pulsar/pull/903#issuecomment-344472454 SUCCESS --none-- This is an automated

[GitHub] rdhabalia opened a new pull request #903: PIP-8: Introduce peer cluster for global namespace redirection

2017-11-14 Thread GitBox
rdhabalia opened a new pull request #903: PIP-8: Introduce peer cluster for global namespace redirection URL: https://github.com/apache/incubator-pulsar/pull/903 ### Motivation As described in [PIP-8](https://github.com/apache/incubator-pulsar/wiki/PIP-8-Pulsar-beyond-1M-topics),

[GitHub] rdhabalia commented on a change in pull request #901: Introduce configuration converter

2017-11-14 Thread GitBox
rdhabalia commented on a change in pull request #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#discussion_r151037495 ## File path:

[GitHub] asfgit commented on issue #901: Introduce configuration converter

2017-11-14 Thread GitBox
asfgit commented on issue #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901#issuecomment-344417331 SUCCESS --none-- This is an automated message from the Apache Git

[GitHub] yush1ga opened a new pull request #901: Introduce configuration converter

2017-11-14 Thread GitBox
yush1ga opened a new pull request #901: Introduce configuration converter URL: https://github.com/apache/incubator-pulsar/pull/901 ### Motivation Although `DiscoveryService`, `WebSocketProxy` and `PulsarProxy` have original configuration classes, these components convert these to

[GitHub] rdhabalia opened a new pull request #902: Set error-message into RestException

2017-11-14 Thread GitBox
rdhabalia opened a new pull request #902: Set error-message into RestException URL: https://github.com/apache/incubator-pulsar/pull/902 ### Motivation Right now, `RestException(code, message)` doesn't set given message into the exception and `restException.getMessage()` doesn't

[GitHub] yush1ga closed pull request #898: E2e in perf-consumer

2017-11-14 Thread GitBox
yush1ga closed pull request #898: E2e in perf-consumer URL: https://github.com/apache/incubator-pulsar/pull/898 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] asfgit commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group

2017-11-14 Thread GitBox
asfgit commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group URL: https://github.com/apache/incubator-pulsar/pull/896#issuecomment-344375080 FAILURE --none-- This is an automated message

[GitHub] rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group

2017-11-14 Thread GitBox
rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group URL: https://github.com/apache/incubator-pulsar/pull/896#issuecomment-344360053 retest this please This is an automated message

[GitHub] rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group

2017-11-14 Thread GitBox
rdhabalia commented on issue #896: Introduce Failure-domain and Anti-affinity-namespace group URL: https://github.com/apache/incubator-pulsar/pull/896#issuecomment-344360053 retest this please This is an automated message