[GitHub] sijie commented on issue #3017: [state][standalone] Fix bugs on running function with multiple instances in standalone

2018-11-20 Thread GitBox
sijie commented on issue #3017: [state][standalone] Fix bugs on running function with multiple instances in standalone URL: https://github.com/apache/pulsar/pull/3017#issuecomment-440181039 run java8 tests This is an automate

[pulsar] branch master updated: add producer flush method in cpp client (#3020)

2018-11-20 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new b982b5d add producer flush method in cpp client

[GitHub] sijie closed pull request #3020: Issue #2984: Add producer flush method in cpp client

2018-11-20 Thread GitBox
sijie closed pull request #3020: Issue #2984: Add producer flush method in cpp client URL: https://github.com/apache/pulsar/pull/3020 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 f

[GitHub] sijie commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint

2018-11-20 Thread GitBox
sijie commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint URL: https://github.com/apache/pulsar/pull/2943#issuecomment-440181896 run java8 tests This is an automated message from the Apach

[GitHub] merlimat opened a new pull request #3023: Moved broker aspectj to zk-utils

2018-11-20 Thread GitBox
merlimat opened a new pull request #3023: Moved broker aspectj to zk-utils URL: https://github.com/apache/pulsar/pull/3023 ### Motivation Move AspectJ class to instrument ZK client into `pulsar-zookeeper-utils` module. This will concentrate all AOJ related stuff out of `pulsar-broker

[GitHub] jiazhai commented on issue #2997: apache pulsar 安全认证问题

2018-11-20 Thread GitBox
jiazhai commented on issue #2997: apache pulsar 安全认证问题 URL: https://github.com/apache/pulsar/issues/2997#issuecomment-440198186 After discuss with @xinln, This is a conf error. If running with pulsar standalone mode, we should change `standalone.conf` instead of `broker.conf` -

[GitHub] jiazhai closed issue #2997: apache pulsar 安全认证问题

2018-11-20 Thread GitBox
jiazhai closed issue #2997: apache pulsar 安全认证问题 URL: https://github.com/apache/pulsar/issues/2997 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL abov

[GitHub] massakam opened a new pull request #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
massakam opened a new pull request #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024 ### Motivation Currently, the C++ client library included in the distributed RPM package can not perform https lookup. > 2018-11-20 21:17:36.7

[GitHub] maskit commented on issue #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
maskit commented on issue #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024#issuecomment-440278043 While you are at it, can you update OpenSSL versions? This is an automated mes

[GitHub] jiazhai commented on issue #2731: Ws client can't receive any more messages when number of timed-out messages reaches receiverQueueSize

2018-11-20 Thread GitBox
jiazhai commented on issue #2731: Ws client can't receive any more messages when number of timed-out messages reaches receiverQueueSize URL: https://github.com/apache/pulsar/issues/2731#issuecomment-440304897 As the [document](https://github.com/apache/pulsar/blob/branch-2.2/pulsar-websock

[GitHub] jiazhai commented on issue #2731: Ws client can't receive any more messages when number of timed-out messages reaches receiverQueueSize

2018-11-20 Thread GitBox
jiazhai commented on issue #2731: Ws client can't receive any more messages when number of timed-out messages reaches receiverQueueSize URL: https://github.com/apache/pulsar/issues/2731#issuecomment-440323006 @merlimat @rdhabalia Do you think we need break the original behaviour? Seems

[GitHub] merlimat commented on a change in pull request #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
merlimat commented on a change in pull request #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024#discussion_r235064344 ## File path: pulsar-client-cpp/docker/Dockerfile ## @@ -46,12 +46,12 @@ RUN curl -O -L https://zlib.net/

[GitHub] merlimat commented on issue #3003: Refactored C++ tests

2018-11-20 Thread GitBox
merlimat commented on issue #3003: Refactored C++ tests URL: https://github.com/apache/pulsar/pull/3003#issuecomment-440329328 run integration tests This is an automated message from the Apache Git Service. To respond to the m

[GitHub] sijie commented on issue #3023: Moved broker aspectj to zk-utils

2018-11-20 Thread GitBox
sijie commented on issue #3023: Moved broker aspectj to zk-utils URL: https://github.com/apache/pulsar/pull/3023#issuecomment-440345250 run integration tests This is an automated message from the Apache Git Service. To respond

[GitHub] sijie closed pull request #3022: [proxy] Use ProxyServlet instead of AsyncProxyServlet

2018-11-20 Thread GitBox
sijie closed pull request #3022: [proxy] Use ProxyServlet instead of AsyncProxyServlet URL: https://github.com/apache/pulsar/pull/3022 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

[GitHub] sijie closed pull request #3017: [state][standalone] Fix bugs on running function with multiple instances in standalone

2018-11-20 Thread GitBox
sijie closed pull request #3017: [state][standalone] Fix bugs on running function with multiple instances in standalone URL: https://github.com/apache/pulsar/pull/3017 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the s

[pulsar] branch master updated: [proxy] Use ProxyServlet instead of AsyncProxyServlet (#3022)

2018-11-20 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new a8023df [proxy] Use ProxyServlet instead of Asyn

[pulsar] branch master updated: [state][standalone] When running with more than one function instance, Function instances will crash because of table service throws INTERNAL_SERVER_ERROR (#3017)

2018-11-20 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 4f1b53c [state][standalone] When running with mo

[GitHub] massakam commented on a change in pull request #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
massakam commented on a change in pull request #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024#discussion_r235089474 ## File path: pulsar-client-cpp/docker/Dockerfile ## @@ -46,12 +46,12 @@ RUN curl -O -L https://zlib.net/

[GitHub] sijie commented on a change in pull request #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
sijie commented on a change in pull request #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#discussion_r235091471 ## File path: pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/InstanceCache.java ##

[GitHub] sijie commented on a change in pull request #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
sijie commented on a change in pull request #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#discussion_r235085963 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/FunctionsBase.java ## @@ -190,7 +1

[GitHub] jerrypeng commented on a change in pull request #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
jerrypeng commented on a change in pull request #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#discussion_r235104150 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/FunctionsBase.java ## @@ -190,

[GitHub] jerrypeng commented on a change in pull request #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
jerrypeng commented on a change in pull request #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#discussion_r235104584 ## File path: pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/InstanceCache.java ###

[GitHub] merlimat commented on a change in pull request #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
merlimat commented on a change in pull request #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024#discussion_r235108396 ## File path: pulsar-client-cpp/docker/Dockerfile ## @@ -46,12 +46,12 @@ RUN curl -O -L https://zlib.net/

[pulsar] branch master updated: Refactored C++ tests (#3003)

2018-11-20 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new f8594e0 Refactored C++ tests (#3003) f8594e0 is

[GitHub] merlimat closed pull request #3003: Refactored C++ tests

2018-11-20 Thread GitBox
merlimat closed pull request #3003: Refactored C++ tests URL: https://github.com/apache/pulsar/pull/3003 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 a fo

[GitHub] merlimat opened a new pull request #3025: Increase tests timeout from 2 to 5min

2018-11-20 Thread GitBox
merlimat opened a new pull request #3025: Increase tests timeout from 2 to 5min URL: https://github.com/apache/pulsar/pull/3025 ### Motivation Some integration tests have running time of close to 2minutes and sometimes spill over and fail with timeout. Since this is an upper b

[GitHub] bruth opened a new pull request #3026: Add bookkeeper service and other fixes

2018-11-20 Thread GitBox
bruth opened a new pull request #3026: Add bookkeeper service and other fixes URL: https://github.com/apache/pulsar/pull/3026 ### Motivation Various issues in the GCP k8s config files that prevented the cluster from working properly. ### Modifications - Add missing book

[GitHub] cbornet commented on issue #271: Consider using gRPC as an externally exposed API

2018-11-20 Thread GitBox
cbornet commented on issue #271: Consider using gRPC as an externally exposed API URL: https://github.com/apache/pulsar/issues/271#issuecomment-440381769 > One of the primary goals of the custom binary protocol we came up with, was to have the client establish a "session" (either producer

[GitHub] cbornet edited a comment on issue #271: Consider using gRPC as an externally exposed API

2018-11-20 Thread GitBox
cbornet edited a comment on issue #271: Consider using gRPC as an externally exposed API URL: https://github.com/apache/pulsar/issues/271#issuecomment-440381769 > One of the primary goals of the custom binary protocol we came up with, was to have the client establish a "session" (either p

[GitHub] cbornet commented on issue #271: Consider using gRPC as an externally exposed API

2018-11-20 Thread GitBox
cbornet commented on issue #271: Consider using gRPC as an externally exposed API URL: https://github.com/apache/pulsar/issues/271#issuecomment-440384538 And if I'm not mistaken BookKeeper uses gRPC internally, so it would be coherent to make it the base protocol in Pulsar also. -

[GitHub] merlimat commented on a change in pull request #3026: Add bookkeeper service and other fixes

2018-11-20 Thread GitBox
merlimat commented on a change in pull request #3026: Add bookkeeper service and other fixes URL: https://github.com/apache/pulsar/pull/3026#discussion_r235127664 ## File path: deployment/kubernetes/google-kubernetes-engine/bookie.yaml ## @@ -155,6 +156,29 @@ spec:

[GitHub] rdhabalia commented on issue #3023: Moved broker aspectj to zk-utils

2018-11-20 Thread GitBox
rdhabalia commented on issue #3023: Moved broker aspectj to zk-utils URL: https://github.com/apache/pulsar/pull/3023#issuecomment-440391786 so, what was the exact issue by keeping aj file under broker? This is an automated mes

[GitHub] bruth commented on a change in pull request #3026: Add bookkeeper service and other fixes

2018-11-20 Thread GitBox
bruth commented on a change in pull request #3026: Add bookkeeper service and other fixes URL: https://github.com/apache/pulsar/pull/3026#discussion_r235131549 ## File path: deployment/kubernetes/google-kubernetes-engine/bookie.yaml ## @@ -155,6 +156,29 @@ spec:

[GitHub] rdhabalia commented on issue #2910: Avoid double encoding for subscription name in pulsar-admin

2018-11-20 Thread GitBox
rdhabalia commented on issue #2910: Avoid double encoding for subscription name in pulsar-admin URL: https://github.com/apache/pulsar/pull/2910#issuecomment-440397447 rerun java8 tests This is an automated message from the Ap

[GitHub] rdhabalia commented on issue #2910: Avoid double encoding for subscription name in pulsar-admin

2018-11-20 Thread GitBox
rdhabalia commented on issue #2910: Avoid double encoding for subscription name in pulsar-admin URL: https://github.com/apache/pulsar/pull/2910#issuecomment-440397471 rerun integration tests This is an automated message from

[GitHub] merlimat opened a new pull request #3027: Fix C++ tests after merge

2018-11-20 Thread GitBox
merlimat opened a new pull request #3027: Fix C++ tests after merge URL: https://github.com/apache/pulsar/pull/3027 ### Motivation There was a conflict between #3003 and #3020 that made C++ tests to fail after merging because of the namespaces changes. --

[GitHub] sijie commented on issue #271: Consider using gRPC as an externally exposed API

2018-11-20 Thread GitBox
sijie commented on issue #271: Consider using gRPC as an externally exposed API URL: https://github.com/apache/pulsar/issues/271#issuecomment-440401047 > That said I have started the work on a gRPC proxy. Consumption and production are working. I need to clean it up then I'll do a PR.

[GitHub] sijie commented on a change in pull request #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
sijie commented on a change in pull request #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#discussion_r235139847 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/FunctionsBase.java ## @@ -190,7 +1

[GitHub] merlimat commented on issue #271: Consider using gRPC as an externally exposed API

2018-11-20 Thread GitBox
merlimat commented on issue #271: Consider using gRPC as an externally exposed API URL: https://github.com/apache/pulsar/issues/271#issuecomment-440401929 @cbornet The reason we haven't used gRPC is that it wasn't available when we started, so we went with custom protocol over protocol buf

[GitHub] rdhabalia closed pull request #2828: Support topic name with / in name

2018-11-20 Thread GitBox
rdhabalia closed pull request #2828: Support topic name with / in name URL: https://github.com/apache/pulsar/pull/2828 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 requ

[pulsar] branch master updated: Fix-2876 Limit the client reconnect behavior (#2977)

2018-11-20 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 3008a4b Fix-2876 Limit the client reconnect beha

[GitHub] sijie commented on issue #2966: Broke Pulsar-IO::HDFS into 3.x and 2.x versions

2018-11-20 Thread GitBox
sijie commented on issue #2966: Broke Pulsar-IO::HDFS into 3.x and 2.x versions URL: https://github.com/apache/pulsar/pull/2966#issuecomment-440413304 run java8 tests This is an automated message from the Apache Git Service. T

[GitHub] sijie closed pull request #2977: Fix-2876 Limit the client reconnect behavior

2018-11-20 Thread GitBox
sijie closed pull request #2977: Fix-2876 Limit the client reconnect behavior URL: https://github.com/apache/pulsar/pull/2977 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 pu

[GitHub] sijie closed pull request #2935: Ensure broker is fully boostrapped before load manager register itself

2018-11-20 Thread GitBox
sijie closed pull request #2935: Ensure broker is fully boostrapped before load manager register itself URL: https://github.com/apache/pulsar/pull/2935 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 provenanc

[pulsar] branch master updated: Ensure broker is fully boostrapped before load manager register itself (#2935)

2018-11-20 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 7efce98 Ensure broker is fully boostrapped befor

[GitHub] sijie commented on issue #2936: [io][docs] introduce annotations for generating connector yaml config files

2018-11-20 Thread GitBox
sijie commented on issue #2936: [io][docs] introduce annotations for generating connector yaml config files URL: https://github.com/apache/pulsar/pull/2936#issuecomment-440413844 retest this please This is an automated messag

[GitHub] sijie commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint

2018-11-20 Thread GitBox
sijie commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint URL: https://github.com/apache/pulsar/pull/2943#issuecomment-440414222 @jiazhai can you fix the license issue? ``` + src/check-binary-license ./distribution/server/target/apache-pulsar

[GitHub] sijie commented on issue #2976: add debezium source documentation

2018-11-20 Thread GitBox
sijie commented on issue #2976: add debezium source documentation URL: https://github.com/apache/pulsar/pull/2976#issuecomment-440414545 run java8 tests This is an automated message from the Apache Git Service. To respond to t

[GitHub] sijie commented on issue #2953: Avoid debug noise by consumer ack-tracker

2018-11-20 Thread GitBox
sijie commented on issue #2953: Avoid debug noise by consumer ack-tracker URL: https://github.com/apache/pulsar/pull/2953#issuecomment-440416467 run java8 tests This is an automated message from the Apache Git Service. To resp

[GitHub] cbornet commented on issue #271: Consider using gRPC as an externally exposed API

2018-11-20 Thread GitBox
cbornet commented on issue #271: Consider using gRPC as an externally exposed API URL: https://github.com/apache/pulsar/issues/271#issuecomment-440419313 Yes. That's a very good reason indeed. Maybe in the future 😄 . I can understand there are bigger priorities. --

[GitHub] merlimat commented on issue #3027: Fix C++ tests after merge

2018-11-20 Thread GitBox
merlimat commented on issue #3027: Fix C++ tests after merge URL: https://github.com/apache/pulsar/pull/3027#issuecomment-440465698 run cpp tests run integration tests This is an automated message from the Apache Git Servic

[GitHub] jai1 commented on issue #3027: Fix C++ tests after merge

2018-11-20 Thread GitBox
jai1 commented on issue #3027: Fix C++ tests after merge URL: https://github.com/apache/pulsar/pull/3027#issuecomment-440487063 run integration tests This is an automated message from the Apache Git Service. To respond

[GitHub] sijie opened a new pull request #3028: [state] use closeAsync to close state storage client

2018-11-20 Thread GitBox
sijie opened a new pull request #3028: [state] use closeAsync to close state storage client URL: https://github.com/apache/pulsar/pull/3028 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] sijie opened a new pull request #3029: [state] make setting and opening state table more robust

2018-11-20 Thread GitBox
sijie opened a new pull request #3029: [state] make setting and opening state table more robust URL: https://github.com/apache/pulsar/pull/3029 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] massakam commented on issue #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
massakam commented on issue #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024#issuecomment-440502578 @merlimat I'll try it. My user id is `massakam`. This is an automated messag

[GitHub] srkukarni closed pull request #2985: Update Function Semantics

2018-11-20 Thread GitBox
srkukarni closed pull request #2985: Update Function Semantics URL: https://github.com/apache/pulsar/pull/2985 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 (fro

[pulsar] branch master updated: Update Function Semantics (#2985)

2018-11-20 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 1b0589b Update Function Semantics (#2985) 1b

[GitHub] jiazhai commented on issue #2976: add debezium source documentation

2018-11-20 Thread GitBox
jiazhai commented on issue #2976: add debezium source documentation URL: https://github.com/apache/pulsar/pull/2976#issuecomment-440527846 run java8 tests This is an automated message from the Apache Git Service. To respond to

[GitHub] sijie edited a comment on issue #2976: add debezium source documentation

2018-11-20 Thread GitBox
sijie edited a comment on issue #2976: add debezium source documentation URL: https://github.com/apache/pulsar/pull/2976#issuecomment-440414545 run java8 tests ``` org.apache.pulsar.broker.admin.AdminApiOffloadTest.testOffloadV1 org.apache.pulsar.functions.worker.PulsarWorkerAssign

[GitHub] massakam commented on issue #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
massakam commented on issue #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024#issuecomment-440531783 Are the images that need to be pushed these? ``` apachepulsar/pulsar-build:manylinux-cp27-cp27mu apachepulsar/pulsar-build:manylinux-

[GitHub] jerrypeng commented on issue #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
jerrypeng commented on issue #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#issuecomment-440535910 rerun cpp tests This is an automated message from the Apache Git Service. To res

[GitHub] jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint

2018-11-20 Thread GitBox
jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint URL: https://github.com/apache/pulsar/pull/2943#issuecomment-440537067 rerun integration tests This is an automated message from

[GitHub] jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint

2018-11-20 Thread GitBox
jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint URL: https://github.com/apache/pulsar/pull/2943#issuecomment-440537388 rerun cpp tests This is an automated message from the Apa

[GitHub] sijie commented on issue #2936: [io][docs] introduce annotations for generating connector yaml config files

2018-11-20 Thread GitBox
sijie commented on issue #2936: [io][docs] introduce annotations for generating connector yaml config files URL: https://github.com/apache/pulsar/pull/2936#issuecomment-440540434 run cpp tests run integration tests This is

[GitHub] merlimat closed pull request #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
merlimat closed pull request #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024 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

[GitHub] merlimat commented on issue #3024: Enable SSL of LibCurl linked to C++ library

2018-11-20 Thread GitBox
merlimat commented on issue #3024: Enable SSL of LibCurl linked to C++ library URL: https://github.com/apache/pulsar/pull/3024#issuecomment-440546553 That looks correct and you should have access now. This is an automated mess

[pulsar] branch master updated: Enable SSL of LibCurl linked to C++ library (#3024)

2018-11-20 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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 8e7e335 Enable SSL of LibCurl linked to C++ lib

[GitHub] jerrypeng commented on issue #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
jerrypeng commented on issue #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#issuecomment-440549694 rerun cpp tests This is an automated message from the Apache Git Service. To res

[GitHub] jerrypeng commented on issue #3021: adding windowed metrics for functions

2018-11-20 Thread GitBox
jerrypeng commented on issue #3021: adding windowed metrics for functions URL: https://github.com/apache/pulsar/pull/3021#issuecomment-440560127 rerun cpp tests This is an automated message from the Apache Git Service. To res

[GitHub] jiazhai commented on issue #3027: Fix C++ tests after merge

2018-11-20 Thread GitBox
jiazhai commented on issue #3027: Fix C++ tests after merge URL: https://github.com/apache/pulsar/pull/3027#issuecomment-440560879 Thanks for the work, just trying to do the same thing. This is an automated message from the A

[GitHub] jiazhai closed pull request #3027: Fix C++ tests after merge

2018-11-20 Thread GitBox
jiazhai closed pull request #3027: Fix C++ tests after merge URL: https://github.com/apache/pulsar/pull/3027 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

[pulsar] branch master updated: Fix C++ tests after merge (#3027)

2018-11-20 Thread zhaijia
This is an automated email from the ASF dual-hosted git repository. zhaijia pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new fae43a9 Fix C++ tests after merge (#3027) fae4

[GitHub] codelipenghui opened a new issue #3030: Expose Redelivery Count By Message Properties

2018-11-20 Thread GitBox
codelipenghui opened a new issue #3030: Expose Redelivery Count By Message Properties URL: https://github.com/apache/pulsar/issues/3030 ### Motivation Use redelivery count as a solution for delayed message delivery feature. For example, set ackTimeout 30 seconds in consumer, i

[GitHub] jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint

2018-11-20 Thread GitBox
jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint URL: https://github.com/apache/pulsar/pull/2943#issuecomment-440566864 rerun cpp tests This is an automated message from the Apa

[GitHub] jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint

2018-11-20 Thread GitBox
jiazhai commented on issue #2943: Issue #2657: change function cli getstate to use REST endpoint URL: https://github.com/apache/pulsar/pull/2943#issuecomment-440566888 rerun integration tests This is an automated message from

[GitHub] codelipenghui commented on issue #2375: Support for delayed message delivery

2018-11-20 Thread GitBox
codelipenghui commented on issue #2375: Support for delayed message delivery URL: https://github.com/apache/pulsar/issues/2375#issuecomment-440567063 Delayed message redelivery feature based on expose redelivery count to client. #3030 --

[GitHub] merlimat closed pull request #3025: Increase tests timeout from 2 to 5min

2018-11-20 Thread GitBox
merlimat closed pull request #3025: Increase tests timeout from 2 to 5min URL: https://github.com/apache/pulsar/pull/3025 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 r

[GitHub] codelipenghui commented on issue #3030: Expose Redelivery Count By Message Properties

2018-11-20 Thread GitBox
codelipenghui commented on issue #3030: Expose Redelivery Count By Message Properties URL: https://github.com/apache/pulsar/issues/3030#issuecomment-440567413 A solution for #2375 (delayed message delivery) This is an automat

[GitHub] codelipenghui edited a comment on issue #2375: Support for delayed message delivery

2018-11-20 Thread GitBox
codelipenghui edited a comment on issue #2375: Support for delayed message delivery URL: https://github.com/apache/pulsar/issues/2375#issuecomment-440567063 Delayed message redelivery solution based on expose redelivery count to client. #3030 -

[GitHub] merlimat commented on issue #3023: Moved broker aspectj to zk-utils

2018-11-20 Thread GitBox
merlimat commented on issue #3023: Moved broker aspectj to zk-utils URL: https://github.com/apache/pulsar/pull/3023#issuecomment-440567999 > so, what was the exact issue by keeping aj file under broker? Everything gets compiled through the AspectJ compiler wrapper. One example of th