[GitHub] zhaijack opened a new pull request #1359: Delete PartitionedConsumerImpl, use TopicsConsumerImpl instead

2018-03-07 Thread GitBox
zhaijack opened a new pull request #1359: Delete PartitionedConsumerImpl, use TopicsConsumerImpl instead URL: https://github.com/apache/incubator-pulsar/pull/1359 ### Motivation Currently we have both TopicsConsumerImpl and PartitionedConsumerImpl. The behaviour and code of them are

[GitHub] jai1 commented on issue #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
jai1 commented on issue #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#issuecomment-371403998 retest this please This is an automated message from the Apache Git S

[GitHub] zhaijack commented on issue #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx

2018-03-07 Thread GitBox
zhaijack commented on issue #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx URL: https://github.com/apache/incubator-pulsar/pull/1354#issuecomment-371340762 retest this please T

[GitHub] zhaijack commented on issue #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx

2018-03-07 Thread GitBox
zhaijack commented on issue #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx URL: https://github.com/apache/incubator-pulsar/pull/1354#issuecomment-371397512 retest this please T

[GitHub] jai1 commented on a change in pull request #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
jai1 commented on a change in pull request #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#discussion_r173070365 ## File path: pulsar-client-cpp/lib/MessageIdImpl.h ## @@ -0,0 +1,36 @@ +/** + * Licensed to the A

[GitHub] jai1 commented on issue #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
jai1 commented on issue #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#issuecomment-371388067 retest this please This is an automated message from the Apache Git S

[GitHub] yush1ga commented on issue #1352: Delete inactive subscriptions automatically

2018-03-07 Thread GitBox
yush1ga commented on issue #1352: Delete inactive subscriptions automatically URL: https://github.com/apache/incubator-pulsar/pull/1352#issuecomment-371378494 retest this please This is an automated message from the Apache Git

[GitHub] yush1ga commented on issue #1352: Delete inactive subscriptions automatically

2018-03-07 Thread GitBox
yush1ga commented on issue #1352: Delete inactive subscriptions automatically URL: https://github.com/apache/incubator-pulsar/pull/1352#issuecomment-371378494 retest this please This is an automated message from the Apache Git

[GitHub] zhaijack commented on issue #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx

2018-03-07 Thread GitBox
zhaijack commented on issue #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx URL: https://github.com/apache/incubator-pulsar/pull/1354#issuecomment-371340762 retest this please T

[GitHub] zhaijack opened a new pull request #1358: Issue 1071: add ratelimiter for subscription

2018-03-07 Thread GitBox
zhaijack opened a new pull request #1358: Issue 1071: add ratelimiter for subscription URL: https://github.com/apache/incubator-pulsar/pull/1358 ### Motivation Currently rate limiting and throttling are applied at namespace/topic level. It would be good to provide a mechanism to app

[GitHub] srkukarni commented on a change in pull request #1357: Pulsar Functions documentation follow-up

2018-03-07 Thread GitBox
srkukarni commented on a change in pull request #1357: Pulsar Functions documentation follow-up URL: https://github.com/apache/incubator-pulsar/pull/1357#discussion_r173002324 ## File path: site/docs/latest/functions/quickstart.md ## @@ -1,29 +1,176 @@ --- title: Getting

[GitHub] srkukarni commented on a change in pull request #1357: Pulsar Functions documentation follow-up

2018-03-07 Thread GitBox
srkukarni commented on a change in pull request #1357: Pulsar Functions documentation follow-up URL: https://github.com/apache/incubator-pulsar/pull/1357#discussion_r172999405 ## File path: site/docs/latest/functions/api.md ## @@ -2,9 +2,70 @@ title: The Pulsar Functions

[GitHub] srkukarni commented on a change in pull request #1357: Pulsar Functions documentation follow-up

2018-03-07 Thread GitBox
srkukarni commented on a change in pull request #1357: Pulsar Functions documentation follow-up URL: https://github.com/apache/incubator-pulsar/pull/1357#discussion_r172999261 ## File path: site/docs/latest/functions/api.md ## @@ -2,9 +2,70 @@ title: The Pulsar Functions

[GitHub] mgodave commented on issue #1319: Schema registry (2/N)

2018-03-07 Thread GitBox
mgodave commented on issue #1319: Schema registry (2/N) URL: https://github.com/apache/incubator-pulsar/pull/1319#issuecomment-371300300 Ping This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] lucperkins opened a new pull request #1357: Pulsar Functions documentation follow-up

2018-03-07 Thread GitBox
lucperkins opened a new pull request #1357: Pulsar Functions documentation follow-up URL: https://github.com/apache/incubator-pulsar/pull/1357 This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] merlimat commented on issue #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
merlimat commented on issue #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#issuecomment-371297087 @jai1 Addressed comments This is an automated message from the Ap

[GitHub] merlimat commented on issue #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
merlimat commented on issue #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#issuecomment-371296027 retest this please This is an automated message from the Apache G

[GitHub] merlimat commented on issue #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
merlimat commented on issue #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#issuecomment-371296027 retest this please This is an automated message from the Apache G

[GitHub] sijie closed pull request #1356: Log Topic for Functions

2018-03-07 Thread GitBox
sijie closed pull request #1356: Log Topic for Functions URL: https://github.com/apache/incubator-pulsar/pull/1356 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

[incubator-pulsar] branch master updated: Log Topic for Functions (#1356)

2018-03-07 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 6b24ab9 Log Topic for Functions (#1356

[GitHub] mgodave closed pull request #1355: Remove as many Thread.sleep calls from REST endpoint

2018-03-07 Thread GitBox
mgodave closed pull request #1355: Remove as many Thread.sleep calls from REST endpoint URL: https://github.com/apache/incubator-pulsar/pull/1355 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

[GitHub] srkukarni commented on issue #1356: Log Topic for Functions

2018-03-07 Thread GitBox
srkukarni commented on issue #1356: Log Topic for Functions URL: https://github.com/apache/incubator-pulsar/pull/1356#issuecomment-371250939 @sijie @merlimat This is an automated message from the Apache Git Service. To respon

[GitHub] srkukarni opened a new pull request #1356: Log Topic for Functions

2018-03-07 Thread GitBox
srkukarni opened a new pull request #1356: Log Topic for Functions URL: https://github.com/apache/incubator-pulsar/pull/1356 ### Motivation Added ability to specify Log Topic where all logging done in a function go. This is often times a very useful way to debug functions

[GitHub] mgodave commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint

2018-03-07 Thread GitBox
mgodave commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint URL: https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371242117 Feel free to close it. On Mar 7, 2018 11:43, "Matteo Merli" wrote: > The point is, sleeping a thread in the

[GitHub] merlimat commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint

2018-03-07 Thread GitBox
merlimat commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint URL: https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371240890 > The point is, sleeping a thread in the REST endpoint is a resource hog and generally not a great idea. I'm not

[GitHub] mgodave commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint

2018-03-07 Thread GitBox
mgodave commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint URL: https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371238796 I *think* I see what you are saying. Either way, scheduling a return after the sync timeout would be a better option. ---

[GitHub] mgodave commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint

2018-03-07 Thread GitBox
mgodave commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint URL: https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371238039 The point is, sleeping a thread in the REST endpoint is a resource hog and generally not a great idea. --

[GitHub] merlimat commented on a change in pull request #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
merlimat commented on a change in pull request #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#discussion_r172939330 ## File path: pulsar-client-cpp/include/pulsar/MessageId.h ## @@ -50,34 +47,44 @@ class Messag

[incubator-pulsar] branch master updated: CompactedTopic should seek to position of cursor, not next position (#1336)

2018-03-07 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 dad679a CompactedTopic should seek to

[GitHub] merlimat closed pull request #1336: CompactedTopic should seek to position of cursor, not next position

2018-03-07 Thread GitBox
merlimat closed pull request #1336: CompactedTopic should seek to position of cursor, not next position URL: https://github.com/apache/incubator-pulsar/pull/1336 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

[GitHub] merlimat commented on a change in pull request #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
merlimat commented on a change in pull request #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#discussion_r172937375 ## File path: pulsar-client-cpp/lib/MessageId.cc ## @@ -30,43 +29,51 @@ namespace pulsar {

[GitHub] merlimat commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint

2018-03-07 Thread GitBox
merlimat commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint URL: https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371232050 @mgodave This change doesn't solve the problem that the ugly `Thread.sleep()` was introduced as a workaround for.

[GitHub] merlimat commented on a change in pull request #1322: Use private impl for MessageId in c++ client

2018-03-07 Thread GitBox
merlimat commented on a change in pull request #1322: Use private impl for MessageId in c++ client URL: https://github.com/apache/incubator-pulsar/pull/1322#discussion_r172932307 ## File path: pulsar-client-cpp/lib/MessageId.cc ## @@ -30,43 +29,51 @@ namespace pulsar {

[GitHub] mgodave opened a new pull request #1355: Remove as many Thread.sleep calls from REST endpoint

2018-03-07 Thread GitBox
mgodave opened a new pull request #1355: Remove as many Thread.sleep calls from REST endpoint URL: https://github.com/apache/incubator-pulsar/pull/1355 Thread.sleep is rarely the correct answer as it can lead to race conditions and incorrect results. In this case sleeping the tread will ti

[GitHub] ivankelly commented on issue #1145: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice

2018-03-07 Thread GitBox
ivankelly commented on issue #1145: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice URL: https://github.com/apache/incubator-pulsar/issues/1145#issuecomment-371220837 Found the race. The builder stuff messed up this test completely, but the race is sti

[GitHub] ivankelly commented on issue #1145: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice

2018-03-07 Thread GitBox
ivankelly commented on issue #1145: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice URL: https://github.com/apache/incubator-pulsar/issues/1145#issuecomment-371209316 There any logs beside that snippet?

[GitHub] maskit commented on issue #1266: Pass all Apache Podling Website Checks

2018-03-07 Thread GitBox
maskit commented on issue #1266: Pass all Apache Podling Website Checks URL: https://github.com/apache/incubator-pulsar/issues/1266#issuecomment-371205912 Generated a 212px width PNG logo image from the SVG in site/img/ that satisfies the requirement[1]. ![pulsar](https://user-imag

[GitHub] zhaijack opened a new pull request #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx

2018-03-07 Thread GitBox
zhaijack opened a new pull request #1354: Issue 1118: refine handlerBase to let only ConsumerImpl and ProducerImpl have client-cnx URL: https://github.com/apache/incubator-pulsar/pull/1354 ### Motivation class HandlerBase contains 2 parts: - handler state; this is need for all h

[GitHub] ivankelly commented on issue #1145: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice

2018-03-07 Thread GitBox
ivankelly commented on issue #1145: Intermittent test failure in PersistentTopicTest.testClosingReplicationProducerTwice URL: https://github.com/apache/incubator-pulsar/issues/1145#issuecomment-371188779 This is a dupe of #617 -

[GitHub] maskit commented on issue #1258: Fix CID 262329

2018-03-07 Thread GitBox
maskit commented on issue #1258: Fix CID 262329 URL: https://github.com/apache/incubator-pulsar/pull/1258#issuecomment-371185764 retest this please This is an automated message from the Apache Git Service. To respond to the me

[incubator-pulsar] branch master updated: Have the ability to send log messages to a topic in Python (#1353)

2018-03-07 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 a8a595d Have the ability to send log

[GitHub] merlimat closed pull request #1353: Have the ability to send log messages to a topic in Python

2018-03-07 Thread GitBox
merlimat closed pull request #1353: Have the ability to send log messages to a topic in Python URL: https://github.com/apache/incubator-pulsar/pull/1353 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 provenan

[GitHub] XiaoZYang commented on issue #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-03-07 Thread GitBox
XiaoZYang commented on issue #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1219#issuecomment-371149305 ping @merlimat Thanks for making a review on this commit. Since t

[GitHub] XiaoZYang commented on issue #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-03-07 Thread GitBox
XiaoZYang commented on issue #1219: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1219#issuecomment-371149305 ping @merlimat Since the earliest position should be the first messa

[GitHub] zhaijack closed issue #1237: Make subscribe for TopicsConsumer using builder

2018-03-07 Thread GitBox
zhaijack closed issue #1237: Make subscribe for TopicsConsumer using builder URL: https://github.com/apache/incubator-pulsar/issues/1237 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] yush1ga commented on a change in pull request #1352: Delete inactive subscriptions automatically

2018-03-07 Thread GitBox
yush1ga commented on a change in pull request #1352: Delete inactive subscriptions automatically URL: https://github.com/apache/incubator-pulsar/pull/1352#discussion_r172765485 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java ##

[GitHub] yush1ga commented on issue #1352: Delete inactive subscriptions automatically

2018-03-07 Thread GitBox
yush1ga commented on issue #1352: Delete inactive subscriptions automatically URL: https://github.com/apache/incubator-pulsar/pull/1352#issuecomment-371057712 @merlimat > Another question is: how can this be enforced across brokers restarts? Did you mean that a scheduler for inact