[GitHub] [pulsar] michaeljmarshall commented on issue #15746: running in docker throws exception

2022-05-24 Thread GitBox
michaeljmarshall commented on issue #15746: URL: https://github.com/apache/pulsar/issues/15746#issuecomment-1136758168 @shibd - just so you know, it looks like #15769 is a proposed solution. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] michaeljmarshall commented on pull request #15769: [Fix][Docker] Add write permissions to /pulsar subdirectories to enable running as non-root user

2022-05-24 Thread GitBox
michaeljmarshall commented on PR #15769: URL: https://github.com/apache/pulsar/pull/15769#issuecomment-1136757327 @drianpauL - thanks for your contribution! I should be able to review this tomorrow. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] merlimat commented on issue #15705: New KeyShared consumers will not get any messages until a consumer that did get messages disconnects or acks/nacks some messages

2022-05-24 Thread GitBox
merlimat commented on issue #15705: URL: https://github.com/apache/pulsar/issues/15705#issuecomment-1136738845 > @codelipenghui thank you for the explanation; that makes sense. > > Two clarifications: > > 1. How does this apply to nacked messages? if a new KeyShared consumer

[GitHub] [pulsar] michaeljmarshall commented on pull request #15763: Support dlog bookie client configuration via pass through config

2022-05-24 Thread GitBox
michaeljmarshall commented on PR #15763: URL: https://github.com/apache/pulsar/pull/15763#issuecomment-1136735304 @Anonymitaet - thank you for your feedback. Making this a draft temporarily. I will include some additional work tomorrow. -- This is an automated message from the

[GitHub] [pulsar] drianpauL commented on pull request #15769: [Fix][Docker] Add write permissions to /pulsar subdirectories to enable running as non-root user

2022-05-24 Thread GitBox
drianpauL commented on PR #15769: URL: https://github.com/apache/pulsar/pull/15769#issuecomment-1136727807 cc @michaeljmarshall for your advice because of your work on #11269 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] mans2singh commented on pull request #15727: [fix][schema-understand][docs] Updated example section

2022-05-24 Thread GitBox
mans2singh commented on PR #15727: URL: https://github.com/apache/pulsar/pull/15727#issuecomment-1136728207 Thanks @Anonymitaet for your time and advice. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [pulsar] drianpauL commented on pull request #15769: [Fix][Docker] Add write permissions to /pulsar subdirectories to enable running as non-root user

2022-05-24 Thread GitBox
drianpauL commented on PR #15769: URL: https://github.com/apache/pulsar/pull/15769#issuecomment-1136726829 This PR will also address #15746 which is effectively a duplicate of #15230 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] drianpauL opened a new pull request, #15769: [Fix][Docker] Add write permissions to /pulsar subdirectories to enable running as non-root user

2022-05-24 Thread GitBox
drianpauL opened a new pull request, #15769: URL: https://github.com/apache/pulsar/pull/15769 Fixes #15230 ### Motivation The Quickstart/[Getting Started in Docker](https://pulsar.apache.org/docs/next/getting-started-docker) example does not work with Pulsar 2.10.0:

[GitHub] [pulsar] Shawyeok commented on pull request #15766: [fix][broker]Sync topicPublishRateLimiter update

2022-05-24 Thread GitBox
Shawyeok commented on PR #15766: URL: https://github.com/apache/pulsar/pull/15766#issuecomment-1136718087 > Thread1: topicPublishRateLimiter is null, and and it will not invoke close in line:1216. Then it stop before line:1219 Thread2: set topicPublishRateLimiter a new value, for

[GitHub] [pulsar] mingyifei opened a new issue, #15768: [ProtocolHandler] It is hoped that the compiled class directory can be generated for the NAR file

2022-05-24 Thread GitBox
mingyifei opened a new issue, #15768: URL: https://github.com/apache/pulsar/issues/15768 When there is a problem with AOP, Kop or ROP, it is difficult to troubleshoot, and the unit test scenario has limitations. Therefore, we hope to provide a way to generate class directory, which can be

[GitHub] [pulsar] congbobo184 commented on a diff in pull request #15654: [admin][txn] Add transaction admin to get recover time in stats.

2022-05-24 Thread GitBox
congbobo184 commented on code in PR #15654: URL: https://github.com/apache/pulsar/pull/15654#discussion_r881200843 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/transaction/buffer/impl/TopicTransactionBuffer.java: ## @@ -98,6 +99,9 @@ public class

[GitHub] [pulsar] congbobo184 commented on a diff in pull request #15675: [Improve][txn] extend admin tools for transaction buffer.

2022-05-24 Thread GitBox
congbobo184 commented on code in PR #15675: URL: https://github.com/apache/pulsar/pull/15675#discussion_r881199719 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java: ## @@ -181,6 +181,16 @@ public void initialize() { } } +

[GitHub] [pulsar] dragonls commented on pull request #14971: [feature][broker] Add new load manager strategy for load manager to select broker having load below average.

2022-05-24 Thread GitBox
dragonls commented on PR #14971: URL: https://github.com/apache/pulsar/pull/14971#issuecomment-1136695236 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] Technoboy- commented on pull request #14260: Fix PersistentAcknowledgmentsGroupingTracker set BitSet issue.

2022-05-24 Thread GitBox
Technoboy- commented on PR #14260: URL: https://github.com/apache/pulsar/pull/14260#issuecomment-1136694442 > > So I have added getOriginalBatchSize to return the user-specified batch size. > > @Technoboy- what does the "user-specified batch size" mean? I'd assume that each batch

[GitHub] [pulsar] Shawyeok commented on pull request #15767: [pulsar-broker] Change dispatcher blocked/unblocked log level to debug

2022-05-24 Thread GitBox
Shawyeok commented on PR #15767: URL: https://github.com/apache/pulsar/pull/15767#issuecomment-1136691680 @wolfstudy @merlimat @eolivelli PTAL -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] Shawyeok commented on pull request #14950: Fix logger lever from warn to debug when uancked threshold is limited

2022-05-24 Thread GitBox
Shawyeok commented on PR #14950: URL: https://github.com/apache/pulsar/pull/14950#issuecomment-1136690874 I've open a follow up PR https://github.com/apache/pulsar/pull/15767 to change other log lines, PTAL. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] Shawyeok opened a new pull request, #15767: [pulsar-broker] Change dispatcher blocked/unblocked log level to debug

2022-05-24 Thread GitBox
Shawyeok opened a new pull request, #15767: URL: https://github.com/apache/pulsar/pull/15767 ### Motivation There are so many dispatcher blocked/unblocked log(more than 30K within 15 min), I supposed change it to debug.

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
BewareMyPower commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881186857 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java: ## @@ -2551,29 +2551,46 @@ public void

[GitHub] [pulsar-client-go] wolfstudy opened a new pull request, #777: Introduce doneCh for ack error

2022-05-24 Thread GitBox
wolfstudy opened a new pull request, #777: URL: https://github.com/apache/pulsar-client-go/pull/777 Signed-off-by: xiaolongran ### Motivation In #456, we support the error information that may occur when the Go SDK side processes the Ack Command, but there is a bug in

[GitHub] [pulsar] merlimat commented on issue #15703: `receiver_queue_size` of 1 does not work; Shared consumers with receiver_queue_size=1 can "steal" messages that other consumers should receive

2022-05-24 Thread GitBox
merlimat commented on issue #15703: URL: https://github.com/apache/pulsar/issues/15703#issuecomment-1136684901 @zbentley The pre-fetching is tied to the `receive()` calls and not the acknowledgements. As long as you keep calling receive, you are making more space in the queue and

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881182201 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java: ## @@ -2551,29 +2551,46 @@ public void

[GitHub] [pulsar] Anonymitaet merged pull request #15756: [fix][doc] Deprecate io-cli page and update occurrences referenced elsewhere

2022-05-24 Thread GitBox
Anonymitaet merged PR #15756: URL: https://github.com/apache/pulsar/pull/15756 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[pulsar] branch master updated (74b108d467a -> 6351dda8f5b)

2022-05-24 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git from 74b108d467a [improve][broker] Add timeout for unload namespace bundle. #15719 add 6351dda8f5b [fix][doc]

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881181741 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java: ## @@ -2551,29 +2551,46 @@ public void

[pulsar] branch master updated: [improve][broker] Add timeout for unload namespace bundle. #15719

2022-05-24 Thread technoboy
This is an automated email from the ASF dual-hosted git repository. technoboy 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 74b108d467a [improve][broker] Add timeout

[GitHub] [pulsar] Technoboy- merged pull request #15719: [fix][broker] Add timeout for unload namespace bundle.

2022-05-24 Thread GitBox
Technoboy- merged PR #15719: URL: https://github.com/apache/pulsar/pull/15719 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881178895 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
BewareMyPower commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881168940 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java: ## @@ -2551,29 +2551,46 @@ public void

[pulsar] branch branch-2.8 updated: Fix can not enable system topic if `AutoUpdateSchemaEnabled=false`. (#15759)

2022-05-24 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.8 by this push: new 7869cd098b1 Fix can not enable system

[GitHub] [pulsar] codelipenghui merged pull request #15759: [branch-2.8][broker] Fix can not enable system topic if `AutoUpdateSchemaEnabled=false`.

2022-05-24 Thread GitBox
codelipenghui merged PR #15759: URL: https://github.com/apache/pulsar/pull/15759 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
BewareMyPower commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881168940 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java: ## @@ -2551,29 +2551,46 @@ public void

[GitHub] [pulsar] AnonHxy opened a new pull request, #15766: [fix][broker]Sync topicPublishRateLimiter update

2022-05-24 Thread GitBox
AnonHxy opened a new pull request, #15766: URL: https://github.com/apache/pulsar/pull/15766 ### Motivation * Synchronized `topicPublishRateLimiter` update action to prevent resource leakages: * Thread1: `topicPublishRateLimiter ` is null, and and it will not invoke

[GitHub] [pulsar] drianpauL commented on issue #15230: [Docker] - Unable to mount /pulsar/data in the 2.10.0 image version

2022-05-24 Thread GitBox
drianpauL commented on issue #15230: URL: https://github.com/apache/pulsar/issues/15230#issuecomment-1136660863 Another workaround is to create a custom/derived Docker image and create the `data` directory with the correct permissions ``` FROM apachepulsar/pulsar:2.10.0 USER 0

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
BewareMyPower commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881155990 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java: ## @@ -2555,29 +2555,46 @@ public void

[GitHub] [pulsar] nodece commented on pull request #15761: [fix][client] Remove consumer when close consumer command is received

2022-05-24 Thread GitBox
nodece commented on PR #15761: URL: https://github.com/apache/pulsar/pull/15761#issuecomment-1136645171 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881154546 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[GitHub] [pulsar] github-actions[bot] commented on pull request #14084: [Doc] Describe default ack timeout and update grammar in ConsumerBuilder javadoc

2022-05-24 Thread GitBox
github-actions[bot] commented on PR #14084: URL: https://github.com/apache/pulsar/pull/14084#issuecomment-1136641095 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] massakam commented on a diff in pull request #15657: [improve][admin] Enable users to specify TTL options in units other than seconds

2022-05-24 Thread GitBox
massakam commented on code in PR #15657: URL: https://github.com/apache/pulsar/pull/15657#discussion_r881151980 ## pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdTopicPolicies.java: ## @@ -277,22 +277,31 @@ private class SetMessageTTL extends CliCommand {

[GitHub] [pulsar] github-actions[bot] commented on pull request #15015: [enh][transaction] Optimize to reuse transaction buffer snapshot writer

2022-05-24 Thread GitBox
github-actions[bot] commented on PR #15015: URL: https://github.com/apache/pulsar/pull/15015#issuecomment-1136639725 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] github-actions[bot] commented on issue #15248: JWT: Generate different tokens when using key file and key inline with same secret key content

2022-05-24 Thread GitBox
github-actions[bot] commented on issue #15248: URL: https://github.com/apache/pulsar/issues/15248#issuecomment-1136639121 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] github-actions[bot] commented on issue #15285: [Bug] pulsar-io-jdbc-sink get table definition possible error

2022-05-24 Thread GitBox
github-actions[bot] commented on issue #15285: URL: https://github.com/apache/pulsar/issues/15285#issuecomment-1136639009 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] github-actions[bot] commented on pull request #15287: [pulsar-io] Specify the database when getting the table definition

2022-05-24 Thread GitBox
github-actions[bot] commented on PR #15287: URL: https://github.com/apache/pulsar/pull/15287#issuecomment-1136638961 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] github-actions[bot] commented on issue #15290: [dependency] The module `pulsar-zookeeper-utils` has potential conflict dependency problem

2022-05-24 Thread GitBox
github-actions[bot] commented on issue #15290: URL: https://github.com/apache/pulsar/issues/15290#issuecomment-1136638930 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] github-actions[bot] commented on issue #15294: [Functions issues on state]

2022-05-24 Thread GitBox
github-actions[bot] commented on issue #15294: URL: https://github.com/apache/pulsar/issues/15294#issuecomment-1136638905 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] github-actions[bot] commented on issue #15299: Macos pulsar-client-cpp 2.8 VersionTest.cc error

2022-05-24 Thread GitBox
github-actions[bot] commented on issue #15299: URL: https://github.com/apache/pulsar/issues/15299#issuecomment-1136638863 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
BewareMyPower commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881147656 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881147422 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[pulsar] branch branch-2.9 updated: [CI] Upgrade zlib version to 1.2.12 (#14964)

2022-05-24 Thread mattisonchao
This is an automated email from the ASF dual-hosted git repository. mattisonchao pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.9 by this push: new ea435d8dd3c [CI] Upgrade zlib

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881144505 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881144339 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
BewareMyPower commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881144151 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #15663: [fix][broker] Fix creating producer failure when set backlog quota.

2022-05-24 Thread GitBox
BewareMyPower commented on code in PR #15663: URL: https://github.com/apache/pulsar/pull/15663#discussion_r881142524 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/systopic/PartitionedSystemTopicTest.java: ## @@ -171,4 +177,49 @@ public void

[GitHub] [pulsar] Anonymitaet commented on a diff in pull request #15763: Support dlog bookie client configuration via pass through config

2022-05-24 Thread GitBox
Anonymitaet commented on code in PR #15763: URL: https://github.com/apache/pulsar/pull/15763#discussion_r881141844 ## conf/broker.conf: ## @@ -1429,6 +1432,10 @@ packagesReplicas=1 # The bookkeeper ledger root path packagesManagementLedgerRootPath=/ledgers +# When using

[GitHub] [pulsar] mattisonchao commented on pull request #15742: [security] Allow to config web server's cipher and protocols.

2022-05-24 Thread GitBox
mattisonchao commented on PR #15742: URL: https://github.com/apache/pulsar/pull/15742#issuecomment-1136627085 Hi @Lico-Tom, Could you have time to check the CI? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[pulsar] branch master updated: [fix][doc] Updated example section (#15727)

2022-05-24 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu 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 eee038870db [fix][doc] Updated example section

[GitHub] [pulsar] Anonymitaet merged pull request #15727: [fix][schema-understand][docs] Updated example section

2022-05-24 Thread GitBox
Anonymitaet merged PR #15727: URL: https://github.com/apache/pulsar/pull/15727 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[pulsar] branch branch-2.9 updated: [branch-2.9][improve][client] Avoid timer task run before previous subscribe complete. (#15747)

2022-05-24 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.9 by this push: new cf341857aad

[GitHub] [pulsar] codelipenghui merged pull request #15747: [branch-2.9][improve][client] Avoid timer task run before previous subscribe complete.

2022-05-24 Thread GitBox
codelipenghui merged PR #15747: URL: https://github.com/apache/pulsar/pull/15747 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] Demogorgon314 commented on pull request #15733: [fix][admin] Unwrap the completion exception on schemas admin API

2022-05-24 Thread GitBox
Demogorgon314 commented on PR #15733: URL: https://github.com/apache/pulsar/pull/15733#issuecomment-1136612803 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] Demogorgon314 commented on pull request #15718: [fix][schema] Fix update schema forward compatibility strategy behavior

2022-05-24 Thread GitBox
Demogorgon314 commented on PR #15718: URL: https://github.com/apache/pulsar/pull/15718#issuecomment-1136612774 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15729: [improve][txn] Support ack message list for transaction

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15729: URL: https://github.com/apache/pulsar/pull/15729#discussion_r881107297 ## pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Consumer.java: ## @@ -491,13 +491,51 @@ CompletableFuture acknowledgeCumulativeAsync(MessageId

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #15729: [improve][txn] Support ack message list for transaction

2022-05-24 Thread GitBox
Technoboy- commented on code in PR #15729: URL: https://github.com/apache/pulsar/pull/15729#discussion_r881106921 ## pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Consumer.java: ## @@ -491,13 +491,51 @@ CompletableFuture acknowledgeCumulativeAsync(MessageId

[pulsar] branch branch-2.9 updated: [branch-2.9] [Tests] Fix flaky WrongTypeOfReturnValue Mockito misuse issue in broker tests. (#15738)

2022-05-24 Thread mattisonchao
This is an automated email from the ASF dual-hosted git repository. mattisonchao pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.9 by this push: new dfbc09f5eee [branch-2.9] [Tests]

[GitHub] [pulsar] mattisonchao merged pull request #15738: [branch-2.9] [Tests] Fix flaky WrongTypeOfReturnValue Mockito misuse issue in broker tests.

2022-05-24 Thread GitBox
mattisonchao merged PR #15738: URL: https://github.com/apache/pulsar/pull/15738 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] shibd commented on pull request #15760: [improve][broker] Make some operation replication clusters methods in…

2022-05-24 Thread GitBox
shibd commented on PR #15760: URL: https://github.com/apache/pulsar/pull/15760#issuecomment-1136590984 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[pulsar] branch branch-2.9 updated: [branch-2.9] [Authorization] Role with namespace produce authz can also get topics. (#15740)

2022-05-24 Thread mattisonchao
This is an automated email from the ASF dual-hosted git repository. mattisonchao pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.9 by this push: new 6ded1808110 [branch-2.9]

[GitHub] [pulsar] mattisonchao merged pull request #15740: [branch-2.9] [Authorization] Role with namespace produce authz can also get topics.

2022-05-24 Thread GitBox
mattisonchao merged PR #15740: URL: https://github.com/apache/pulsar/pull/15740 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[pulsar] branch branch-2.10 updated: [Broker] Disable memory limit controller for broker client and replication clients (#15723)

2022-05-24 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-2.10 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.10 by this push: new a56f04140e9 [Broker] Disable memory

[pulsar] branch master updated: [Broker] Disable memory limit controller for broker client and replication clients (#15723)

2022-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 563a7cb2341 [Broker] Disable memory limit

[GitHub] [pulsar] merlimat merged pull request #15723: [Broker] Disable memory limit controller for broker client and replication clients

2022-05-24 Thread GitBox
merlimat merged PR #15723: URL: https://github.com/apache/pulsar/pull/15723 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] merlimat closed issue #15691: [Broker] Support configuration of the memory allocated to replication producer

2022-05-24 Thread GitBox
merlimat closed issue #15691: [Broker] Support configuration of the memory allocated to replication producer URL: https://github.com/apache/pulsar/issues/15691 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[pulsar] branch branch-2.9 updated: [branch-2.9][enh][monitor]: add metrics for pulsar web service thread pool (#15741)

2022-05-24 Thread mattisonchao
This is an automated email from the ASF dual-hosted git repository. mattisonchao pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.9 by this push: new f7058ba58be

[GitHub] [pulsar] mattisonchao merged pull request #15741: [branch-2.9][enh][monitor]: add metrics for pulsar web service thread pool

2022-05-24 Thread GitBox
mattisonchao merged PR #15741: URL: https://github.com/apache/pulsar/pull/15741 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] nlu90 commented on pull request #15250: [function] refine file io connector

2022-05-24 Thread GitBox
nlu90 commented on PR #15250: URL: https://github.com/apache/pulsar/pull/15250#issuecomment-1136507866 @freeznet Could you help take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] nlu90 commented on pull request #15250: [function] refine file io connector

2022-05-24 Thread GitBox
nlu90 commented on PR #15250: URL: https://github.com/apache/pulsar/pull/15250#issuecomment-1136506779 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] nlu90 commented on pull request #15728: [Function] provide default error handler

2022-05-24 Thread GitBox
nlu90 commented on PR #15728: URL: https://github.com/apache/pulsar/pull/15728#issuecomment-1136506267 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] github-actions[bot] commented on pull request #15765: Bump spring-core from 5.3.19 to 5.3.20 in /pulsar-io/canal

2022-05-24 Thread GitBox
github-actions[bot] commented on PR #15765: URL: https://github.com/apache/pulsar/pull/15765#issuecomment-1136505657 @dependabot[bot]:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about

[pulsar] branch dependabot/maven/pulsar-io/canal/org.springframework-spring-core-5.3.20 created (now 1e6ddb30927)

2022-05-24 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/pulsar-io/canal/org.springframework-spring-core-5.3.20 in repository https://gitbox.apache.org/repos/asf/pulsar.git at 1e6ddb30927 Bump spring-core from 5.3.19 to

[GitHub] [pulsar] dependabot[bot] opened a new pull request, #15765: Bump spring-core from 5.3.19 to 5.3.20 in /pulsar-io/canal

2022-05-24 Thread GitBox
dependabot[bot] opened a new pull request, #15765: URL: https://github.com/apache/pulsar/pull/15765 Bumps [spring-core](https://github.com/spring-projects/spring-framework) from 5.3.19 to 5.3.20. Release notes Sourced from

[GitHub] [pulsar] michaeljmarshall opened a new issue, #15764: Flaky-test: ConnectionTimeoutTest.testLowTimeout

2022-05-24 Thread GitBox
michaeljmarshall opened a new issue, #15764: URL: https://github.com/apache/pulsar/issues/15764 ConnectionTimeoutTest.testLowTimeout is flaky. It fails sporadically. https://github.com/apache/pulsar/runs/6582172714 ``` Error:

[GitHub] [pulsar] merlimat closed issue #15074: Failing to build Alpine wheel

2022-05-24 Thread GitBox
merlimat closed issue #15074: Failing to build Alpine wheel URL: https://github.com/apache/pulsar/issues/15074 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

[pulsar] branch master updated (6b40749dd4d -> bf194b557c4)

2022-05-24 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git from 6b40749dd4d [C++] Unblock all threads when Pulsar client is closed (#15726) add bf194b557c4 PIP-117: Change

[GitHub] [pulsar] merlimat merged pull request #15478: PIP-117: Change Pulsar standalone defaults

2022-05-24 Thread GitBox
merlimat merged PR #15478: URL: https://github.com/apache/pulsar/pull/15478 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] michaeljmarshall opened a new pull request, #15763: Support dlog bookie client configuration via pass through config

2022-05-24 Thread GitBox
michaeljmarshall opened a new pull request, #15763: URL: https://github.com/apache/pulsar/pull/15763 ### Motivation Enable pass through configuration support for the bookkeeper clients created by `DistributedLog` and `BookKeeperPackagesStorage`. This support is already present for

[GitHub] [pulsar] mattisonchao closed pull request #15747: [branch-2.9][improve][client] Avoid timer task run before previous subscribe complete.

2022-05-24 Thread GitBox
mattisonchao closed pull request #15747: [branch-2.9][improve][client] Avoid timer task run before previous subscribe complete. URL: https://github.com/apache/pulsar/pull/15747 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] heesung-sn opened a new pull request, #15762: [improve][deploy] changed the default GC options to ZGC from G1GC

2022-05-24 Thread GitBox
heesung-sn opened a new pull request, #15762: URL: https://github.com/apache/pulsar/pull/15762 Master Issue: #15207 ### Motivation # Pulsar Server Default GC Update As Java 17 will be officially required for pulsar-2.11+, it would be worth to revisit Pulsar’s GC

[GitHub] [pulsar] dlg99 commented on a diff in pull request #15757: Issue 15750: PIP-105: Store Subscription properties

2022-05-24 Thread GitBox
dlg99 commented on code in PR #15757: URL: https://github.com/apache/pulsar/pull/15757#discussion_r880926599 ## managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java: ## @@ -361,6 +369,18 @@ public void operationComplete(ManagedCursorInfo info,

[GitHub] [pulsar] eolivelli commented on pull request #15757: Issue 15750: PIP-105: Store Subscription properties

2022-05-24 Thread GitBox
eolivelli commented on PR #15757: URL: https://github.com/apache/pulsar/pull/15757#issuecomment-1136370590 > How do the consumers change the subscription properties? The change looks like the properties will not change after the cursor is created. > > The previous behavior is the

[GitHub] [pulsar] eolivelli commented on a diff in pull request #15757: Issue 15750: PIP-105: Store Subscription properties

2022-05-24 Thread GitBox
eolivelli commented on code in PR #15757: URL: https://github.com/apache/pulsar/pull/15757#discussion_r880893062 ## managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java: ## @@ -845,11 +845,12 @@ public ManagedCursor openCursor(String

[GitHub] [pulsar] eolivelli commented on a diff in pull request #15757: Issue 15750: PIP-105: Store Subscription properties

2022-05-24 Thread GitBox
eolivelli commented on code in PR #15757: URL: https://github.com/apache/pulsar/pull/15757#discussion_r880892876 ## managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java: ## @@ -361,6 +369,18 @@ public void operationComplete(ManagedCursorInfo

[GitHub] [pulsar] dlg99 commented on a diff in pull request #15757: Issue 15750: PIP-105: Store Subscription properties

2022-05-24 Thread GitBox
dlg99 commented on code in PR #15757: URL: https://github.com/apache/pulsar/pull/15757#discussion_r880803341 ## managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java: ## @@ -845,11 +845,12 @@ public ManagedCursor openCursor(String cursorName)

[pulsar] branch master updated: [C++] Unblock all threads when Pulsar client is closed (#15726)

2022-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 6b40749dd4d [C++] Unblock all threads when

[GitHub] [pulsar] merlimat merged pull request #15726: [C++] Unblock all threads when Pulsar client is closed

2022-05-24 Thread GitBox
merlimat merged PR #15726: URL: https://github.com/apache/pulsar/pull/15726 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [pulsar] merlimat commented on issue #15585: [python] Pulsar Client 2.10.0 package is missing from PyPi for Macosx 12

2022-05-24 Thread GitBox
merlimat commented on issue #15585: URL: https://github.com/apache/pulsar/issues/15585#issuecomment-1136250680 @ca-simone-chiorazzo @zhavir Can you try with this wheel file?

[GitHub] [pulsar] Shawyeok commented on pull request #15594: [pulsar-client] Throw more meaningful derived exception instead of PulsarClientException

2022-05-24 Thread GitBox
Shawyeok commented on PR #15594: URL: https://github.com/apache/pulsar/pull/15594#issuecomment-1136243777 @lhotari @codelipenghui PTAL -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] nodece opened a new pull request, #15761: [fix][client] Remove consumer when close consumer command is received

2022-05-24 Thread GitBox
nodece opened a new pull request, #15761: URL: https://github.com/apache/pulsar/pull/15761 Signed-off-by: Zixuan Liu ### Motivation When the close consumer command is received, we just disconnected and didn't remove the cache, which will cause us to receive repeated messages.

[GitHub] [pulsar] shibd commented on issue #15746: running in docker throws exception

2022-05-24 Thread GitBox
shibd commented on issue #15746: URL: https://github.com/apache/pulsar/issues/15746#issuecomment-1136135668 I reproduce this on MAC OS, i will try to fix it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [pulsar] shibd opened a new pull request, #15760: [improve][broker] Make some operation replication clusters methods in…

2022-05-24 Thread GitBox
shibd opened a new pull request, #15760: URL: https://github.com/apache/pulsar/pull/15760 ### Motivation See PIP https://github.com/apache/pulsar/issues/14365 and change tracker https://github.com/apache/pulsar/issues/15043. Make NamespacesBase `getNamespaceReplicationClusters

[GitHub] [pulsar] merlimat commented on issue #15585: [python] Pulsar Client 2.10.0 package is missing from PyPi for Macosx 12

2022-05-24 Thread GitBox
merlimat commented on issue #15585: URL: https://github.com/apache/pulsar/issues/15585#issuecomment-1136092703 @zhavir Looks like you're on MacOS 10.15? The wheels are compiled with 11.0 as the min version. We wanted to try to target 10.15 instead. There were few issues before,

[GitHub] [pulsar] coderzc commented on pull request #15678: [improve][proxy] Support zero-copy of NIC to NIC on Proxy

2022-05-24 Thread GitBox
coderzc commented on PR #15678: URL: https://github.com/apache/pulsar/pull/15678#issuecomment-1136067278 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

  1   2   3   >