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

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

[GitHub] rdhabalia closed pull request #2213: Add authorization support on function apis

2018-07-23 Thread GitBox
rdhabalia closed pull request #2213: Add authorization support on function apis URL: https://github.com/apache/incubator-pulsar/pull/2213 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

[incubator-pulsar] branch master updated: Add authorization support on function apis (#2213)

2018-07-23 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 ce6fe8b Add authorization support

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

2018-07-23 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-407265088 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-23 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-407247003 retest this please This is an automated message from the Apache Git

[GitHub] rdhabalia commented on issue #2213: Add authorization support on function apis

2018-07-23 Thread GitBox
rdhabalia commented on issue #2213: Add authorization support on function apis URL: https://github.com/apache/incubator-pulsar/pull/2213#issuecomment-407244184 retest this please This is an automated message from the Apache

[GitHub] rdhabalia removed a comment on issue #2213: Add authorization support on function apis

2018-07-23 Thread GitBox
rdhabalia removed a comment on issue #2213: Add authorization support on function apis URL: https://github.com/apache/incubator-pulsar/pull/2213#issuecomment-407244184 retest this please This is an automated message from the

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

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

[GitHub] rdhabalia removed a comment on issue #2213: Add authorization support on function apis

2018-07-23 Thread GitBox
rdhabalia removed a comment on issue #2213: Add authorization support on function apis URL: https://github.com/apache/incubator-pulsar/pull/2213#issuecomment-407230595 retest this please This is an automated message from the

[GitHub] rdhabalia commented on issue #2213: Add authorization support on function apis

2018-07-23 Thread GitBox
rdhabalia commented on issue #2213: Add authorization support on function apis URL: https://github.com/apache/incubator-pulsar/pull/2213#issuecomment-407230595 retest this please This is an automated message from the Apache

[GitHub] aahmed-se edited a comment on issue #2180: Cleanup Arquillian consolidate projects

2018-07-23 Thread GitBox
aahmed-se edited a comment on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-407226704 Can we merge this, the tests have passed , I will push another PR later.

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

2018-07-23 Thread GitBox
aahmed-se commented on issue #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#issuecomment-407226704 Can we merge this the test have passed I will push another PR later.

[GitHub] srkukarni commented on issue #2216: function status list json returns strings for counts and timestamp

2018-07-23 Thread GitBox
srkukarni commented on issue #2216: function status list json returns strings for counts and timestamp URL: https://github.com/apache/incubator-pulsar/issues/2216#issuecomment-407211442 We are using JsonFormat class provided in the google protobuf library to convert the an internal

[GitHub] sijie commented on issue #2189: dumpContainerDirToTargetCompressed should copy files to a containerName named directory

2018-07-23 Thread GitBox
sijie commented on issue #2189: dumpContainerDirToTargetCompressed should copy files to a containerName named directory URL: https://github.com/apache/incubator-pulsar/pull/2189#issuecomment-407208300 retest this please This

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

2018-07-23 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-407208059 retest this please This is an automated message from

[GitHub] sijie commented on issue #2212: Pull docker images before running integration tests

2018-07-23 Thread GitBox
sijie commented on issue #2212: Pull docker images before running integration tests URL: https://github.com/apache/incubator-pulsar/pull/2212#issuecomment-407207803 retest this please This is an automated message from the

[GitHub] sijie commented on a change in pull request #2214: Add tls support to authenticate client to access function admin-api

2018-07-23 Thread GitBox
sijie commented on a change in pull request #2214: Add tls support to authenticate client to access function admin-api URL: https://github.com/apache/incubator-pulsar/pull/2214#discussion_r204560304 ## File path: conf/functions_worker.yml ## @@ -20,6 +20,7 @@ workerId:

[GitHub] sijie commented on issue #2152: GCS offload support(4): add documentations for GCS

2018-07-23 Thread GitBox
sijie commented on issue #2152: GCS offload support(4): add documentations for GCS URL: https://github.com/apache/incubator-pulsar/pull/2152#issuecomment-407207128 @ivankelly can you review this? This is an automated message

[GitHub] sijie commented on issue #2151: GCS offload support(3): add configs to support GCS driver

2018-07-23 Thread GitBox
sijie commented on issue #2151: GCS offload support(3): add configs to support GCS driver URL: https://github.com/apache/incubator-pulsar/pull/2151#issuecomment-407206660 @ivankelly can you review this? This is an automated

[GitHub] sijie closed pull request #1996: Cpp client: add multiTopicsConsumer

2018-07-23 Thread GitBox
sijie closed pull request #1996: Cpp client: add multiTopicsConsumer URL: https://github.com/apache/incubator-pulsar/pull/1996 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: Cpp client: add multiTopicsConsumer (#1996)

2018-07-23 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 2543ccf Cpp client: add

[GitHub] sijie commented on a change in pull request #2180: Cleanup Arquillian consolidate projects

2018-07-23 Thread GitBox
sijie commented on a change in pull request #2180: Cleanup Arquillian consolidate projects URL: https://github.com/apache/incubator-pulsar/pull/2180#discussion_r204557310 ## File path:

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

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

[GitHub] sijie commented on issue #2213: Add authorization support on function apis

2018-07-23 Thread GitBox
sijie commented on issue #2213: Add authorization support on function apis URL: https://github.com/apache/incubator-pulsar/pull/2213#issuecomment-407154285 @rdhabalia FunctionApiV2ResourceTest is failing with NPE This is an

[GitHub] sijie closed pull request #2208: Fix License Issue

2018-07-23 Thread GitBox
sijie closed pull request #2208: Fix License Issue URL: https://github.com/apache/incubator-pulsar/pull/2208 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 (from

[incubator-pulsar] branch master updated: Fix License Issue (#2208)

2018-07-23 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 b82f66e Fix License Issue (#2208)

[GitHub] sijie closed pull request #2209: Exclude helm files from Apache Rat Check.

2018-07-23 Thread GitBox
sijie closed pull request #2209: Exclude helm files from Apache Rat Check. URL: https://github.com/apache/incubator-pulsar/pull/2209 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

[incubator-pulsar] branch master updated: Exclude helm files from Apache Rat Check. (#2209)

2018-07-23 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 977cc00 Exclude helm files from

[GitHub] grantwwu opened a new issue #2217: Add v2 to Admin API REST documentation

2018-07-23 Thread GitBox
grantwwu opened a new issue #2217: Add v2 to Admin API REST documentation URL: https://github.com/apache/incubator-pulsar/issues/2217 Just putting this here so I don't forget to do this This is an automated message from the

[GitHub] zhaijack commented on issue #2152: GCS offload support(4): add documentations for GCS

2018-07-23 Thread GitBox
zhaijack commented on issue #2152: GCS offload support(4): add documentations for GCS URL: https://github.com/apache/incubator-pulsar/pull/2152#issuecomment-407105954 retest this please This is an automated message from the

[GitHub] zhaijack commented on issue #2151: GCS offload support(3): add configs to support GCS driver

2018-07-23 Thread GitBox
zhaijack commented on issue #2151: GCS offload support(3): add configs to support GCS driver URL: https://github.com/apache/incubator-pulsar/pull/2151#issuecomment-407105851 retest this please This is an automated message

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

2018-07-23 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-407103685 @merlimat - can you please help with the Step#2 above as I don't see "copy" is supported in zookeeper shell..

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

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

[GitHub] merlimat closed pull request #2215: Add option to retrieve schema for a topic from pulsar client

2018-07-23 Thread GitBox
merlimat closed pull request #2215: Add option to retrieve schema for a topic from pulsar client URL: https://github.com/apache/incubator-pulsar/pull/2215 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: Add option to retrieve schema for a topic from pulsar client (#2215)

2018-07-23 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 90908e9 Add option to retrieve