[GitHub] merlimat closed pull request #611: Small typo fixes on website

merlimat closed pull request #611: Small typo fixes on website URL: https://github.com/apache/incubator-pulsar/pull/611 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] merlimat commented on a change in pull request #620: added new entrypoint for reader to websocket proxy

merlimat commented on a change in pull request #620: added new entrypoint for reader to websocket proxy URL: https://github.com/apache/incubator-pulsar/pull/620#discussion_r130781706 ## File path: pulsar-websocket/src/main/java/org/apache/pulsar/websocket/ReaderHandler.java

Podling Report Reminder - August 2017

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, 16 August 2017, 10:30 am PDT. The report for your podling will form

[GitHub] merlimat commented on issue #621: Documentation for deployment of a single cluster in a data center

merlimat commented on issue #621: Documentation for deployment of a single cluster in a data center URL: https://github.com/apache/incubator-pulsar/issues/621#issuecomment-319554900 cc/ @lucperkins This is an automated

[GitHub] merlimat opened a new issue #621: Documentation for deployment of a single cluster in a data center

merlimat opened a new issue #621: Documentation for deployment of a single cluster in a data center URL: https://github.com/apache/incubator-pulsar/issues/621 Current deployment section in Pulsar for bare metal directly talks about deploying clusters across different geographical

[GitHub] jai1 commented on a change in pull request #619: make batch-ack thread-safe

jai1 commented on a change in pull request #619: make batch-ack thread-safe URL: https://github.com/apache/incubator-pulsar/pull/619#discussion_r130764896 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -338,15 +338,25 @@

[GitHub] rdhabalia commented on a change in pull request #619: make batch-ack thread-safe

rdhabalia commented on a change in pull request #619: make batch-ack thread-safe URL: https://github.com/apache/incubator-pulsar/pull/619#discussion_r130762148 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -377,7 +379,6 @@

[GitHub] merlimat commented on a change in pull request #619: make batch-ack thread-safe

merlimat commented on a change in pull request #619: make batch-ack thread-safe URL: https://github.com/apache/incubator-pulsar/pull/619#discussion_r130761843 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -377,7 +379,6 @@

[GitHub] hrsakai opened a new pull request #620: added new entrypoint for reader to websocket proxy

hrsakai opened a new pull request #620: added new entrypoint for reader to websocket proxy URL: https://github.com/apache/incubator-pulsar/pull/620 ### Motivation no reader entrypoint on websocket proxy . ### Modifications added reader entrypoint to websocket proxy. ###

[GitHub] rdhabalia commented on a change in pull request #619: make batch-ack thread-safe

rdhabalia commented on a change in pull request #619: make batch-ack thread-safe URL: https://github.com/apache/incubator-pulsar/pull/619#discussion_r130761640 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -338,15 +338,25

[GitHub] jai1 commented on a change in pull request #619: make batch-ack thread-safe

jai1 commented on a change in pull request #619: make batch-ack thread-safe URL: https://github.com/apache/incubator-pulsar/pull/619#discussion_r130760041 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -338,15 +338,25 @@

[GitHub] rdhabalia opened a new pull request #619: make batch-ack thread-safe

rdhabalia opened a new pull request #619: make batch-ack thread-safe URL: https://github.com/apache/incubator-pulsar/pull/619 ### Motivation if consumer acks batch-messages concurrently, then client-consumer is not acking the message to broker even if consumer acks all

[GitHub] jai1 commented on issue #618: Intermittent failures in C++ unit test BasicEndToEndTest.testResendViaSendCallback

jai1 commented on issue #618: Intermittent failures in C++ unit test BasicEndToEndTest.testResendViaSendCallback URL: https://github.com/apache/incubator-pulsar/issues/618#issuecomment-319528915 I know about this issue, I will look into it this weekend since it's a bit hard to reproduce

[GitHub] merlimat commented on issue #618: Intermittent failures in C++ unit test BasicEndToEndTest.testResendViaSendCallback

merlimat commented on issue #618: Intermittent failures in C++ unit test BasicEndToEndTest.testResendViaSendCallback URL: https://github.com/apache/incubator-pulsar/issues/618#issuecomment-319516570 It looks it tries to use the shared_pointer after it was already reset or destroyed

[GitHub] merlimat opened a new issue #618: Intermittent failures in C++ unit test BasicEndToEndTest.testResendViaSendCallback

merlimat opened a new issue #618: Intermittent failures in C++ unit test BasicEndToEndTest.testResendViaSendCallback URL: https://github.com/apache/incubator-pulsar/issues/618 Seen a few times already. Example: https://builds.apache.org/job/pulsar-pull-request/20/console ```

Re: Aug Podling report - Draft

I have updated the Podling report (including the 2 edits from Dave and Matteo). Mentors,  please review and sign-off. Cheers,Joe On Monday, July 31, 2017 11:56 PM, Nozomi Kurihara wrote: Hi Joe, Thank you for the draft and it looks good to me. Regards,

[GitHub] merlimat closed pull request #613: Reduce memory usage in producer and subscriptions maps

merlimat closed pull request #613: Reduce memory usage in producer and subscriptions maps URL: https://github.com/apache/incubator-pulsar/pull/613 This is an automated message from the Apache Git Service. To respond to the

[GitHub] merlimat opened a new issue #617: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice

merlimat opened a new issue #617: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice URL: https://github.com/apache/incubator-pulsar/issues/617 As seen in

[GitHub] merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode

merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615#discussion_r130738285 ## File path: pulsar-client-cpp/CMakeLists.txt ## @@ -23,13 +23,20 @@ project (pulsar-cpp)

[GitHub] merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode

merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615#discussion_r130737857 ## File path: pulsar-client-cpp/CMakeLists.txt ## @@ -23,13 +23,20 @@ project (pulsar-cpp)

[GitHub] jai1 commented on a change in pull request #615: Allow to compile C++ client lib in debug mode

jai1 commented on a change in pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615#discussion_r130737106 ## File path: pulsar-client-cpp/CMakeLists.txt ## @@ -23,13 +23,20 @@ project (pulsar-cpp)

[GitHub] merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode

merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615#discussion_r130730785 ## File path: pulsar-client-cpp/docker-build.sh ## @@ -20,6 +20,8 @@ # Build Pulsar C++

[GitHub] merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode

merlimat commented on a change in pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615#discussion_r130730421 ## File path: pulsar-client-cpp/CMakeLists.txt ## @@ -23,13 +23,24 @@ project (pulsar-cpp)

[GitHub] jai1 commented on a change in pull request #615: Allow to compile C++ client lib in debug mode

jai1 commented on a change in pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615#discussion_r130728604 ## File path: pulsar-client-cpp/CMakeLists.txt ## @@ -23,13 +23,24 @@ project (pulsar-cpp)

[GitHub] jai1 commented on a change in pull request #615: Allow to compile C++ client lib in debug mode

jai1 commented on a change in pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615#discussion_r130729150 ## File path: pulsar-client-cpp/docker-build.sh ## @@ -20,6 +20,8 @@ # Build Pulsar C++ client

[GitHub] merlimat opened a new pull request #616: Don't attach pulsar-*-{src,bin}.tar.gz to maven artifacts

merlimat opened a new pull request #616: Don't attach pulsar-*-{src,bin}.tar.gz to maven artifacts URL: https://github.com/apache/incubator-pulsar/pull/616 ### Motivation Currently, the src and bin distribution archives are getting attached as maven artifacts and thus deployed and

[GitHub] merlimat opened a new pull request #615: Allow to compile C++ client lib in debug mode

merlimat opened a new pull request #615: Allow to compile C++ client lib in debug mode URL: https://github.com/apache/incubator-pulsar/pull/615 ### Motivation Allow to compile C++ code in debug mode by adding `-DDEBUG_MODE=ON` when invoking CMake. Additionally, when

[GitHub] jai1 closed issue #584: Upgrade CPP Client to C++11

jai1 closed issue #584: Upgrade CPP Client to C++11 URL: https://github.com/apache/incubator-pulsar/issues/584 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] merlimat commented on issue #551: Problems in lookup flow in C++ Client

merlimat commented on issue #551: Problems in lookup flow in C++ Client URL: https://github.com/apache/incubator-pulsar/issues/551#issuecomment-319474221 @jai1 Were all the 3 items in this issue fixed in 1.19? This is an

[GitHub] merlimat opened a new issue #614: Itermittent test failure in ReplicatorTest.testResumptionAfterBacklogRelaxed

merlimat opened a new issue #614: Itermittent test failure in ReplicatorTest.testResumptionAfterBacklogRelaxed URL: https://github.com/apache/incubator-pulsar/issues/614 Test logs at:

[GitHub] merlimat opened a new pull request #613: Reduce memory usage in producer and subscriptions maps

merlimat opened a new pull request #613: Reduce memory usage in producer and subscriptions maps URL: https://github.com/apache/incubator-pulsar/pull/613 ### Motivation In the topic object, we're creating a map to reference producers, subscriptions and replicators. This map is not

[GitHub] merlimat closed pull request #609: Bumped version to 1.20.0-incubating-SNAPSHOT

merlimat closed pull request #609: Bumped version to 1.20.0-incubating-SNAPSHOT URL: https://github.com/apache/incubator-pulsar/pull/609 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] merlimat closed pull request #610: Redirect unit tests logs to file

merlimat closed pull request #610: Redirect unit tests logs to file URL: https://github.com/apache/incubator-pulsar/pull/610 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] merlimat commented on a change in pull request #610: Redirect unit tests logs to file

merlimat commented on a change in pull request #610: Redirect unit tests logs to file URL: https://github.com/apache/incubator-pulsar/pull/610#discussion_r130671094 ## File path: pom.xml ## @@ -557,10 +596,17 @@ flexible messaging model and an intuitive client API.

[GitHub] saandrews commented on a change in pull request #610: Redirect unit tests logs to file

saandrews commented on a change in pull request #610: Redirect unit tests logs to file URL: https://github.com/apache/incubator-pulsar/pull/610#discussion_r130668918 ## File path: pom.xml ## @@ -557,10 +596,17 @@ flexible messaging model and an intuitive client API.

[GitHub] merlimat opened a new issue #612: Intermittent test failure on NonPersistentTopicTest

merlimat opened a new issue #612: Intermittent test failure on NonPersistentTopicTest URL: https://github.com/apache/incubator-pulsar/issues/612 Seen at

[GitHub] merlimat opened a new pull request #611: Small typo fixes on website

merlimat opened a new pull request #611: Small typo fixes on website URL: https://github.com/apache/incubator-pulsar/pull/611 This is an automated message from the Apache Git Service. To respond to the message, please log

CI builds on Jenkins

As we discussed some time back, I have added CI builds in Jenkins. Jenkins seems to be much more stable compare to Travis and it's bit more flexible in terms of what we can do. For example, for C++ build we can use custom Docker image with all the dependencies we require to build the Pulsar

Podling Report Reminder - August 2017

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, 16 August 2017, 10:30 am PDT. The report for your podling will form

Re: Aug Podling report - Draft

Hi Joe, Thank you for the draft and it looks good to me. Regards, Nozomi 2017/08/01 7:20 に、"Joe Francis" を書き込みました: Thanks, Dave and Matteo. Will add those edits. I will wait a day for any other comments, and submit with all edits tomorrow. Joe

[GitHub] rdhabalia commented on a change in pull request #610: Redirect unit tests logs to file

rdhabalia commented on a change in pull request #610: Redirect unit tests logs to file URL: https://github.com/apache/incubator-pulsar/pull/610#discussion_r130527854 ## File path: pulsar-broker/src/test/resources/log4j.properties ## @@ -1,41 +0,0 @@ -# -# Licensed to the

[GitHub] merlimat commented on a change in pull request #610: Redirect unit tests logs to file

merlimat commented on a change in pull request #610: Redirect unit tests logs to file URL: https://github.com/apache/incubator-pulsar/pull/610#discussion_r130527374 ## File path: pulsar-broker/src/test/resources/log4j.properties ## @@ -1,41 +0,0 @@ -# -# Licensed to the

[GitHub] rdhabalia commented on a change in pull request #610: Redirect unit tests logs to file

rdhabalia commented on a change in pull request #610: Redirect unit tests logs to file URL: https://github.com/apache/incubator-pulsar/pull/610#discussion_r130526876 ## File path: pulsar-broker/src/test/resources/log4j.properties ## @@ -1,41 +0,0 @@ -# -# Licensed to the