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

2018-07-07 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 407e1ab Updated site at revision

[GitHub] jerrypeng commented on issue #2098: Add context to source and sink

2018-07-07 Thread GitBox
jerrypeng commented on issue #2098: Add context to source and sink URL: https://github.com/apache/incubator-pulsar/pull/2098#issuecomment-403191586 retest this please This is an automated message from the Apache Git Service.

[GitHub] sijie commented on issue #2100: Issue 2077: SubscriptionInitialPosition doesn't work with multiple-topic type subscription

2018-07-07 Thread GitBox
sijie commented on issue #2100: Issue 2077: SubscriptionInitialPosition doesn't work with multiple-topic type subscription URL: https://github.com/apache/incubator-pulsar/pull/2100#issuecomment-403236129 retest this please

[incubator-pulsar] branch master updated: Issue 2077: SubscriptionInitialPosition doesn't work with multiple-topic type subscription (#2100)

2018-07-07 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 ffe7b62 Issue 2077:

[GitHub] sijie closed issue #2077: SubscriptionInitialPosition doesn't work with multiple-topic type subscription

2018-07-07 Thread GitBox
sijie closed issue #2077: SubscriptionInitialPosition doesn't work with multiple-topic type subscription URL: https://github.com/apache/incubator-pulsar/issues/2077 This is an automated message from the Apache Git Service.

[GitHub] sijie closed issue #2090: Links to connector source code are broken

2018-07-07 Thread GitBox
sijie closed issue #2090: Links to connector source code are broken URL: https://github.com/apache/incubator-pulsar/issues/2090 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-pulsar] branch master updated: Issue 2090: (#2092)

2018-07-07 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 027f535 Issue 2090: (#2092) 027f535

[GitHub] jerrypeng closed pull request #2098: Add context to source and sink

2018-07-07 Thread GitBox
jerrypeng closed pull request #2098: Add context to source and sink URL: https://github.com/apache/incubator-pulsar/pull/2098 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

[incubator-pulsar] branch master updated: Add context to source and sink (#2098)

2018-07-07 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 423a9af Add context to source and

[GitHub] sijie commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar

2018-07-07 Thread GitBox
sijie commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar URL: https://github.com/apache/incubator-pulsar/pull/2080#issuecomment-403238392 @jerrypeng @merlimat please review this This

[GitHub] sijie closed pull request #2100: Issue 2077: SubscriptionInitialPosition doesn't work with multiple-topic type subscription

2018-07-07 Thread GitBox
sijie closed pull request #2100: Issue 2077: SubscriptionInitialPosition doesn't work with multiple-topic type subscription URL: https://github.com/apache/incubator-pulsar/pull/2100 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] sijie commented on issue #2092: Issue 2090: Links to connector source code are broken

2018-07-07 Thread GitBox
sijie commented on issue #2092: Issue 2090: Links to connector source code are broken URL: https://github.com/apache/incubator-pulsar/pull/2092#issuecomment-403238351 retest this please This is an automated message from the

[GitHub] sijie commented on issue #2051: Move Security related documentation to a top-level category

2018-07-07 Thread GitBox
sijie commented on issue #2051: Move Security related documentation to a top-level category URL: https://github.com/apache/incubator-pulsar/pull/2051#issuecomment-403242657 retest this please This is an automated message

[GitHub] sijie commented on issue #2107: patching CmdSink and adding extensive unit tests

2018-07-07 Thread GitBox
sijie commented on issue #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107#issuecomment-403242930 retest this please This is an automated message from the Apache

[GitHub] gordeevbr commented on issue #2025: [WiP] Allow non-pesistent topics to be retrieved along with persistent ones from the "GetTopicsOfNamespace" method

2018-07-07 Thread GitBox
gordeevbr commented on issue #2025: [WiP] Allow non-pesistent topics to be retrieved along with persistent ones from the "GetTopicsOfNamespace" method URL: https://github.com/apache/incubator-pulsar/pull/2025#issuecomment-403250526 @merlimat @jiazhai I have updated this feature as you

[GitHub] aahmed-se commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar

2018-07-07 Thread GitBox
aahmed-se commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar URL: https://github.com/apache/incubator-pulsar/pull/2080#issuecomment-403252754 retest this please This is an automated

[GitHub] jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests

2018-07-07 Thread GitBox
jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107#issuecomment-403260731 retest this please This is an automated message from the

[GitHub] sijie opened a new pull request #2109: Remove popover from security overview page

2018-07-07 Thread GitBox
sijie opened a new pull request #2109: Remove popover from security overview page URL: https://github.com/apache/incubator-pulsar/pull/2109 *Motivation* Currently website uses a customized plugin for popover. If you don't define the popover, it will fail the build. It becomes a

[GitHub] sijie closed pull request #2051: Move Security related documentation to a top-level category

2018-07-07 Thread GitBox
sijie closed pull request #2051: Move Security related documentation to a top-level category URL: https://github.com/apache/incubator-pulsar/pull/2051 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

[incubator-pulsar] branch master updated: Move Security related documentation to a top-level category (#2051)

2018-07-07 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 5402d21 Move Security related

[GitHub] merlimat commented on issue #2106: Provide API and CLI to access list of connectors supported by system

2018-07-07 Thread GitBox
merlimat commented on issue #2106: Provide API and CLI to access list of connectors supported by system URL: https://github.com/apache/incubator-pulsar/pull/2106#issuecomment-403247981 Rebased on master This is an automated

[GitHub] jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests

2018-07-07 Thread GitBox
jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107#issuecomment-403258008 retest this please This is an automated message from the

[GitHub] sijie commented on a change in pull request #2107: patching CmdSink and adding extensive unit tests

2018-07-07 Thread GitBox
sijie commented on a change in pull request #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107#discussion_r200828739 ## File path: pulsar-client-tools/pom.xml ## @@ -86,6 +86,40 @@ + + +

[GitHub] sijie commented on issue #2109: Remove popover from security overview page

2018-07-07 Thread GitBox
sijie commented on issue #2109: Remove popover from security overview page URL: https://github.com/apache/incubator-pulsar/pull/2109#issuecomment-403261207 /cc @cckellogg since he knows how tricky to use this popover stuffs

[incubator-pulsar] branch master updated: Load connectors from NAR archives (#2102)

2018-07-07 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 2bf2568 Load connectors from NAR

[GitHub] merlimat closed pull request #2102: Load connectors from NAR archives

2018-07-07 Thread GitBox
merlimat closed pull request #2102: Load connectors from NAR archives URL: https://github.com/apache/incubator-pulsar/pull/2102 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

[GitHub] jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests

2018-07-07 Thread GitBox
jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107#issuecomment-403253997 retest this please This is an automated message from the

[GitHub] rdhabalia opened a new pull request #2108: add flatbuffer option to serialize kinesis-message

2018-07-07 Thread GitBox
rdhabalia opened a new pull request #2108: add flatbuffer option to serialize kinesis-message URL: https://github.com/apache/incubator-pulsar/pull/2108 ### Motivation with #2079, supports publishing kinesis message from KinesisSink in multiple format: raw_body, json. adding support

[GitHub] rdhabalia commented on issue #1462: Delayed acks impl

2018-07-07 Thread GitBox
rdhabalia commented on issue #1462: Delayed acks impl URL: https://github.com/apache/incubator-pulsar/pull/1462#issuecomment-403256971 @merlimat this is not a backward compatible change with old broker (version < 2.0) because, old brokers don't understand group-ack command so, combination

[GitHub] rdhabalia commented on issue #1462: Delayed acks impl

2018-07-07 Thread GitBox
rdhabalia commented on issue #1462: Delayed acks impl URL: https://github.com/apache/incubator-pulsar/pull/1462#issuecomment-403257185 can we add `CommandAckDeprecated` in proto-schema which can be used by client if broker-verison is < `V12` ?

[GitHub] jerrypeng commented on a change in pull request #2107: patching CmdSink and adding extensive unit tests

2018-07-07 Thread GitBox
jerrypeng commented on a change in pull request #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107#discussion_r200828766 ## File path: pulsar-client-tools/pom.xml ## @@ -86,6 +86,40 @@ + + +

[GitHub] jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests

2018-07-07 Thread GitBox
jerrypeng commented on issue #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107#issuecomment-403263846 retest this please This is an automated message from the