[GitHub] [pulsar] wolfstudy commented on issue #4366: [cpp client]support key shared for cpp client

2019-05-24 Thread GitBox
wolfstudy commented on issue #4366: [cpp client]support key shared for cpp client URL: https://github.com/apache/pulsar/pull/4366#issuecomment-495871391 run integration tests This is an automated message from the Apache Git S

[GitHub] [pulsar] jerrypeng commented on issue #4363: renaming sinks and sources api to be consistent with the rest of Pulsar

2019-05-24 Thread GitBox
jerrypeng commented on issue #4363: renaming sinks and sources api to be consistent with the rest of Pulsar URL: https://github.com/apache/pulsar/pull/4363#issuecomment-495863006 rerun java8 tests This is an automated messag

[GitHub] [pulsar] wolfstudy commented on issue #4348: Support max message size for cpp and go client

2019-05-24 Thread GitBox
wolfstudy commented on issue #4348: Support max message size for cpp and go client URL: https://github.com/apache/pulsar/pull/4348#issuecomment-495860536 @merlimat @sijie @jiazhai @zymap can you review this again, thks. This

[GitHub] [pulsar] jiazhai commented on issue #4331: Feature - reset cursor on Reader to current position

2019-05-24 Thread GitBox
jiazhai commented on issue #4331: Feature - reset cursor on Reader to current position URL: https://github.com/apache/pulsar/pull/4331#issuecomment-495857202 @lovelle As @merlimat suggested, Would you please also help add a test parameter for batch/non-batch, this is a setting for producer

[GitHub] [pulsar] tuteng opened a new pull request #4367: [document][swagger]Update topics rest api document

2019-05-24 Thread GitBox
tuteng opened a new pull request #4367: [document][swagger]Update topics rest api document URL: https://github.com/apache/pulsar/pull/4367 ### Motivation The current rest api document parameters are missing descriptions and some parameters are missing. ### Modifications

[GitHub] [pulsar] wolfstudy commented on issue #4366: [cpp client]support key shared for cpp client

2019-05-24 Thread GitBox
wolfstudy commented on issue #4366: [cpp client]support key shared for cpp client URL: https://github.com/apache/pulsar/pull/4366#issuecomment-495846378 @codelipenghui @jiazhai @merlimat @sijie PTAL This is an automated messa

[GitHub] [pulsar] wolfstudy opened a new pull request #4366: [cpp client]support key shared for cpp client

2019-05-24 Thread GitBox
wolfstudy opened a new pull request #4366: [cpp client]support key shared for cpp client URL: https://github.com/apache/pulsar/pull/4366 Signed-off-by: xiaolong.ran ### Motivation support key_shared for pulsar-cpp-client --

[GitHub] [pulsar] jiazhai commented on issue #4365: Authentication provider getAuthData called before initialization

2019-05-24 Thread GitBox
jiazhai commented on issue #4365: Authentication provider getAuthData called before initialization URL: https://github.com/apache/pulsar/issues/4365#issuecomment-495845162 seems the code is added in PR #3550, indeed called getAuthData before start https://user-images.githubusercontent.co

[GitHub] [pulsar] jiazhai commented on issue #4306: [pulsar-proxy] Clean up deprecated Zookeeper client

2019-05-24 Thread GitBox
jiazhai commented on issue #4306: [pulsar-proxy] Clean up deprecated Zookeeper client URL: https://github.com/apache/pulsar/pull/4306#issuecomment-495838938 run java8 tests This is an automated message from the Apache Git Ser

[GitHub] [pulsar] jiazhai commented on issue #4345: Intermittent test failure in ReplicatorRateLimiterTest

2019-05-24 Thread GitBox
jiazhai commented on issue #4345: Intermittent test failure in ReplicatorRateLimiterTest URL: https://github.com/apache/pulsar/issues/4345#issuecomment-49583 @merlimat looked at the log. not able to find any clue. It is a timeout in setup(), except setting the timeout to 30s, Replicato

[GitHub] [pulsar] jerrypeng commented on issue #4363: renaming sinks and sources api to be consistent with the rest of Pulsar

2019-05-24 Thread GitBox
jerrypeng commented on issue #4363: renaming sinks and sources api to be consistent with the rest of Pulsar URL: https://github.com/apache/pulsar/pull/4363#issuecomment-495836569 rerun java8 tests This is an automated messag

[GitHub] [pulsar] tuteng commented on issue #4358: [pulsar-io][jdbc sink]Support delete and update event for JDBC Sink

2019-05-24 Thread GitBox
tuteng commented on issue #4358: [pulsar-io][jdbc sink]Support delete and update event for JDBC Sink URL: https://github.com/apache/pulsar/pull/4358#issuecomment-495835182 run Integration Tests This is an automated message fr

[GitHub] [pulsar] tuteng commented on issue #4358: [pulsar-io][jdbc sink]Support delete and update event for JDBC Sink

2019-05-24 Thread GitBox
tuteng commented on issue #4358: [pulsar-io][jdbc sink]Support delete and update event for JDBC Sink URL: https://github.com/apache/pulsar/pull/4358#issuecomment-495834953 run integraton tests This is an automated message fro

[GitHub] [pulsar] jiazhai edited a comment on issue #4357: tenants list not match between brokers

2019-05-24 Thread GitBox
jiazhai edited a comment on issue #4357: tenants list not match between brokers URL: https://github.com/apache/pulsar/issues/4357#issuecomment-495462403 A possible that will cause this isssue is zk disconnection and in-sync. -

[GitHub] [pulsar] jiazhai commented on issue #4362: Fix C++ consumer unsubscribe when regex doesn't match

2019-05-24 Thread GitBox
jiazhai commented on issue #4362: Fix C++ consumer unsubscribe when regex doesn't match URL: https://github.com/apache/pulsar/pull/4362#issuecomment-495834501 run integration tests This is an automated message from the Apache

[GitHub] [pulsar] sijie commented on issue #4151: (WIP) Pulsar SQL supports pulsar's primitive schemas

2019-05-24 Thread GitBox
sijie commented on issue #4151: (WIP) Pulsar SQL supports pulsar's primitive schemas URL: https://github.com/apache/pulsar/pull/4151#issuecomment-495832896 @jerrypeng write the unit tests and integration tests. and also addressed your comment. -

[GitHub] [pulsar] wolfstudy commented on issue #4348: Support max message size for cpp and go client

2019-05-24 Thread GitBox
wolfstudy commented on issue #4348: Support max message size for cpp and go client URL: https://github.com/apache/pulsar/pull/4348#issuecomment-495828594 run java8 tests This is an automated message from the Apache Git Servic

[GitHub] [pulsar] hapihu commented on issue #4350: Not enough arguments for Logger's method

2019-05-24 Thread GitBox
hapihu commented on issue #4350: Not enough arguments for Logger's method URL: https://github.com/apache/pulsar/issues/4350#issuecomment-495820871 This issue has been resolved. https://github.com/apache/pulsar/pull/4351

[GitHub] [pulsar] hapihu closed issue #4350: Not enough arguments for Logger's method

2019-05-24 Thread GitBox
hapihu closed issue #4350: Not enough arguments for Logger's method URL: https://github.com/apache/pulsar/issues/4350 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [pulsar] jerrypeng commented on issue #4151: (WIP) Pulsar SQL supports pulsar's primitive schemas

2019-05-24 Thread GitBox
jerrypeng commented on issue #4151: (WIP) Pulsar SQL supports pulsar's primitive schemas URL: https://github.com/apache/pulsar/pull/4151#issuecomment-495820322 @sijie what else are you planning on including in this PR? This i

[GitHub] [pulsar] srkukarni merged pull request #4364: Expose state to sources and sinks

2019-05-24 Thread GitBox
srkukarni merged pull request #4364: Expose state to sources and sinks URL: https://github.com/apache/pulsar/pull/4364 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[pulsar] branch master updated: Expose state to sources and sinks (#4364)

2019-05-24 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 f8349e2 Expose state to sources and sinks (#

[GitHub] [pulsar] jerrypeng commented on issue #4363: renaming sinks and sources api to be consistent with the rest of Pulsar

2019-05-24 Thread GitBox
jerrypeng commented on issue #4363: renaming sinks and sources api to be consistent with the rest of Pulsar URL: https://github.com/apache/pulsar/pull/4363#issuecomment-495814012 rerun java8 tests This is an automated messag

[pulsar] branch master updated: shade additional dependencies in java-instance.jar (#4356)

2019-05-24 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 7cadc93 shade additional dependencies in jav

[GitHub] [pulsar] jerrypeng merged pull request #4356: shade additional dependencies in java-instance.jar

2019-05-24 Thread GitBox
jerrypeng merged pull request #4356: shade additional dependencies in java-instance.jar URL: https://github.com/apache/pulsar/pull/4356 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [pulsar] jerrypeng commented on a change in pull request #4196: Configure static PulsarByteBufAllocator to handle OOM errors

2019-05-24 Thread GitBox
jerrypeng commented on a change in pull request #4196: Configure static PulsarByteBufAllocator to handle OOM errors URL: https://github.com/apache/pulsar/pull/4196#discussion_r287537503 ## File path: pulsar-common/pom.xml ## @@ -74,6 +74,11 @@ netty-all +

[GitHub] [pulsar] srkukarni commented on issue #4364: Expose state to sources and sinks

2019-05-24 Thread GitBox
srkukarni commented on issue #4364: Expose state to sources and sinks URL: https://github.com/apache/pulsar/pull/4364#issuecomment-495810786 run integration tests This is an automated message from the Apache Git Service. To r

[pulsar] branch master updated: Fixed the display of schema information (#4318)

2019-05-24 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 5d688ee Fixed the display of schema informat

[GitHub] [pulsar] srkukarni merged pull request #4318: Fixed the display of schema information

2019-05-24 Thread GitBox
srkukarni merged pull request #4318: Fixed the display of schema information URL: https://github.com/apache/pulsar/pull/4318 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [pulsar] sleungtoast commented on issue #4365: Authentication provider getAuthData called before initialization

2019-05-24 Thread GitBox
sleungtoast commented on issue #4365: Authentication provider getAuthData called before initialization URL: https://github.com/apache/pulsar/issues/4365#issuecomment-495797120 Sure thing ``` Hit NPE in our CustomPulsarAuthentication.getAuthData() ! at org.apache.pulsar.client.impl

[pulsar] branch master updated: [pulsar-broker] When calling Logger's method, set the appropriate parameters.(#4350) (#4351)

2019-05-24 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 27eb5ca [pulsar-broker] When calling Logger's m

[GitHub] [pulsar] merlimat merged pull request #4351: [pulsar-broker] When calling Logger's method, set the appropriate parameters.(#4350)

2019-05-24 Thread GitBox
merlimat merged pull request #4351: [pulsar-broker] When calling Logger's method, set the appropriate parameters.(#4350) URL: https://github.com/apache/pulsar/pull/4351 This is an automated message from the Apache Git Servic

[GitHub] [pulsar] merlimat edited a comment on issue #4365: Authentication provider getAuthData called before initialization

2019-05-24 Thread GitBox
merlimat edited a comment on issue #4365: Authentication provider getAuthData called before initialization URL: https://github.com/apache/pulsar/issues/4365#issuecomment-495792719 @sleungtoast Can you paste the stack trace for the initialization sequence? --

[GitHub] [pulsar] merlimat commented on issue #4365: Authentication provider getAuthData called before initialization

2019-05-24 Thread GitBox
merlimat commented on issue #4365: Authentication provider getAuthData called before initialization URL: https://github.com/apache/pulsar/issues/4365#issuecomment-495792719 @sleungtoast Can you past the stack trace for the initialization sequence? --

[GitHub] [pulsar] jerrypeng commented on issue #4356: shade additional dependencies in java-instance.jar

2019-05-24 Thread GitBox
jerrypeng commented on issue #4356: shade additional dependencies in java-instance.jar URL: https://github.com/apache/pulsar/pull/4356#issuecomment-495791300 rerun java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] srkukarni commented on issue #4364: Expose state to sources and sinks

2019-05-24 Thread GitBox
srkukarni commented on issue #4364: Expose state to sources and sinks URL: https://github.com/apache/pulsar/pull/4364#issuecomment-495791077 run integration tests run java8 tests This is an automated message from the Apache

[GitHub] [pulsar] sleungtoast opened a new issue #4365: Authentication provider getAuthData called before initialization

2019-05-24 Thread GitBox
sleungtoast opened a new issue #4365: Authentication provider getAuthData called before initialization URL: https://github.com/apache/pulsar/issues/4365 **Describe the bug** When I wrote a custom authentication provider, I expected `.getAuthData()` to not be called before `.start()` was

[GitHub] [pulsar] merlimat commented on issue #4062: Delayed message delivery implementation

2019-05-24 Thread GitBox
merlimat commented on issue #4062: Delayed message delivery implementation URL: https://github.com/apache/pulsar/pull/4062#issuecomment-495783703 run java8 tests run integration tests This is an automated message from the A

[GitHub] [pulsar] merlimat commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client

2019-05-24 Thread GitBox
merlimat commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client URL: https://github.com/apache/pulsar/pull/4256#issuecomment-495781806 One more LICENSE check : ``` jetty-alpn-java-client-9.4.11.v20180605.jar mentioned in LICENSE, but not

[GitHub] [pulsar] srkukarni commented on issue #4318: Fixed the display of schema information

2019-05-24 Thread GitBox
srkukarni commented on issue #4318: Fixed the display of schema information URL: https://github.com/apache/pulsar/pull/4318#issuecomment-495777882 run java8 tests This is an automated message from the Apache Git Service. To re

[GitHub] [pulsar] srkukarni commented on issue #4318: Fixed the display of schema information

2019-05-24 Thread GitBox
srkukarni commented on issue #4318: Fixed the display of schema information URL: https://github.com/apache/pulsar/pull/4318#issuecomment-495777218 run integration tests This is an automated message from the Apache Git Service.

[GitHub] [pulsar] jerrypeng commented on issue #4356: shade additional dependencies in java-instance.jar

2019-05-24 Thread GitBox
jerrypeng commented on issue #4356: shade additional dependencies in java-instance.jar URL: https://github.com/apache/pulsar/pull/4356#issuecomment-495765160 rerun java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] srkukarni opened a new pull request #4364: Expose state to sources and sinks

2019-05-24 Thread GitBox
srkukarni opened a new pull request #4364: Expose state to sources and sinks URL: https://github.com/apache/pulsar/pull/4364 ### Modifications There has been an ask from the community of the desire to have state exposed for sources and sinks just like in Functions. This pr addres

[GitHub] [pulsar] jerrypeng commented on issue #4356: shade additional dependencies in java-instance.jar

2019-05-24 Thread GitBox
jerrypeng commented on issue #4356: shade additional dependencies in java-instance.jar URL: https://github.com/apache/pulsar/pull/4356#issuecomment-495753742 rerun java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] jerrypeng opened a new pull request #4363: renaming sinks and sources api to be consistent with the rest of Pulsar

2019-05-24 Thread GitBox
jerrypeng opened a new pull request #4363: renaming sinks and sources api to be consistent with the rest of Pulsar URL: https://github.com/apache/pulsar/pull/4363 ### Motivation Users have complained that the sinks and sources REST and admin API is not consistent with the rest

[GitHub] [pulsar] BJTiso commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics

2019-05-24 Thread GitBox
BJTiso commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics URL: https://github.com/apache/pulsar/issues/4361#issuecomment-495747360 @merlimat many thanks! This is an automat

[GitHub] [pulsar] merlimat commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics

2019-05-24 Thread GitBox
merlimat commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics URL: https://github.com/apache/pulsar/issues/4361#issuecomment-495746490 @BJTiso Added fix (though the issue it was on the underlying C++ lib ) -

[GitHub] [pulsar] merlimat opened a new pull request #4362: Fix C++ consumer unsubscribe when regex doesn't match

2019-05-24 Thread GitBox
merlimat opened a new pull request #4362: Fix C++ consumer unsubscribe when regex doesn't match URL: https://github.com/apache/pulsar/pull/4362 ### Motivation In C++ client the consumer.unsubscribe() hangs if the topic regex gave an empty list of topics. Fix #4361 -

[GitHub] [pulsar] merlimat commented on a change in pull request #4331: Feature - reset cursor on Reader to current position

2019-05-24 Thread GitBox
merlimat commented on a change in pull request #4331: Feature - reset cursor on Reader to current position URL: https://github.com/apache/pulsar/pull/4331#discussion_r287461921 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/client/api/TopicReaderTest.java ##

[GitHub] [pulsar] merlimat commented on issue #4354: Replicated subscriptions - Create snapshots

2019-05-24 Thread GitBox
merlimat commented on issue #4354: Replicated subscriptions - Create snapshots URL: https://github.com/apache/pulsar/pull/4354#issuecomment-495732768 @zymap that's indeed 33 :) . Fixed the description, thanks This is an automa

[GitHub] [pulsar] merlimat commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics

2019-05-24 Thread GitBox
merlimat commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics URL: https://github.com/apache/pulsar/issues/4361#issuecomment-495728913 Got it.. I think there should be an easy fix ---

[pulsar-client-go] branch merlimat-patch-1 deleted (was 694ad77)

2019-05-24 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch merlimat-patch-1 in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git. was 694ad77 Removed non-needed section in LICENSE The revisions that were on this branch are still

[GitHub] [pulsar] BJTiso edited a comment on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics

2019-05-24 Thread GitBox
BJTiso edited a comment on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics URL: https://github.com/apache/pulsar/issues/4361#issuecomment-495726155 When I reproduce, no topics exist, in fact ```curl localhost:8080/admin/v2/persistent/public/defaul

[GitHub] [pulsar] BJTiso commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics

2019-05-24 Thread GitBox
BJTiso commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics URL: https://github.com/apache/pulsar/issues/4361#issuecomment-495726155 When I reproduce, the no topics exist, in fact ```curl localhost:8080/admin/v2/persistent/public/default``

[GitHub] [pulsar] merlimat commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics

2019-05-24 Thread GitBox
merlimat commented on issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics URL: https://github.com/apache/pulsar/issues/4361#issuecomment-495721958 @BJTiso are the topics existing or the regex leads to an empty list? ---

[GitHub] [pulsar] BJTiso opened a new issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics

2019-05-24 Thread GitBox
BJTiso opened a new issue #4361: Go client: multi-topic unsubscribe hangs when regex does not match any topics URL: https://github.com/apache/pulsar/issues/4361 **Describe the bug** When performing multi-topic unsubscribe on a topic regex that does not not match any topics, then Unsubsc

[pulsar] branch master updated: Bugfix - release and recycle on discarded messages (#4342)

2019-05-24 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 71f3928 Bugfix - release and recycle on discard

[GitHub] [pulsar] merlimat merged pull request #4342: Bugfix - release and recycle on discarded messages

2019-05-24 Thread GitBox
merlimat merged pull request #4342: Bugfix - release and recycle on discarded messages URL: https://github.com/apache/pulsar/pull/4342 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [pulsar] merlimat commented on issue #4351: [pulsar-broker] When calling Logger's method, set the appropriate parameters.(#4350)

2019-05-24 Thread GitBox
merlimat commented on issue #4351: [pulsar-broker] When calling Logger's method, set the appropriate parameters.(#4350) URL: https://github.com/apache/pulsar/pull/4351#issuecomment-495702180 run java8 tests run integration tests --

[GitHub] [pulsar] merlimat closed issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector

2019-05-24 Thread GitBox
merlimat closed issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector URL: https://github.com/apache/pulsar/issues/4353 This is an automate

[GitHub] [pulsar] merlimat commented on issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector

2019-05-24 Thread GitBox
merlimat commented on issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector URL: https://github.com/apache/pulsar/issues/4353#issuecomment-495694029 s --

[GitHub] [pulsar] merlimat commented on issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector

2019-05-24 Thread GitBox
merlimat commented on issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector URL: https://github.com/apache/pulsar/issues/4353#issuecomment-495693988 Yes, the pending acks needs to be filled in from the same

[GitHub] [pulsar] jiazhai commented on issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector

2019-05-24 Thread GitBox
jiazhai commented on issue #4353: Intermittent test failure int KeySharedSubscriptionTest.testConsumerCrashSendAndReceiveWithHashRangeStickyKeyConsumerSelector URL: https://github.com/apache/pulsar/issues/4353#issuecomment-495664129 @codelipenghui Hi Penghui is this the fix? https://g

[GitHub] [pulsar] liketic commented on issue #4262: Delete schema when deleting inactive topic

2019-05-24 Thread GitBox
liketic commented on issue #4262: Delete schema when deleting inactive topic URL: https://github.com/apache/pulsar/pull/4262#issuecomment-495655151 run java8 tests This is an automated message from the Apache Git Service. To r

[GitHub] [pulsar] merlimat merged pull request #4352: Add messaging integration tests for different subscriptions.

2019-05-24 Thread GitBox
merlimat merged pull request #4352: Add messaging integration tests for different subscriptions. URL: https://github.com/apache/pulsar/pull/4352 This is an automated message from the Apache Git Service. To respond to the mes

[pulsar] branch master updated: Add messaging integration tests for different subscriptions. (#4352)

2019-05-24 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 b161328 Add messaging integration tests for dif

[GitHub] [pulsar] kimcs commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client

2019-05-24 Thread GitBox
kimcs commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client URL: https://github.com/apache/pulsar/pull/4256#issuecomment-495605989 run cpp tests This is an automated message from th

[GitHub] [pulsar] codelipenghui commented on a change in pull request #4360: [schema] Introduce schema data validator

2019-05-24 Thread GitBox
codelipenghui commented on a change in pull request #4360: [schema] Introduce schema data validator URL: https://github.com/apache/pulsar/pull/4360#discussion_r287324415 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/schema/validator/SchemaDataValidator.

[GitHub] [pulsar] kimcs commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client

2019-05-24 Thread GitBox
kimcs commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client URL: https://github.com/apache/pulsar/pull/4256#issuecomment-495585780 run java8 tests This is an automated message from

[GitHub] [pulsar] sijie opened a new pull request #4360: [schema] Introduce schema data validator

2019-05-24 Thread GitBox
sijie opened a new pull request #4360: [schema] Introduce schema data validator URL: https://github.com/apache/pulsar/pull/4360 *Motivation* Currently the schema data is only validated in compatibility checker. If the schema data is uploaded from admin api, there is no validation if

[GitHub] [pulsar] zymap commented on issue #4354: Replicated subscriptions - Create snapshots

2019-05-24 Thread GitBox
zymap commented on issue #4354: Replicated subscriptions - Create snapshots URL: https://github.com/apache/pulsar/pull/4354#issuecomment-495553431 > ### Motivation > 3rd part of changes for PIP-36. Includes the creation of the snapshots when the replicated subscriptions feature is enable

[GitHub] [pulsar] codelipenghui commented on issue #4352: Add messaging integration tests for different subscriptions.

2019-05-24 Thread GitBox
codelipenghui commented on issue #4352: Add messaging integration tests for different subscriptions. URL: https://github.com/apache/pulsar/pull/4352#issuecomment-495551343 run Java8 Tests This is an automated message from the

[GitHub] [pulsar] davidtinker opened a new issue #4359: Proxy doesn't start on machines with <= 2 CPUs with default proxy.conf

2019-05-24 Thread GitBox
davidtinker opened a new issue #4359: Proxy doesn't start on machines with <= 2 CPUs with default proxy.conf URL: https://github.com/apache/pulsar/issues/4359 **Describe the bug** The proxy fails to start with 'Uncaught exception in thread main: Failed to start HTTP server on ports [808

[pulsar] branch asf-site updated: Updated site at revision 009bc29

2019-05-24 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/asf-site by this push: new 4d92a43 Updated site at revision 009bc29 4d

[GitHub] [pulsar] Anonymitaet commented on issue #4338: [Doc] Add *Connect Pulsar to MySQL*

2019-05-24 Thread GitBox
Anonymitaet commented on issue #4338: [Doc] Add *Connect Pulsar to MySQL* URL: https://github.com/apache/pulsar/pull/4338#issuecomment-495518727 run java8 tests This is an automated message from the Apache Git Service. To resp

[GitHub] [pulsar] kimcs commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client

2019-05-24 Thread GitBox
kimcs commented on issue #4256: Use same activation dependency in pulsar-client-admin and pulsar-client URL: https://github.com/apache/pulsar/pull/4256#issuecomment-495513906 The very last commit https://github.com/apache/pulsar/pull/4256/commits/fe36c862e40cdca9bb54351ccdba4154743296e3 i

[GitHub] [pulsar] tuteng opened a new pull request #4358: [pulsar-io sink][jdbc]Support delete and update event

2019-05-24 Thread GitBox
tuteng opened a new pull request #4358: [pulsar-io sink][jdbc]Support delete and update event URL: https://github.com/apache/pulsar/pull/4358 ### Motivation Currently our JDBC Sink not support deletion and update events. Support for delete and update events. ### Modific

[GitHub] [pulsar] ambition119 commented on issue #4347: [ClientAPI]ReaderBuilder.loadConf() not working

2019-05-24 Thread GitBox
ambition119 commented on issue #4347: [ClientAPI]ReaderBuilder.loadConf() not working URL: https://github.com/apache/pulsar/issues/4347#issuecomment-495510994 > Put `startMessageId` before `loadConf` in the builder chain could reproduce it. thanks, I can try solve.

[GitHub] [pulsar] codelipenghui commented on issue #4352: Add messaging integration tests for different subscriptions.

2019-05-24 Thread GitBox
codelipenghui commented on issue #4352: Add messaging integration tests for different subscriptions. URL: https://github.com/apache/pulsar/pull/4352#issuecomment-495508058 run Integration Tests This is an automated message f

[GitHub] [pulsar] jiazhai commented on a change in pull request #4331: Feature - reset cursor on Reader to current position

2019-05-24 Thread GitBox
jiazhai commented on a change in pull request #4331: Feature - reset cursor on Reader to current position URL: https://github.com/apache/pulsar/pull/4331#discussion_r287245543 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/client/api/TopicReaderTest.java ## @

[GitHub] [pulsar] merlimat commented on issue #4352: Add messaging integration tests for different subscriptions.

2019-05-24 Thread GitBox
merlimat commented on issue #4352: Add messaging integration tests for different subscriptions. URL: https://github.com/apache/pulsar/pull/4352#issuecomment-495498331 Thanks for fixing this. It was indeed a regression in my change. -