[GitHub] sijie commented on issue #2028: Original principal authorization for admin API

2018-06-26 Thread GitBox
sijie commented on issue #2028: Original principal authorization for admin API URL: https://github.com/apache/incubator-pulsar/pull/2028#issuecomment-400559764 @jai1 can you review this PR? This is an automated message from th

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198349118 ## File path: tests/integration-tc/src/main/java/org/apache/pulsar/tests/integration/FUNC_MODE.java ##

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198350288 ## File path: tests/integration-tc/src/main/java/org/apache/pulsar/tests/integration/PulsarStandAloneCluster.ja

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198350183 ## File path: tests/integration-tc/pom.xml ## @@ -0,0 +1,140 @@ + + +http://maven.apache.org/POM/4.0.0

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198350881 ## File path: tests/integration-tc/src/main/java/org/apache/pulsar/tests/integration/PulsarStandAloneCluster.ja

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198350023 ## File path: tests/integration-tc/pom.xml ## @@ -0,0 +1,140 @@ + + +http://maven.apache.org/POM/4.0.0

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198351887 ## File path: tests/integration-tc/src/test/java/org/apache/pulsar/tests/integration/PulsarSimpleTest.java ###

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198349831 ## File path: tests/integration-tc/src/test/java/org/apache/pulsar/tests/integration/PulsarSimpleTest.java ###

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198349528 ## File path: tests/integration-tc/src/main/java/org/apache/pulsar/tests/integration/PulsarStandAloneCluster.ja

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198350053 ## File path: tests/integration-tc/pom.xml ## @@ -0,0 +1,140 @@ + + +http://maven.apache.org/POM/4.0.0

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198350993 ## File path: tests/integration-tc/src/main/java/org/apache/pulsar/tests/integration/PulsarStandAloneCluster.ja

[GitHub] sijie commented on a change in pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
sijie commented on a change in pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036#discussion_r198348941 ## File path: tests/integration-tc/src/main/java/org/apache/pulsar/tests/integration/FUNC_MODE.java ##

svn commit: r27747 - in /dev/incubator/pulsar: pulsar-1.22.1-incubating-candidate-1/ pulsar-2.1.0-incubating-candidate-1/ pulsar-2.1.0-incubating-candidate-2/ pulsar-2.1.0-incubating-candidate-2/DEB/

2018-06-26 Thread sijie
Author: sijie Date: Wed Jun 27 01:30:01 2018 New Revision: 27747 Log: Staging artifacts and signature for Pulsar release 2.1.0-incubating Added: dev/incubator/pulsar/pulsar-2.1.0-incubating-candidate-2/ dev/incubator/pulsar/pulsar-2.1.0-incubating-candidate-2/DEB/ dev/incubator/pulsar

[GitHub] aahmed-se opened a new pull request #2036: Add testcontainer based standalone test

2018-06-26 Thread GitBox
aahmed-se opened a new pull request #2036: Add testcontainer based standalone test URL: https://github.com/apache/incubator-pulsar/pull/2036 This adds a standalone cluster test and introduces the test container library. This

[incubator-pulsar] annotated tag v2.1.0-incubating-candidate-2 updated (30f6128 -> b71a38c)

2018-06-26 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to annotated tag v2.1.0-incubating-candidate-2 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git. *** WARNING: tag v2.1.0-incubating-candidate-2 was modified! *** from 30f6128 (co

[incubator-pulsar] annotated tag v2.1.0-incubating-candidate-2 deleted (was 20d3eee)

2018-06-26 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to annotated tag v2.1.0-incubating-candidate-2 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git. *** WARNING: tag v2.1.0-incubating-candidate-2 was deleted! *** tag was 20d3eee T

[incubator-pulsar] branch master updated: Exclude a few files from apache-rat:check (#2035)

2018-06-26 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 174fd02 Exclude a few files from apach

[GitHub] sijie closed pull request #2035: Exclude a few files from apache-rat:check

2018-06-26 Thread GitBox
sijie closed pull request #2035: Exclude a few files from apache-rat:check URL: https://github.com/apache/incubator-pulsar/pull/2035 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 for

[GitHub] merlimat closed pull request #2034: adding a module that shades managed-ledge

2018-06-26 Thread GitBox
merlimat closed pull request #2034: adding a module that shades managed-ledge URL: https://github.com/apache/incubator-pulsar/pull/2034 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: adding a module that shades managed-ledge (#2034)

2018-06-26 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 b0292a1 adding a module that shades

[GitHub] sijie opened a new pull request #2035: Exclude a few files from apache-rat:check

2018-06-26 Thread GitBox
sijie opened a new pull request #2035: Exclude a few files from apache-rat:check URL: https://github.com/apache/incubator-pulsar/pull/2035 *Motivation* Apache-rat check is failing *Changes* - Exclude two BSD licensed files - Exclude files related to certs used for tes

[incubator-pulsar] annotated tag v2.1.0-incubating-candidate-2 updated (3fbe4ac -> 20d3eee)

2018-06-26 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to annotated tag v2.1.0-incubating-candidate-2 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git. *** WARNING: tag v2.1.0-incubating-candidate-2 was modified! *** from 3fbe4ac (co

[GitHub] jerrypeng opened a new pull request #2034: adding a module that shades managed-ledger 6f31887

2018-06-26 Thread GitBox
jerrypeng opened a new pull request #2034: adding a module that shades managed-ledger 6f31887 URL: https://github.com/apache/incubator-pulsar/pull/2034 Shading the managed-ledger API so that using the API externally do not cause class clashes -

[incubator-pulsar] branch master updated: Added custom compiled OpenSSL into docker image for RPM/Deb (#2032)

2018-06-26 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 ca93d4a Added custom compiled OpenSSL

[GitHub] merlimat closed pull request #2032: Added custom compiled OpenSSL into docker image for RPM/Deb

2018-06-26 Thread GitBox
merlimat closed pull request #2032: Added custom compiled OpenSSL into docker image for RPM/Deb URL: https://github.com/apache/incubator-pulsar/pull/2032 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 provena

[incubator-pulsar] 02/03: Added integer types casting to avoid compile errors with gcc-4.8 (#2031)

2018-06-26 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-2.1 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git commit e959f09291021a5916fcf77345113d97f3580274 Author: Matteo Merli AuthorDate: Tue Jun 26 12:46:22 2018 -0700

[incubator-pulsar] 01/03: Fixed C99 variable initialization in cgo header (#2029)

2018-06-26 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-2.1 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git commit 7f8985a37c0fb237b807be28332e4ce131ab2044 Author: Matteo Merli AuthorDate: Tue Jun 26 11:14:02 2018 -0700

[incubator-pulsar] 03/03: Added custom compiled OpenSSL into docker image for RPM/Deb

2018-06-26 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-2.1 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git commit 3fbe4ac72e33487a534acce4493a3be959195592 Author: Matteo Merli AuthorDate: Tue Jun 26 11:17:06 2018 -0700

[incubator-pulsar] branch branch-2.1 updated (840ad88 -> 3fbe4ac)

2018-06-26 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch branch-2.1 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git. from 840ad88 Update LICENSE & NOTICE for newly introduced or version-upgraded dependencies (#2023) n

[GitHub] merlimat closed pull request #2031: Added integer types casting to avoid compile errors with gcc-4.8

2018-06-26 Thread GitBox
merlimat closed pull request #2031: Added integer types casting to avoid compile errors with gcc-4.8 URL: https://github.com/apache/incubator-pulsar/pull/2031 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: Added integer types casting to avoid compile errors with gcc-4.8 (#2031)

2018-06-26 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 bdf52bc Added integer types casting t

[incubator-pulsar] branch jerrypeng-patch-1 deleted (was 169a6e5)

2018-06-26 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch jerrypeng-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git. was 169a6e5 remove static The revisions that were on this branch are still contained in other refe

[GitHub] merlimat closed pull request #2030: Change MessageProcessor Interface to be public

2018-06-26 Thread GitBox
merlimat closed pull request #2030: Change MessageProcessor Interface to be public URL: https://github.com/apache/incubator-pulsar/pull/2030 This is an automated message from the Apache Git Service. To respond to the message

[incubator-pulsar] branch master updated: Change MessageProcessor Interface to be public (#2030)

2018-06-26 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 ae90f22 Change MessageProcessor Inter

[incubator-pulsar.wiki] branch master updated: Updated Home (markdown)

2018-06-26 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.wiki.git The following commit(s) were added to refs/heads/master by this push: new c2cdb88 Updated Home (markdown)

[incubator-pulsar.wiki] branch master updated: Updated Client Features Matrix (markdown)

2018-06-26 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.wiki.git The following commit(s) were added to refs/heads/master by this push: new 040a3a8 Updated Client Features

[incubator-pulsar.wiki] branch master updated: Created Client Features Matrix (markdown)

2018-06-26 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.wiki.git The following commit(s) were added to refs/heads/master by this push: new b9c1b07 Created Client Features

[GitHub] merlimat commented on issue #2033: Python pulsar-client lacks multi-topic support

2018-06-26 Thread GitBox
merlimat commented on issue #2033: Python pulsar-client lacks multi-topic support URL: https://github.com/apache/incubator-pulsar/issues/2033#issuecomment-400423805 @georgewilk01 The multi-topic / regex subscribe for C++/Python client lib is currently being worked on and it's planned for

[GitHub] georgewilk01 opened a new issue #2033: Python pulsar-client lacks multi-topic support

2018-06-26 Thread GitBox
georgewilk01 opened a new issue #2033: Python pulsar-client lacks multi-topic support URL: https://github.com/apache/incubator-pulsar/issues/2033 Expected behavior According to documentation, post 1.7 API supports multi-topic subscriptions via regx and list. Actual

[GitHub] merlimat opened a new pull request #2032: Added custom compiled OpenSSL into docker image for RPM/Deb

2018-06-26 Thread GitBox
merlimat opened a new pull request #2032: Added custom compiled OpenSSL into docker image for RPM/Deb URL: https://github.com/apache/incubator-pulsar/pull/2032 ### Motivation The static libssl that was being installed in Debian and Centos was giving link errors when trying to build

[GitHub] merlimat opened a new pull request #2031: Added integer types casting to avoid compile errors with gcc-4.8

2018-06-26 Thread GitBox
merlimat opened a new pull request #2031: Added integer types casting to avoid compile errors with gcc-4.8 URL: https://github.com/apache/incubator-pulsar/pull/2031 ### Motivation Added extra castings to fix compile issues in gcc-4.8 -

[GitHub] merlimat closed pull request #2029: Fixed C99 variable initialization in cgo header

2018-06-26 Thread GitBox
merlimat closed pull request #2029: Fixed C99 variable initialization in cgo header URL: https://github.com/apache/incubator-pulsar/pull/2029 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 thi

[incubator-pulsar] branch master updated: Fixed C99 variable initialization in cgo header (#2029)

2018-06-26 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 97d06c9 Fixed C99 variable initializa

[incubator-pulsar] branch jerrypeng-patch-1 updated (0297e43 -> 169a6e5)

2018-06-26 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng pushed a change to branch jerrypeng-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git. from 0297e43 Change MessageProcessor Interface to be public add 169a6e5 remove static No n

[GitHub] jerrypeng opened a new pull request #2030: Change MessageProcessor Interface to be public

2018-06-26 Thread GitBox
jerrypeng opened a new pull request #2030: Change MessageProcessor Interface to be public URL: https://github.com/apache/incubator-pulsar/pull/2030 This is an automated message from the Apache Git Service. To respond to the

[GitHub] merlimat opened a new pull request #2029: Fixed C99 variable initialization in cgo header

2018-06-26 Thread GitBox
merlimat opened a new pull request #2029: Fixed C99 variable initialization in cgo header URL: https://github.com/apache/incubator-pulsar/pull/2029 ### Motivation Getting this error on Centos7 when fetching and compiling the go client lib. ``` $ go get -u github.com/apache/

[GitHub] ivankelly opened a new pull request #2028: Original principal authorization for admin API

2018-06-26 Thread GitBox
ivankelly opened a new pull request #2028: Original principal authorization for admin API URL: https://github.com/apache/incubator-pulsar/pull/2028 The broker admin checks if the authenticated user is listed as a proxy user. If so, it checks looks for the header X-Original-Principal, and

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

2018-06-26 Thread GitBox
merlimat commented on issue #2025: 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-400321766 > @merlimat we expected this feature to work through Pulsar Java

[GitHub] ivankelly edited a comment on issue #1991: TLS auth cannot be used between proxy and brokers

2018-06-26 Thread GitBox
ivankelly edited a comment on issue #1991: TLS auth cannot be used between proxy and brokers URL: https://github.com/apache/incubator-pulsar/issues/1991#issuecomment-400237543 > " *forwardAuthorizationCredentials*" on the proxy and enable " *authenticateOriginalAuthData*" I assume

[GitHub] ivankelly commented on issue #1991: TLS auth cannot be used between proxy and brokers

2018-06-26 Thread GitBox
ivankelly commented on issue #1991: TLS auth cannot be used between proxy and brokers URL: https://github.com/apache/incubator-pulsar/issues/1991#issuecomment-400237543 > " *forwardAuthorizationCredentials*" on the proxy and enable " *authenticateOriginalAuthData*" I assume this on

[GitHub] jai1 commented on issue #1991: TLS auth cannot be used between proxy and brokers

2018-06-26 Thread GitBox
jai1 commented on issue #1991: TLS auth cannot be used between proxy and brokers URL: https://github.com/apache/incubator-pulsar/issues/1991#issuecomment-400220446 Hi Ivan, Since proxy was meant to be on a public IP we have added extra authentication checks to prevent MITM attacks