[GitHub] rdhabalia commented on issue #569: Revert back to default ZookeeperClientFactoryImpl

2017-07-17 Thread git
rdhabalia commented on issue #569: Revert back to default ZookeeperClientFactoryImpl URL: https://github.com/apache/incubator-pulsar/pull/569#issuecomment-315687079 After enabling debug log, found out that build exists because `ZooKeeperSessionWatcher` couldn't get heartbeat with in

[GitHub] rdhabalia commented on issue #569: Revert back to default ZookeeperClientFactoryImpl

2017-07-17 Thread git
rdhabalia commented on issue #569: Revert back to default ZookeeperClientFactoryImpl URL: https://github.com/apache/incubator-pulsar/pull/569#issuecomment-315824395 I haven't seen any deadlock or any of the thread is blocked. I took jstack when zksession timed out.

[GitHub] merlimat commented on issue #569: Revert back to default ZookeeperClientFactoryImpl

2017-07-17 Thread git
merlimat commented on issue #569: Revert back to default ZookeeperClientFactoryImpl URL: https://github.com/apache/incubator-pulsar/pull/569#issuecomment-315822121 @rdhabalia Sounds like there is some potential deadlock, right?

[GitHub] rdhabalia opened a new pull request #573: Fix: register zk-stats listener after broker service started

2017-07-17 Thread git
rdhabalia opened a new pull request #573: Fix: register zk-stats listener after broker service started URL: https://github.com/apache/incubator-pulsar/pull/573 ### Motivation As #568 build couldn't complete intermittently: After enabling debug log, found out that build exists

[GitHub] lucperkins opened a new pull request #574: Documentation changes (work in progress)

2017-07-17 Thread git
lucperkins opened a new pull request #574: Documentation changes (work in progress) URL: https://github.com/apache/incubator-pulsar/pull/574 ### Motivation At the moment, the (new) docs are essentially publishable but there are some warts that we should get ironed out ASAP. This PR

[GitHub] merlimat commented on issue #569: Revert back to default ZookeeperClientFactoryImpl

2017-07-17 Thread git
merlimat commented on issue #569: Revert back to default ZookeeperClientFactoryImpl URL: https://github.com/apache/incubator-pulsar/pull/569#issuecomment-315855696 Closing this one in favor of #573 This is an automated

[GitHub] merlimat opened a new pull request #575: Build and publish docker images

2017-07-17 Thread git
merlimat opened a new pull request #575: Build and publish docker images URL: https://github.com/apache/incubator-pulsar/pull/575 ### Motivation Build the docker images during the CI build and push them to DockerHub when a final tag is created.

[GitHub] merlimat closed pull request #570: Simplified the instantiation of Netty EventLoopGroup

2017-07-17 Thread git
merlimat closed pull request #570: Simplified the instantiation of Netty EventLoopGroup URL: https://github.com/apache/incubator-pulsar/pull/570 This is an automated message from the Apache Git Service. To respond to the

[GitHub] merlimat closed pull request #563: Fixes for NOTICE and LICENSE files for src and bin distribution

2017-07-17 Thread git
merlimat closed pull request #563: Fixes for NOTICE and LICENSE files for src and bin distribution URL: https://github.com/apache/incubator-pulsar/pull/563 This is an automated message from the Apache Git Service. To

[GitHub] rdhabalia commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest

2017-07-17 Thread git
rdhabalia commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest URL: https://github.com/apache/incubator-pulsar/issues/578#issuecomment-315927739 it seems it's a deadlock while loading runtime library. `Epoll.isAvailable()` tries to load library and that

[GitHub] jai1 opened a new issue #579: Upgrading to c++11

2017-07-17 Thread git
jai1 opened a new issue #579: Upgrading to c++11 URL: https://github.com/apache/incubator-pulsar/issues/579 Our c++ client uses really old gcc compiler version, boost libraries and C++ version. In due time we will need to upgrade them to use current versions of the same. As of

[GitHub] merlimat commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest

2017-07-17 Thread git
merlimat commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest URL: https://github.com/apache/incubator-pulsar/issues/578#issuecomment-315922611 `WebSocketService.java:169` This is an automated

[GitHub] merlimat commented on issue #575: Build and publish docker images

2017-07-17 Thread git
merlimat commented on issue #575: Build and publish docker images URL: https://github.com/apache/incubator-pulsar/pull/575#issuecomment-315924177 Tested pushing from my repo to personal docker hub account: https://hub.docker.com/u/merlimat/

[GitHub] rdhabalia opened a new issue #578: Intermittent test failure: Websocket.LookupProtocolTest

2017-07-17 Thread git
rdhabalia opened a new issue #578: Intermittent test failure: Websocket.LookupProtocolTest URL: https://github.com/apache/incubator-pulsar/issues/578 Intermittent test failure of

[GitHub] rdhabalia commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest

2017-07-17 Thread git
rdhabalia commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest URL: https://github.com/apache/incubator-pulsar/issues/578#issuecomment-315923124

[GitHub] rdhabalia commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest

2017-07-17 Thread git
rdhabalia commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest URL: https://github.com/apache/incubator-pulsar/issues/578#issuecomment-315935179 [full jsatck](https://gist.github.com/rdhabalia/5d9d5053b19807ab4b91614bf1c1038f)

[GitHub] rdhabalia opened a new pull request #577: CPP add receiveAsync API

2017-07-17 Thread git
rdhabalia opened a new pull request #577: CPP add receiveAsync API URL: https://github.com/apache/incubator-pulsar/pull/577 ### Motivation Sometimes, client wants to receive a message async to complete specific depending task or request. eg. java-client library has

[GitHub] merlimat commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest

2017-07-17 Thread git
merlimat commented on issue #578: Intermittent test failure: Websocket.LookupProtocolTest URL: https://github.com/apache/incubator-pulsar/issues/578#issuecomment-315922554 Not necessarily creating the `PulsarClientImpl` instance. It could also be the line: ```java // If not

[GitHub] merlimat opened a new pull request #576: Only deploy releases and website from apache/incubator-pulsar repo

2017-07-17 Thread git
merlimat opened a new pull request #576: Only deploy releases and website from apache/incubator-pulsar repo URL: https://github.com/apache/incubator-pulsar/pull/576 ### Motivation We should not attempt to publish the release binaries from any other fork of the original