[GitHub] sijie opened a new pull request #1223: Add a `backend` admin restful endpoint for query backend information

sijie opened a new pull request #1223: Add a `backend` admin restful endpoint for query backend information URL: https://github.com/apache/incubator-pulsar/pull/1223 ### Motivation Expose `backend` information through admin restful endpoint. so other components can use it to get the

[GitHub] XiaoZYang commented on a change in pull request #1219: Xzy initialize subscription config

XiaoZYang commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167464774 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedger.java ## @@ -138

[GitHub] XiaoZYang commented on a change in pull request #1219: Xzy initialize subscription config

XiaoZYang commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167464388 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/PersistentMessageFinderTest.java ###

[GitHub] XiaoZYang commented on a change in pull request #1219: Xzy initialize subscription config

XiaoZYang commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167463881 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedger.java ## @@ -138

[GitHub] zhaijack commented on a change in pull request #1219: Xzy initialize subscription config

zhaijack commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167462456 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/PersistentMessageFinderTest.java

[GitHub] zhaijack commented on a change in pull request #1219: Xzy initialize subscription config

zhaijack commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167461954 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ##

[GitHub] zhaijack commented on a change in pull request #1219: Xzy initialize subscription config

zhaijack commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167461546 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedger.java ## @@ -138,

[GitHub] zhaijack commented on a change in pull request #1219: Xzy initialize subscription config

zhaijack commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167461546 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedger.java ## @@ -138,

[GitHub] zhaijack commented on a change in pull request #1066: Issue 937: add CommandGetLastMessageId to make reader know the end of topic

zhaijack commented on a change in pull request #1066: Issue 937: add CommandGetLastMessageId to make reader know the end of topic URL: https://github.com/apache/incubator-pulsar/pull/1066#discussion_r167457406 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/

[GitHub] maskit closed pull request #1216: Use SecurityUtility (part 2)

maskit closed pull request #1216: Use SecurityUtility (part 2) URL: https://github.com/apache/incubator-pulsar/pull/1216 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 re

[incubator-pulsar] branch master updated: Use SecurityUtility (part 2) (#1216)

This is an automated email from the ASF dual-hosted git repository. maskit 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 d38cf24 Use SecurityUtility (part 2)

[GitHub] merlimat closed pull request #1222: Remove org.apache.pulsar.discovery.service.PulsarServerException

merlimat closed pull request #1222: Remove org.apache.pulsar.discovery.service.PulsarServerException URL: https://github.com/apache/incubator-pulsar/pull/1222 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 pr

[incubator-pulsar] branch master updated: Remove org.apache.pulsar.discovery.service.PulsarServerException (#1222)

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 3fd3ab4 Remove org.apache.pulsar.dis

[GitHub] merlimat closed pull request #1221: Remove org.apache.pulsar.proxy.socket.client.MockAuthenticationProvider

merlimat closed pull request #1221: Remove org.apache.pulsar.proxy.socket.client.MockAuthenticationProvider URL: https://github.com/apache/incubator-pulsar/pull/1221 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sak

[incubator-pulsar] branch master updated: Remove org.apache.pulsar.proxy.socket.client.MockAuthenticationProvider (#1221)

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 8e5b11a Remove org.apache.pulsar.pro

[incubator-pulsar] branch master updated: Remove org.apache.pulsar.proxy.authentication.AuthenticationServiceTest (#1220)

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 3bfc505 Remove org.apache.pulsar.pro

[GitHub] merlimat closed pull request #1220: Remove org.apache.pulsar.proxy.authentication.AuthenticationServiceTest

merlimat closed pull request #1220: Remove org.apache.pulsar.proxy.authentication.AuthenticationServiceTest URL: https://github.com/apache/incubator-pulsar/pull/1220 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sak

[GitHub] merlimat closed pull request #1217: Remove org.apache.pulsar.proxy.socket.client.PaddingDecimalFormat

merlimat closed pull request #1217: Remove org.apache.pulsar.proxy.socket.client.PaddingDecimalFormat URL: https://github.com/apache/incubator-pulsar/pull/1217 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 p

[incubator-pulsar] branch master updated: Remove org.apache.pulsar.proxy.socket.client.PaddingDecimalFormat (#1217)

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 71b3b18 Remove org.apache.pulsar.pro

[incubator-pulsar] branch master updated: Support hostname verification on proxy to broker connection (#1214)

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 a27a1e2 Support hostname verificat

[GitHub] rdhabalia closed pull request #1214: Support hostname verification on proxy to broker connection

rdhabalia closed pull request #1214: Support hostname verification on proxy to broker connection URL: https://github.com/apache/incubator-pulsar/pull/1214 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 proven

[GitHub] maskit commented on issue #1222: Remove org.apache.pulsar.discovery.service.PulsarServerException

maskit commented on issue #1222: Remove org.apache.pulsar.discovery.service.PulsarServerException URL: https://github.com/apache/incubator-pulsar/pull/1222#issuecomment-364753054 retest this please This is an automated messag

[GitHub] maskit opened a new pull request #1222: Remove org.apache.pulsar.discovery.service.PulsarServerException

maskit opened a new pull request #1222: Remove org.apache.pulsar.discovery.service.PulsarServerException URL: https://github.com/apache/incubator-pulsar/pull/1222 ### Motivation There are two `PulsarServerException` but there's no differences between them, and I don't see any reason

[GitHub] maskit opened a new pull request #1221: Remove org.apache.pulsar.proxy.socket.client.MockAuthenticationProvider

maskit opened a new pull request #1221: Remove org.apache.pulsar.proxy.socket.client.MockAuthenticationProvider URL: https://github.com/apache/incubator-pulsar/pull/1221 ### Motivation Nobody use it, and there are lot of copies. ### Modifications Remove `org.apache.puls

[GitHub] XiaoZYang commented on issue #1069: Provide a setting in consumer configuration to specify where to start consuming messages

XiaoZYang commented on issue #1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/issues/1069#issuecomment-364734900 @sijie @merlimat please review [this PR](https://github.com/apache/incubator-pul

[GitHub] maskit opened a new pull request #1220: Remove org.apache.pulsar.proxy.authentication.AuthenticationServiceTest

maskit opened a new pull request #1220: Remove org.apache.pulsar.proxy.authentication.AuthenticationServiceTest URL: https://github.com/apache/incubator-pulsar/pull/1220 ### Motivation `org.apache.pulsar.proxy.authentication.AuthenticationServiceTest` and `org.apache.pulsar.broker.a

[GitHub] XiaoZYang opened a new pull request #1219: Xzy initialize subscription config

XiaoZYang opened a new pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219 ### Motivation This is a fix try for [Issue 1069](https://github.com/apache/incubator-pulsar/issues/1069), which expects a setting in consumer conf

[GitHub] XiaoZYang closed pull request #1218: set initial cursor position when subscribe

XiaoZYang closed pull request #1218: set initial cursor position when subscribe URL: https://github.com/apache/incubator-pulsar/pull/1218 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] XiaoZYang commented on issue #1069: Provide a setting in consumer configuration to specify where to start consuming messages

XiaoZYang commented on issue #1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/issues/1069#issuecomment-364734900 @sijie @merlimat please review [this PR](https://github.com/apache/incubator-pul

[GitHub] XiaoZYang opened a new pull request #1218: set initial cursor position when subscribe

XiaoZYang opened a new pull request #1218: set initial cursor position when subscribe URL: https://github.com/apache/incubator-pulsar/pull/1218 ### Motivation This is a fix try for [Issue 1069](https://github.com/apache/incubator-pulsar/issues/1069), which expects a setting in cons

[GitHub] maskit opened a new pull request #1217: Remove org.apache.pulsar.proxy.socket.client.PaddingDecimalFormat

maskit opened a new pull request #1217: Remove org.apache.pulsar.proxy.socket.client.PaddingDecimalFormat URL: https://github.com/apache/incubator-pulsar/pull/1217 ### Motivation `org.apache.pulsar.proxy.socket.client.PaddingDecimalFormat` is exactly the same as the one in `org.apac