[GitHub] zhaijack commented on issue #1996: Cpp client: add multiTopicsConsumer

2018-07-20 Thread GitBox
zhaijack commented on issue #1996: Cpp client: add multiTopicsConsumer URL: https://github.com/apache/incubator-pulsar/pull/1996#issuecomment-406772365 retest this please This is an automated message from the Apache Git

[GitHub] aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406767098 retest this please This is an automated message from the Apache Git

[GitHub] aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406749962 retest this please This is an automated message from the Apache Git

[GitHub] aahmed-se commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406747390 review is ready , there are random failure probably due to resource contention.

[GitHub] aahmed-se commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406737559 retest this please This is an automated message from the

[GitHub] rdhabalia opened a new pull request #2214: Add tls support to authenticate client to access function admin-api

2018-07-20 Thread GitBox
rdhabalia opened a new pull request #2214: Add tls support to authenticate client to access function admin-api URL: https://github.com/apache/incubator-pulsar/pull/2214 ### Motivation Add TLS support on function webservice so, function-authentication in #2213 can leverage TLS

[GitHub] sijie commented on issue #2206: Pulsar website using docusaurus

2018-07-20 Thread GitBox
sijie commented on issue #2206: Pulsar website using docusaurus URL: https://github.com/apache/incubator-pulsar/pull/2206#issuecomment-406737134 @merlimat I think we can merge this when it passed CI. since all the changes are under `site2`. we can put the built html files under

[GitHub] rdhabalia opened a new pull request #2213: Add authorization support on function apis

2018-07-20 Thread GitBox
rdhabalia opened a new pull request #2213: Add authorization support on function apis URL: https://github.com/apache/incubator-pulsar/pull/2213 ### Motivation Function admin api should capability to perform authorization before creating/modifying function admin-api. ###

[GitHub] sijie opened a new pull request #2212: Pull docker images before running integration tests

2018-07-20 Thread GitBox
sijie opened a new pull request #2212: Pull docker images before running integration tests URL: https://github.com/apache/incubator-pulsar/pull/2212 *Motivation* Integration tests sometime failed with "No such image:". *Changes* This change attempts to pull docker

[GitHub] merlimat commented on issue #2192: Rename Netty native libs to reflect shading prefix

2018-07-20 Thread GitBox
merlimat commented on issue #2192: Rename Netty native libs to reflect shading prefix URL: https://github.com/apache/incubator-pulsar/pull/2192#issuecomment-406729706 retest this please This is an automated message from the

[GitHub] aahmed-se commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406725444 retest this please This is an automated message from the

[GitHub] sijie commented on issue #2209: Exclude helm files from Apache Rat Check.

2018-07-20 Thread GitBox
sijie commented on issue #2209: Exclude helm files from Apache Rat Check. URL: https://github.com/apache/incubator-pulsar/pull/2209#issuecomment-406719222 retest this please This is an automated message from the Apache Git

[GitHub] sijie commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
sijie commented on issue #2180: [WIP] Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406718933 >that's defeats of the purpose of the PR, h I don't think this defeats the purpose of the PR. I have seen a general problem

[GitHub] aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406701670 making this wip till I figure out the jar issues. This is an

[GitHub] aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406693811 that's defeats of the purpose of the PR, here is the new error I will spend some time to try and fix it. ```

[GitHub] dsambandam commented on issue #2195: Can't start Global Zookeeper in a Multi-cluster setup

2018-07-20 Thread GitBox
dsambandam commented on issue #2195: Can't start Global Zookeeper in a Multi-cluster setup URL: https://github.com/apache/incubator-pulsar/issues/2195#issuecomment-406656844 Thanks @merlimat I shall try this out . This is

[GitHub] merlimat commented on issue #2195: Can't start Global Zookeeper in a Multi-cluster setup

2018-07-20 Thread GitBox
merlimat commented on issue #2195: Can't start Global Zookeeper in a Multi-cluster setup URL: https://github.com/apache/incubator-pulsar/issues/2195#issuecomment-406656250 @dsambandam We currently don't have a ready-made command to do that. The procedure would look like: 1.

[GitHub] zhaijack commented on issue #2204: in cpp client, NamespaceName not support V2 type

2018-07-20 Thread GitBox
zhaijack commented on issue #2204: in cpp client, NamespaceName not support V2 type URL: https://github.com/apache/incubator-pulsar/issues/2204#issuecomment-406596716 #2205 merged This is an automated message from the

[GitHub] zhaijack closed issue #2204: in cpp client, NamespaceName not support V2 type

2018-07-20 Thread GitBox
zhaijack closed issue #2204: in cpp client, NamespaceName not support V2 type URL: https://github.com/apache/incubator-pulsar/issues/2204 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] jiazhai commented on issue #2208: Fix License Issue

2018-07-20 Thread GitBox
jiazhai commented on issue #2208: Fix License Issue URL: https://github.com/apache/incubator-pulsar/pull/2208#issuecomment-406596443 @sijie, Thanks for the work. right, it was on latest master. your branch is good. This is

[GitHub] jnioche opened a new issue #2211: Expose redeliverUnacknowledgedMessages(Set messageIds) in the public API

2018-07-20 Thread GitBox
jnioche opened a new issue #2211: Expose redeliverUnacknowledgedMessages(Set messageIds) in the public API URL: https://github.com/apache/incubator-pulsar/issues/2211 See discussion on [slack](https://apache-pulsar.slack.com/archives/C5Z4T36F7/p151820466561) Ideally, I'd love

[GitHub] jnioche opened a new issue #2210: PulsarSpout to use consumer.redeliverUnacknowledgedMessages() ?

2018-07-20 Thread GitBox
jnioche opened a new issue #2210: PulsarSpout to use consumer.redeliverUnacknowledgedMessages() ? URL: https://github.com/apache/incubator-pulsar/issues/2210 The PulsarSpout keeps track of the pending messages as well as the failed ones in a queue and map. If the JVM containing a

[GitHub] sijie commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
sijie commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406519492 we don't have to put everything into one module if dependencies management because problematic.

[GitHub] sijie commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
sijie commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406519368 @aahmed-se : a different netty version is coming from datastax client. you can try to exclud it from datastax. if exclusion doesn't work,

[GitHub] zhaijack removed a comment on issue #1996: Cpp client: add multiTopicsConsumer

2018-07-20 Thread GitBox
zhaijack removed a comment on issue #1996: Cpp client: add multiTopicsConsumer URL: https://github.com/apache/incubator-pulsar/pull/1996#issuecomment-406510257 retest this please This is an automated message from the Apache

[GitHub] aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406512397 I have tried it results different method not found issues from netty.

[GitHub] sijie commented on issue #2180: Cleanup Arquillian consolidate projects

2018-07-20 Thread GitBox
sijie commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-406511898 @aahmed-se you need to change pulsar-client to pulsar-client-original. pulsar-client is a shaded version, it will be conflicting

[GitHub] sijie closed pull request #2205: Issue #2204: support v2 namespace in cpp client

2018-07-20 Thread GitBox
sijie closed pull request #2205: Issue #2204: support v2 namespace in cpp client URL: https://github.com/apache/incubator-pulsar/pull/2205 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

[incubator-pulsar] branch master updated: support v2 namespace in cppclient (#2205)

2018-07-20 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 41792f1 support v2 namespace in

[GitHub] zhaijack commented on issue #1996: Cpp client: add multiTopicsConsumer

2018-07-20 Thread GitBox
zhaijack commented on issue #1996: Cpp client: add multiTopicsConsumer URL: https://github.com/apache/incubator-pulsar/pull/1996#issuecomment-406510257 retest this please This is an automated message from the Apache Git

[GitHub] sijie commented on issue #2203: Make sure schema is initialized before the topic is loaded

2018-07-20 Thread GitBox
sijie commented on issue #2203: Make sure schema is initialized before the topic is loaded URL: https://github.com/apache/incubator-pulsar/pull/2203#issuecomment-406509097 retest this please This is an automated message from

[incubator-pulsar] branch master updated: support subscription-type to be passed in sink-function (#2200)

2018-07-20 Thread rdhabalia
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 7ba5c2d support subscription-type

[GitHub] rdhabalia closed pull request #2200: support subscription-type to be passed in sink-function

2018-07-20 Thread GitBox
rdhabalia closed pull request #2200: support subscription-type to be passed in sink-function URL: https://github.com/apache/incubator-pulsar/pull/2200 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] zhaijack commented on issue #2204: in cpp client, NamespaceName not support V2 type

2018-07-20 Thread GitBox
zhaijack commented on issue #2204: in cpp client, NamespaceName not support V2 type URL: https://github.com/apache/incubator-pulsar/issues/2204#issuecomment-406505224 https://github.com/apache/incubator-pulsar/pull/2205 is for this