[GitHub] jai1 commented on a change in pull request #541: Message size validation on post compression

2017-06-29 Thread git
jai1 commented on a change in pull request #541: Message size validation on post compression URL: https://github.com/apache/incubator-pulsar/pull/541#discussion_r124968783 ## File path: pulsar-client-cpp/lib/ProducerImpl.cc ## @@ -268,6 +264,14 @@ void

[GitHub] jai1 commented on a change in pull request #541: Message size validation on post compression

2017-06-29 Thread git
jai1 commented on a change in pull request #541: Message size validation on post compression URL: https://github.com/apache/incubator-pulsar/pull/541#discussion_r124968700 ## File path: pulsar-client-cpp/lib/ProducerImpl.cc ## @@ -268,6 +264,14 @@ void

[GitHub] tkb77 opened a new pull request #542: Fixed license link

2017-06-29 Thread git
tkb77 opened a new pull request #542: Fixed license link URL: https://github.com/apache/incubator-pulsar/pull/542 ### Motivation I discover wrong link in README.md markdown ### Modifications Fixed apache license link ### Result Users can jump to license

[GitHub] msb-at-yahoo commented on a change in pull request #541: Message size validation on post compression

2017-06-29 Thread git
msb-at-yahoo commented on a change in pull request #541: Message size validation on post compression URL: https://github.com/apache/incubator-pulsar/pull/541#discussion_r124965142 ## File path: pulsar-client-cpp/lib/ProducerImpl.cc ## @@ -268,6 +264,14 @@ void

[GitHub] msb-at-yahoo commented on a change in pull request #541: Message size validation on post compression

2017-06-29 Thread git
msb-at-yahoo commented on a change in pull request #541: Message size validation on post compression URL: https://github.com/apache/incubator-pulsar/pull/541#discussion_r124956427 ## File path: pulsar-client-cpp/lib/ProducerImpl.cc ## @@ -268,6 +264,14 @@ void

Re: [DISCUSS] PIP-1 Pulsar proxy component

2017-06-29 Thread Dave Fisher
Sent from my iPhone > On Jun 29, 2017, at 5:19 PM, Matteo Merli wrote: > >> On Thu, Jun 29, 2017 at 4:06 PM, Dave Fisher wrote: >> >> Does Apache Zookeeper meet these needs? >> > > Dave, do you mean with respect to SSL? I'm asking as a

Podling Report Reminder - July 2017

2017-06-29 Thread johndament
Dear podling, This email was sent by an automated system on behalf of the Apache Incubator PMC. It is an initial reminder to give you plenty of time to prepare your quarterly board report. The board meeting is scheduled for Wed, 19 July 2017, 10:30 am PDT. The report for your podling will form a

Re: Avoid duplicate notification

2017-06-29 Thread Matteo Merli
I think you can setup a filter in gmail and attach a label + skip the inbox for the github notifications to the mailing list. Example of gmail filter: from:(g...@git.apache.org) to:(dev@pulsar.incubator.apache.org,) Send to Pulsar/Git folder -- Matteo Merli On Thu,

Re: [DISCUSS] PIP-1 Pulsar proxy component

2017-06-29 Thread Maurice Barnum
this seems like it should be a solved problem.from a quick look, for example, it seems like nginx can do this already. https://www.nginx.com/resources/admin-guide/tcp-load-balancing/ On Thursday, June 29, 2017, 4:54:29 PM PDT, Rajan Dhabalia wrote: >> or if people

Avoid duplicate notification

2017-06-29 Thread Sahaya Andrews
I and probably the rest of us gets email notification from GitHub as well as from our dev mailing list for every commit/comment. Is there a way to avoid such duplicate and get one notification instead? The GitHub notification seems to be more readable. Andrews.

Re: [DISCUSS] PIP-1 Pulsar proxy component

2017-06-29 Thread Rajan Dhabalia
>> or if people prefer we can create a GitHub issue to have the discussion on the proposal I think github issue would be more helpful for discussion. so, my vote to github-issue. Thanks, Rajan On Thu, Jun 29, 2017 at 1:51 PM, Matteo Merli wrote: > I have created the wiki

[GitHub] jai1 commented on a change in pull request #536: Added CMake build file and instructions for Pulsar Wireshark dissector

2017-06-29 Thread git
jai1 commented on a change in pull request #536: Added CMake build file and instructions for Pulsar Wireshark dissector URL: https://github.com/apache/incubator-pulsar/pull/536#discussion_r124938532 ## File path: pulsar-client-cpp/wireshark/CMakeLists.txt ## @@ -0,0 +1,56

Re: [jira] [Commented] (INFRA-14376) Pulsar GitHub Integration

2017-06-29 Thread Dave Fisher
Hi Matteo, Thanks. I’ll be trying this out for myself tomorrow. Regards, Dave > On Jun 29, 2017, at 3:42 PM, Matteo Merli wrote: > > The GitBox integration is complete. > > I can see the "pulsar committers" teams in the Apache org in GitHub. > > All the committers should

Fwd: [jira] [Commented] (INFRA-14376) Pulsar GitHub Integration

2017-06-29 Thread Matteo Merli
The GitBox integration is complete. I can see the "pulsar committers" teams in the Apache org in GitHub. All the committers should now link their GitHub with the Apache account and they will be automatically granted write permissions on the repository. I can see 4 of us so far are enabled.

[GitHub] nkurihar opened a new pull request #540: Bump up athenz version

2017-06-29 Thread git
nkurihar opened a new pull request #540: Bump up athenz version URL: https://github.com/apache/incubator-pulsar/pull/540 ### Modifications * Update Athenz version from 1.1.8 to 1.7.17 which includes * artifactId change * signature change on `Token#validate`: a flag

Re: [DISCUSS] PIP-1 Pulsar proxy component

2017-06-29 Thread Joe Francis
 SSL will be supported? Joe On Thursday, June 29, 2017 1:51 PM, Matteo Merli wrote: I have created the wiki page with a first proposal (as we discussed earlier). https://github.com/apache/incubator-pulsar/wiki/PIP-1 Please add feedback in this thread (or if people

[DISCUSS] PIP-1 Pulsar proxy component

2017-06-29 Thread Matteo Merli
I have created the wiki page with a first proposal (as we discussed earlier). https://github.com/apache/incubator-pulsar/wiki/PIP-1 Please add feedback in this thread (or if people prefer we can create a GitHub issue to have the discussion on the proposal). Other than the proposal itself, it

[GitHub] merlimat commented on issue #539: C++ unit tests in less than a minute

2017-06-29 Thread git
merlimat commented on issue #539: C++ unit tests in less than a minute URL: https://github.com/apache/incubator-pulsar/pull/539#issuecomment-312087418 @jai1 One C++ test failure: ``` /home/travis/build/apache/incubator-pulsar/pulsar-client-cpp/tests/BatchMessageTest.cc:309:

[GitHub] incubator-pulsar issue #537: Speed up CPP Unit Tests

2017-06-29 Thread jai1
Github user jai1 closed the issue at: https://github.com/apache/incubator-pulsar/issues/537 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] incubator-pulsar issue #537: Speed up CPP Unit Tests

2017-06-29 Thread jai1
Github user jai1 closed the issue at: https://github.com/apache/incubator-pulsar/issues/537 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] incubator-pulsar issue #507: Add zk-stats instrumentation to get zk-client s...

2017-06-29 Thread merlimat
Github user merlimat commented on the issue: https://github.com/apache/incubator-pulsar/pull/507 @rdhabalia I'll get to this one soon --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] incubator-pulsar issue #507: Add zk-stats instrumentation to get zk-client s...

2017-06-29 Thread merlimat
Github user merlimat commented on the issue: https://github.com/apache/incubator-pulsar/pull/507 @rdhabalia I'll get to this one soon --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] incubator-pulsar pull request #539: C++ unit tests in less than a minute

2017-06-29 Thread merlimat
Github user merlimat commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/539#discussion_r124862843 --- Diff: pulsar-client-cpp/tests/BasicEndToEndTest.cc --- @@ -811,9 +812,14 @@ TEST(BasicEndToEndTest, testMessageListenerPause)

[GitHub] incubator-pulsar pull request #539: C++ unit tests in less than a minute

2017-06-29 Thread merlimat
Github user merlimat commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/539#discussion_r124853117 --- Diff: pulsar-client-cpp/tests/BasicEndToEndTest.cc --- @@ -578,8 +578,13 @@ TEST(BasicEndToEndTest, testSinglePartitionRoutingPolicy)

[GitHub] incubator-pulsar pull request #539: C++ unit tests in less than a minute

2017-06-29 Thread merlimat
Github user merlimat commented on a diff in the pull request: https://github.com/apache/incubator-pulsar/pull/539#discussion_r124853117 --- Diff: pulsar-client-cpp/tests/BasicEndToEndTest.cc --- @@ -578,8 +578,13 @@ TEST(BasicEndToEndTest, testSinglePartitionRoutingPolicy)