Re: [PR] [fix][broker] The topic might reference a closed ledger [pulsar]

2024-05-17 Thread via GitHub
dao-jun commented on PR #22739: URL: https://github.com/apache/pulsar/pull/22739#issuecomment-2118652021 Great work! -- 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 unsu

(pulsar) branch branch-3.0 updated: [fix][admin] Fix can't delete tenant for v1 (#22550)

2024-05-17 Thread technoboy
This is an automated email from the ASF dual-hosted git repository. technoboy pushed a commit to branch branch-3.0 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.0 by this push: new 5d0f1d6609e [fix][admin] Fix can't de

(pulsar) branch master updated: [improve][build] Remove invalid relativePath definitions in pom.xml files (#22741)

2024-05-17 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj 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 4593cc33f93 [improve][build] Remove invalid re

Re: [PR] [improve][build] Remove invalid relativePath definitions in pom.xml files [pulsar]

2024-05-17 Thread via GitHub
shoothzj merged PR #22741: URL: https://github.com/apache/pulsar/pull/22741 -- 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: commits-unsubscr...@pulsar.a

(pulsar-site) branch main updated: Docs sync done from apache/pulsar (#2308f27)

2024-05-17 Thread urfree
This is an automated email from the ASF dual-hosted git repository. urfree pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git The following commit(s) were added to refs/heads/main by this push: new a775cee21206 Docs sync done from apache/pulsar

Re: [PR] [fix][broker] Make ExtensibleLoadManagerImpl.getOwnedServiceUnits async [pulsar]

2024-05-17 Thread via GitHub
heesung-sn commented on PR #22727: URL: https://github.com/apache/pulsar/pull/22727#issuecomment-2118408957 I cherry-picked this to 3.0, 3.1, 3.2, and 3.3. -- 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

(pulsar) branch branch-3.3 updated: [fix][broker] Make ExtensibleLoadManagerImpl.getOwnedServiceUnits async (#22727)

2024-05-17 Thread heesung
This is an automated email from the ASF dual-hosted git repository. heesung pushed a commit to branch branch-3.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.3 by this push: new 11821c21e16 [fix][broker] Make Extensi

(pulsar) branch branch-3.2 updated: [fix][broker] Make ExtensibleLoadManagerImpl.getOwnedServiceUnits async (#22727)

2024-05-17 Thread heesung
This is an automated email from the ASF dual-hosted git repository. heesung pushed a commit to branch branch-3.2 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.2 by this push: new aa8914a1ef3 [fix][broker] Make Extensi

(pulsar) branch branch-3.1 updated: [fix][broker] Make ExtensibleLoadManagerImpl.getOwnedServiceUnits async (#22727)

2024-05-17 Thread heesung
This is an automated email from the ASF dual-hosted git repository. heesung pushed a commit to branch branch-3.1 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.1 by this push: new 18f4226d58f [fix][broker] Make Extensi

(pulsar) branch branch-3.0 updated: [fix][broker] Make ExtensibleLoadManagerImpl.getOwnedServiceUnits async (#22727)

2024-05-17 Thread heesung
This is an automated email from the ASF dual-hosted git repository. heesung pushed a commit to branch branch-3.0 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.0 by this push: new 05317c8a660 [fix][broker] Make Extensi

Re: [I] [Bug] After updating broker from 3.0.3 to 3.0.4, reader cannot receive new messages from compacted topics. [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on issue #22634: URL: https://github.com/apache/pulsar/issues/22634#issuecomment-2118380057 Pulsar 3.0.5 has been released. Are you able to test whether the problem is fixed now? -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [I] [Bug] Message loss on bookies and brokers restart [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on issue #22709: URL: https://github.com/apache/pulsar/issues/22709#issuecomment-2118374823 Pulsar 3.2.3 and 3.0.5 have been released. Would you be able to test with either version? Please also make sure to upgrade the clients, just to be sure that everything has been test

Re: [PR] Support NodePort Proxy service [pulsar-helm-chart]

2024-05-17 Thread via GitHub
MMirelli commented on PR #500: URL: https://github.com/apache/pulsar-helm-chart/pull/500#issuecomment-2118362972 I addressed the comments, @lhotari could you PTAL? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[PR] Support NodePort Proxy service [pulsar-helm-chart]

2024-05-17 Thread via GitHub
MMirelli opened a new pull request, #500: URL: https://github.com/apache/pulsar-helm-chart/pull/500 Fixes #281 -- 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 unsubscrib

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
nodece commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117955367 @coderzc Could you cherry-pick this PR to branch-3.3? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

(pulsar) branch master updated: [fix][build] Fix pulsar-client-python installation on ARM arch (#22733)

2024-05-17 Thread zixuan
This is an automated email from the ASF dual-hosted git repository. zixuan 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 2308f27c555 [fix][build] Fix pulsar-client-pytho

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
nodece merged PR #22733: URL: https://github.com/apache/pulsar/pull/22733 -- 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: commits-unsubscr...@pulsar.apa

(pulsar-client-reactive) branch main updated: [Release 0.5.5] Update latest version in README.adoc to 0.5.5 (#172)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-client-reactive.git The following commit(s) were added to refs/heads/main by this push: new ea3a2e4 [Release 0.5.5] Update lat

Re: [PR] [Release 0.5.5] Update latest version in README.adoc to 0.5.5 [pulsar-client-reactive]

2024-05-17 Thread via GitHub
lhotari merged PR #172: URL: https://github.com/apache/pulsar-client-reactive/pull/172 -- 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: commits-unsubscr.

(pulsar-client-reactive) branch main updated: [Release 0.5.5] Update next snapshot version to 0.5.6-SNAPSHOT (#173)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-client-reactive.git The following commit(s) were added to refs/heads/main by this push: new e719a88 [Release 0.5.5] Update nex

Re: [PR] [Release 0.5.5] Update next snapshot version to 0.5.6-SNAPSHOT [pulsar-client-reactive]

2024-05-17 Thread via GitHub
lhotari merged PR #173: URL: https://github.com/apache/pulsar-client-reactive/pull/173 -- 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: commits-unsubscr.

[PR] Upgrade Pulsar to 3.2.3 version [pulsar-client-reactive]

2024-05-17 Thread via GitHub
lhotari opened a new pull request, #174: URL: https://github.com/apache/pulsar-client-reactive/pull/174 (no comment) -- 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 unsu

Re: [PR] [branch-3.0][feat][broker] Implementation of PIP-323: Complete Backlog Quota Telemetry (#21816) [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on code in PR #22740: URL: https://github.com/apache/pulsar/pull/22740#discussion_r1605205281 ## pulsar-broker/pom.xml: ## @@ -417,6 +417,12 @@ javax.activation + + io.rest-assured + rest-assured + test + + Review Comment:

Re: [PR] [branch-3.0][feat][broker] Implementation of PIP-323: Complete Backlog Quota Telemetry (#21816) [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on code in PR #22740: URL: https://github.com/apache/pulsar/pull/22740#discussion_r1605194713 ## pulsar-broker/pom.xml: ## @@ -417,6 +417,12 @@ javax.activation + + io.rest-assured + rest-assured + test + + Review Comment:

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
nodece commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117816326 > > I'm using podman to build the Pulsar image, it works fine. The docker may not support HEREDOC in the GitHub Actions. > > Use `echo -e` instead of HEREDOC. > > I provided the

(pulsar-helm-chart) annotated tag pulsar-3.4.1-candidate-1 updated (3ecc2ba -> bcafa6d)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to annotated tag pulsar-3.4.1-candidate-1 in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git *** WARNING: tag pulsar-3.4.1-candidate-1 was modified! *** from 3ecc2ba (commit)

svn commit: r69256 - /dev/pulsar/helm-chart/3.4.1-candidate-1/

2024-05-17 Thread lhotari
Author: lhotari Date: Fri May 17 14:57:26 2024 New Revision: 69256 Log: Add artifacts for Helm Chart 3.4.1-candidate-1 Added: dev/pulsar/helm-chart/3.4.1-candidate-1/index.yaml dev/pulsar/helm-chart/3.4.1-candidate-1/pulsar-3.4.1.tgz (with props) dev/pulsar/helm-chart/3.4.1-candidat

svn commit: r69255 - /dev/pulsar/helm-chart/3.4.1-candidate-1/

2024-05-17 Thread lhotari
Author: lhotari Date: Fri May 17 14:57:13 2024 New Revision: 69255 Log: Add directory for pulsar-helm-chart 3.4.1-candidate-1 release Added: dev/pulsar/helm-chart/3.4.1-candidate-1/

(pulsar-helm-chart) branch master updated: Chart: Bump version to 3.4.1

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git The following commit(s) were added to refs/heads/master by this push: new 3ecc2ba Chart: Bump version to 3.4.

(pulsar-helm-chart) branch master updated: Use Pulsar 3.0.5 as the default Pulsar version (appVersion) (#499)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git The following commit(s) were added to refs/heads/master by this push: new 6795ad5 Use Pulsar 3.0.5 as the def

Re: [PR] Use Pulsar 3.0.5 as the default Pulsar version (appVersion) [pulsar-helm-chart]

2024-05-17 Thread via GitHub
lhotari merged PR #499: URL: https://github.com/apache/pulsar-helm-chart/pull/499 -- 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: commits-unsubscr...@pu

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117779314 > I'm using podman to build the Pulsar image, it works fine. The docker may not support HEREDOC in the GitHub Actions. > > Use `echo -e` instead of HEREDOC. I provided the l

(pulsar) branch master updated: [fix][test] Fix flaky AuthorizationTest.testGetListWithGetBundleOp (#22713)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 158960a7e2a [fix][test] Fix flaky Authorizatio

Re: [I] Flaky-test: AuthorizationTest.testGetListWithGetBundleOp [pulsar]

2024-05-17 Thread via GitHub
lhotari closed issue #22712: Flaky-test: AuthorizationTest.testGetListWithGetBundleOp URL: https://github.com/apache/pulsar/issues/22712 -- 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

Re: [PR] [fix][test] Fix flaky AuthorizationTest.testGetListWithGetBundleOp [pulsar]

2024-05-17 Thread via GitHub
lhotari merged PR #22713: URL: https://github.com/apache/pulsar/pull/22713 -- 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: commits-unsubscr...@pulsar.ap

Re: [PR] [fix][test] Fix flaky AuthorizationTest.testGetListWithGetBundleOp [pulsar]

2024-05-17 Thread via GitHub
codecov-commenter commented on PR #22713: URL: https://github.com/apache/pulsar/pull/22713#issuecomment-2117710738 ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/22713?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

Re: [I] Pulsar manager service fails saying "2024-05-07 15:02:17,476 INFO exited: pulsar-manager-backend (terminated by SIGKILL; not expected)" [pulsar-helm-chart]

2024-05-17 Thread via GitHub
lhotari commented on issue #496: URL: https://github.com/apache/pulsar-helm-chart/issues/496#issuecomment-2117709462 there's a fix #485 which will be part of the next release. It might help with the issue. -- This is an automated message from the Apache Git Service. To respond to the mes

[PR] Use Pulsar 3.0.5 as the default Pulsar version (appVersion) [pulsar-helm-chart]

2024-05-17 Thread via GitHub
lhotari opened a new pull request, #499: URL: https://github.com/apache/pulsar-helm-chart/pull/499 ### Motivation - Pulsar 3.0.5 has been releases with many fixes ### Modifications - update chart's `appVersion` to `3.0.5` ### Verifying this change - [ ] Make

Re: [PR] [branch-3.0][feat][broker] Implementation of PIP-323: Complete Backlog Quota Telemetry (#21816) [pulsar]

2024-05-17 Thread via GitHub
codelipenghui commented on code in PR #22740: URL: https://github.com/apache/pulsar/pull/22740#discussion_r1605057473 ## pulsar-broker/pom.xml: ## @@ -417,6 +417,12 @@ javax.activation + + io.rest-assured + rest-assured + test + + Review Co

Re: [PR] [improve] [broker] Trigger offload on topic load [pulsar]

2024-05-17 Thread via GitHub
dao-jun merged PR #22652: URL: https://github.com/apache/pulsar/pull/22652 -- 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: commits-unsubscr...@pulsar.ap

(pulsar) branch master updated (e35c00e9c09 -> 400a286ca09)

2024-05-17 Thread daojun
This is an automated email from the ASF dual-hosted git repository. daojun pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git from e35c00e9c09 [fix][offload] Break the fillbuffer loop when met EOF (#22722) add 400a286ca09 [improve] [broker] Tr

(pulsar-site) 05/05: Add release notes for 3.2.3

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git commit babfe34bef51259230101dd2c8e74e1e2f222fdb Author: Lari Hotari AuthorDate: Fri May 17 16:42:21 2024 +0300 Add rel

(pulsar-site) 04/05: Fix version in client-java-3.2.2.md file

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git commit 106a5e565b16a480d4750eb63002d26b361db43a Author: Lari Hotari AuthorDate: Fri May 17 16:40:20 2024 +0300 Fix ver

(pulsar-site) branch main updated (f57d8ece2744 -> babfe34bef51)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git from f57d8ece2744 Docs sync done from apache/pulsar (#4e132d3) new 8f0af505bc3c update rest-apidoc for 3.0.5

(pulsar-site) 03/05: Add release notes for 3.0.5

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git commit f2a885ac1cdce0206654c7ce747c9d5614679d8b Author: Lari Hotari AuthorDate: Fri May 17 16:35:56 2024 +0300 Add rel

Re: [PR] [branch-3.0][feat][broker] Implementation of PIP-323: Complete Backlog Quota Telemetry (#21816) [pulsar]

2024-05-17 Thread via GitHub
Technoboy- commented on code in PR #22740: URL: https://github.com/apache/pulsar/pull/22740#discussion_r1605046858 ## pulsar-broker/pom.xml: ## @@ -417,6 +417,12 @@ javax.activation + + io.rest-assured + rest-assured + test + + Review Comme

Re: [PR] [fix] [broker] fix deadlock when disable topic level Geo-Replication [pulsar]

2024-05-17 Thread via GitHub
shibd closed pull request #22738: [fix] [broker] fix deadlock when disable topic level Geo-Replication URL: https://github.com/apache/pulsar/pull/22738 -- 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 t

Re: [PR] [branch-3.0][feat][broker] Implementation of PIP-323: Complete Backlog Quota Telemetry (#21816) [pulsar]

2024-05-17 Thread via GitHub
codelipenghui commented on PR #22740: URL: https://github.com/apache/pulsar/pull/22740#issuecomment-2117629867 /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 sp

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
nodece commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117629438 ``` 2024-05-17T12:01:21.386067417Z 2024-05-17T12:01:21,385+ [function-timer-thread-102-1] INFO org.apache.pulsar.functions.runtime.process.ProcessRuntime - Started process succes

[PR] [branch-3.0][feat][broker] Implementation of PIP-323: Complete Backlog Quota Telemetry (#21816) [pulsar]

2024-05-17 Thread via GitHub
codelipenghui opened a new pull request, #22740: URL: https://github.com/apache/pulsar/pull/22740 (cherry picked from commit 59782b3c1f63baa68b2f5f33a33f1b00212159d7) ### Documentation - [ ] `doc` - [ ] `doc-required` - [x] `doc-not-needed` - [ ] `doc-complet

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
nodece commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117603529 Any suggestions? ``` 2024-05-17T12:00:21.504002141Z Traceback (most recent call last): 2024-05-17T12:00:21.504028610Z File "/pulsar/instances/python-instance/python_instance_mai

Re: [PR] [fix][broker] The topic might reference a closed ledger [pulsar]

2024-05-17 Thread via GitHub
shibd commented on code in PR #22739: URL: https://github.com/apache/pulsar/pull/22739#discussion_r1604992786 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java: ## @@ -1007,6 +1007,9 @@ public CompletableFuture> getTopic(final TopicName topicNam

Re: [PR] [fix][broker] The topic might reference a closed ledger [pulsar]

2024-05-17 Thread via GitHub
shibd commented on code in PR #22739: URL: https://github.com/apache/pulsar/pull/22739#discussion_r1604992786 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java: ## @@ -1007,6 +1007,9 @@ public CompletableFuture> getTopic(final TopicName topicNam

svn commit: r69251 - /dev/pulsar/pulsar-3.2.3-candidate-1/ /release/pulsar/pulsar-3.2.3/

2024-05-17 Thread lhotari
Author: lhotari Date: Fri May 17 13:13:14 2024 New Revision: 69251 Log: Release Apache Pulsar 3.2.3 Added: release/pulsar/pulsar-3.2.3/ - copied from r69250, dev/pulsar/pulsar-3.2.3-candidate-1/ Removed: dev/pulsar/pulsar-3.2.3-candidate-1/

(pulsar) annotated tag v3.2.3 updated (2f8a2bd5fd5 -> 0a6e0945258)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to annotated tag v3.2.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git *** WARNING: tag v3.2.3 was modified! *** from 2f8a2bd5fd5 (commit) to 0a6e0945258 (tag) tagging 2f8a2bd5fd

svn commit: r69250 - /dev/pulsar/pulsar-3.0.5-candidate-1/ /release/pulsar/pulsar-3.0.5/

2024-05-17 Thread lhotari
Author: lhotari Date: Fri May 17 13:08:44 2024 New Revision: 69250 Log: Release Apache Pulsar 3.0.5 Added: release/pulsar/pulsar-3.0.5/ - copied from r69249, dev/pulsar/pulsar-3.0.5-candidate-1/ Removed: dev/pulsar/pulsar-3.0.5-candidate-1/

(pulsar) annotated tag v3.0.5 updated (2da571eb2dc -> 9863b6c192c)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to annotated tag v3.0.5 in repository https://gitbox.apache.org/repos/asf/pulsar.git *** WARNING: tag v3.0.5 was modified! *** from 2da571eb2dc (commit) to 9863b6c192c (tag) tagging 2da571eb2d

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
nodece commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117438611 I'm using podman to build the Pulsar image, it works fine. The docker may not support HEREDOC in the GitHub Actions. Use `echo -e` instead of HEREDOC. -- This is an automated mes

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
nodece commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117432884 /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 specifi

Re: [PR] [fix][broker] The topic might reference a closed ledger [pulsar]

2024-05-17 Thread via GitHub
dao-jun commented on code in PR #22739: URL: https://github.com/apache/pulsar/pull/22739#discussion_r1604843101 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java: ## @@ -1007,6 +1007,9 @@ public CompletableFuture> getTopic(final TopicName topicN

Re: [PR] [fix][broker] The topic might reference a closed ledger [pulsar]

2024-05-17 Thread via GitHub
shibd closed pull request #22739: [fix][broker] The topic might reference a closed ledger URL: https://github.com/apache/pulsar/pull/22739 -- 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 specifi

[PR] [fix][broker] The topic might reference a closed ledger [pulsar]

2024-05-17 Thread via GitHub
shibd opened a new pull request, #22739: URL: https://github.com/apache/pulsar/pull/22739 ### Motivation We observe that a `normal topic` might reference a `closed ledger` and it never auto recover. will cause the producer and customer stuck. The root cause is related to `topic cre

(pulsar) branch branch-3.0 updated: [fix][offload] Break the fillbuffer loop when met EOF (#22722)

2024-05-17 Thread baodi
This is an automated email from the ASF dual-hosted git repository. baodi pushed a commit to branch branch-3.0 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.0 by this push: new 22b3a27c422 [fix][offload] Break the fill

(pulsar) branch branch-3.2 updated: [fix][offload] Break the fillbuffer loop when met EOF (#22722)

2024-05-17 Thread baodi
This is an automated email from the ASF dual-hosted git repository. baodi pushed a commit to branch branch-3.2 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.2 by this push: new 43b4a43b1d4 [fix][offload] Break the fill

(pulsar) branch branch-3.3 updated: [fix][offload] Break the fillbuffer loop when met EOF (#22722)

2024-05-17 Thread baodi
This is an automated email from the ASF dual-hosted git repository. baodi pushed a commit to branch branch-3.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.3 by this push: new 46ee50928ed [fix][offload] Break the fill

(pulsar) branch master updated: [fix][offload] Break the fillbuffer loop when met EOF (#22722)

2024-05-17 Thread baodi
This is an automated email from the ASF dual-hosted git repository. baodi 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 e35c00e9c09 [fix][offload] Break the fillbuffer l

Re: [PR] [fix][offload] Break the fillbuffer loop when met EOF [pulsar]

2024-05-17 Thread via GitHub
shibd merged PR #22722: URL: https://github.com/apache/pulsar/pull/22722 -- 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: commits-unsubscr...@pulsar.apac

[PR] [fix] [broker] fix deadlock when disable topic level Geo-Replication [pulsar]

2024-05-17 Thread via GitHub
poorbarcode opened a new pull request, #22738: URL: https://github.com/apache/pulsar/pull/22738 ### Motivation fix deadlock when disable topic level Geo-Replication, you can reproduce the issue by `OneWayReplicatorUsingGlobalZKTest.testDeleteNonPartitionedTopic` logs

Re: [PR] [improve][cli] Supports creating tokens with additional headers [pulsar]

2024-05-17 Thread via GitHub
codecov-commenter commented on PR #22690: URL: https://github.com/apache/pulsar/pull/22690#issuecomment-2117278848 ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/22690?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

Re: [D] Differences Between Packaging IO Connectors and Functions in .jar vs .nar [pulsar]

2024-05-17 Thread via GitHub
GitHub user NurramoX added a comment to the discussion: Differences Between Packaging IO Connectors and Functions in .jar vs .nar I see, that clarified the differences for me. Thank you for your response! And thank you for responding so quickly! GitHub link: https://github.com/apache/pulsar/

Re: [PR] [improve][cli] Supports creating tokens with additional headers [pulsar]

2024-05-17 Thread via GitHub
coderzc commented on code in PR #22690: URL: https://github.com/apache/pulsar/pull/22690#discussion_r1604623450 ## pulsar-broker/src/test/java/org/apache/pulsar/utils/auth/tokens/TokensCliUtilsTest.java: ## @@ -31,6 +36,37 @@ */ public class TokensCliUtilsTest { +@Test

Re: [PR] [fix][broker] Handle BucketDelayedDeliveryTracker recover failed [pulsar]

2024-05-17 Thread via GitHub
dao-jun closed pull request #22735: [fix][broker] Handle BucketDelayedDeliveryTracker recover failed URL: https://github.com/apache/pulsar/pull/22735 -- 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

Re: [PR] [improve][misc][WIP] Detect "double release" and "use after release" bugs with recycled objects [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on PR #22110: URL: https://github.com/apache/pulsar/pull/22110#issuecomment-2117095913 Pattern used in Netty to detect when already released instances are accessed: https://github.com/netty/netty/blob/c07dec11a90b82674db1529d0402af8e62d6589f/buffer/src/main/java/io/n

Re: [PR] [fix] [broker] fix subscribe a non-existent cluster/namespace/topic. [pulsar]

2024-05-17 Thread via GitHub
thetumbled commented on PR #22715: URL: https://github.com/apache/pulsar/pull/22715#issuecomment-2117074263 PTAL, thanks. @lhotari @dao-jun @BewareMyPower @poorbarcode -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] [fix][build] Fix pulsar-client-python installation on ARM arch [pulsar]

2024-05-17 Thread via GitHub
coderzc commented on PR #22733: URL: https://github.com/apache/pulsar/pull/22733#issuecomment-2117040213 @nodece Can you take a look as soon as possible, because this PR is marked as release/blocker -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [I] [Bug] parseMessageMetadata error when broker entry metadata enable with high loading [pulsar]

2024-05-17 Thread via GitHub
semistone commented on issue #22601: URL: https://github.com/apache/pulsar/issues/22601#issuecomment-2117009710 I also test again if publish payload always 20K, it won't happen only happen when normal is 2K but some data bigger than 16K( sound like netty receive buffer size but I also

Re: [PR] [improve][broker] Close protocol handlers before unloading namespace bundles [pulsar]

2024-05-17 Thread via GitHub
BewareMyPower commented on code in PR #22728: URL: https://github.com/apache/pulsar/pull/22728#discussion_r1604524990 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java: ## @@ -444,13 +444,20 @@ public void close() throws PulsarServerException { publ

Re: [PR] [improve][broker] Close protocol handlers before unloading namespace bundles [pulsar]

2024-05-17 Thread via GitHub
BewareMyPower commented on code in PR #22728: URL: https://github.com/apache/pulsar/pull/22728#discussion_r1604524990 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java: ## @@ -444,13 +444,20 @@ public void close() throws PulsarServerException { publ

Re: [I] [Bug] parseMessageMetadata error when broker entry metadata enable with high loading [pulsar]

2024-05-17 Thread via GitHub
semistone commented on issue #22601: URL: https://github.com/apache/pulsar/issues/22601#issuecomment-2116971327 @lhotari I am checking when that byteBuf become wrong data and in OpAddEntry.java I verify data when construct this object and save original data and during run(

Re: [PR] [fix][broker] Fix BucketDelayedDeliveryTracker unable to recover [pulsar]

2024-05-17 Thread via GitHub
dao-jun commented on PR #22735: URL: https://github.com/apache/pulsar/pull/22735#issuecomment-2116947331 @coderzc could you please also help review https://github.com/apache/pulsar/pull/22707 when you are available? -- This is an automated message from the Apache Git Service. To respond t

Re: [I] Flaky-test: MockedPulsarServiceBaseTest.deleteNamespaceWithRetry times out [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on issue #22736: URL: https://github.com/apache/pulsar/issues/22736#issuecomment-2116946173 Causes also #20553 -- 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 comme

Re: [PR] [improve][test] Restart broker in SimpleProducerConsumerTest if ns deletion times out [pulsar]

2024-05-17 Thread via GitHub
lhotari commented on PR #22734: URL: https://github.com/apache/pulsar/pull/22734#issuecomment-2116945505 Similar problem in #22736 -- 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 co

[I] Flaky-test: MockedPulsarServiceBaseTest.deleteNamespaceWithRetry times out [pulsar]

2024-05-17 Thread via GitHub
lhotari opened a new issue, #22736: URL: https://github.com/apache/pulsar/issues/22736 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Example failure https://github.com/apache/pulsar/

(pulsar) branch branch-3.3 updated: [fix][build] Fixed creation of `packages-storage` directory in docker image (#22730)

2024-05-17 Thread zhaocong
This is an automated email from the ASF dual-hosted git repository. zhaocong pushed a commit to branch branch-3.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.3 by this push: new ba498fc3b7b [fix][build] Fixed creatio

Re: [PR] [fix][build] Fixed creation of `packages-storage` directory in docker image [pulsar]

2024-05-17 Thread via GitHub
coderzc merged PR #22730: URL: https://github.com/apache/pulsar/pull/22730 -- 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: commits-unsubscr...@pulsar.ap

Re: [I] [Bug] Delayed Message Delivery immediately after owner broker restart [pulsar]

2024-05-17 Thread via GitHub
dao-jun commented on issue #22716: URL: https://github.com/apache/pulsar/issues/22716#issuecomment-2116941088 @coderzc I cannot reproduce the issue on the master branch, could you please take a look? -- This is an automated message from the Apache Git Service. To respond to the message,

(pulsar) branch master updated: [fix][build] Fixed creation of `packages-storage` directory in docker image (#22730)

2024-05-17 Thread zhaocong
This is an automated email from the ASF dual-hosted git repository. zhaocong 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 0c6f2480685 [fix][build] Fixed creation of `pa

Re: [PR] [fix][broker] Fix BucketDelayedDeliveryTracker unable to recover [pulsar]

2024-05-17 Thread via GitHub
coderzc commented on code in PR #22735: URL: https://github.com/apache/pulsar/pull/22735#discussion_r1604480063 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java: ## @@ -1174,6 +1175,47 @@ public boolean trackDe

Re: [I] [Bug] Delayed Message Delivery immediately after owner broker restart [pulsar]

2024-05-17 Thread via GitHub
SennoYuki commented on issue #22716: URL: https://github.com/apache/pulsar/issues/22716#issuecomment-2116930529 > If `isDurable` is true, it means your cursor is durable So I can't fix the situation. I can't restart the broker to tuning config because of duplicated consume happening

Re: [PR] [fix][broker] Fix BucketDelayedDeliveryTracker unable to recover [pulsar]

2024-05-17 Thread via GitHub
dao-jun commented on code in PR #22735: URL: https://github.com/apache/pulsar/pull/22735#discussion_r1604476485 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/DelayedDeliveryTrackerDisable.java: ## @@ -0,0 +1,72 @@ +/* + * Licensed to the Apache Software Foundat

Re: [PR] [fix][broker] Fix BucketDelayedDeliveryTracker unable to recover [pulsar]

2024-05-17 Thread via GitHub
dao-jun commented on PR #22735: URL: https://github.com/apache/pulsar/pull/22735#issuecomment-2116926202 > @dao-jun Can you add tests to cover this case? Yes, but before add test, I want to get more feedbacks, to make sure this change is reasonable -- This is an automated message f

(pulsar) branch branch-3.3 updated: [fix][build] Fix CVE-2024-2511 by upgrading to OpenSSL in docker image (#22731)

2024-05-17 Thread zhaocong
This is an automated email from the ASF dual-hosted git repository. zhaocong pushed a commit to branch branch-3.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-3.3 by this push: new 3fcba6ca685 [fix][build] Fix CVE-2024-

(pulsar) 01/01: [fix][build] Add curl command for pulsar image (#22732)

2024-05-17 Thread zhaocong
This is an automated email from the ASF dual-hosted git repository. zhaocong pushed a commit to branch branch-3.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 62bfb0a170bd1402c9a5eb9143097dd43494f585 Author: Cong Zhao AuthorDate: Fri May 17 15:26:47 2024 +0800 [fix][b

(pulsar) branch branch-3.3 updated (e453a02bb9a -> 62bfb0a170b)

2024-05-17 Thread zhaocong
This is an automated email from the ASF dual-hosted git repository. zhaocong pushed a change to branch branch-3.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git discard e453a02bb9a [fix][schema] Error checking schema compatibility on a schema-less topic via REST API (#22720)

(pulsar) branch master updated: [fix][build] Add curl command for pulsar image (#22732)

2024-05-17 Thread daojun
This is an automated email from the ASF dual-hosted git repository. daojun 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 edde408f781 [fix][build] Add curl command for pu

Re: [PR] [fix][build] Add curl command for pulsar image [pulsar]

2024-05-17 Thread via GitHub
dao-jun merged PR #22732: URL: https://github.com/apache/pulsar/pull/22732 -- 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: commits-unsubscr...@pulsar.ap

Re: [PR] [fix][build] Add curl command for pulsar image [pulsar]

2024-05-17 Thread via GitHub
codecov-commenter commented on PR #22732: URL: https://github.com/apache/pulsar/pull/22732#issuecomment-2116919575 ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/22732?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

Re: [PR] [fix][broker] Fix BucketDelayedDeliveryTracker unable to recover [pulsar]

2024-05-17 Thread via GitHub
coderzc commented on PR #22735: URL: https://github.com/apache/pulsar/pull/22735#issuecomment-2116919101 @dao-jun Can you add tests to cover this case? -- 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 g

Re: [PR] [fix][broker] Fix BucketDelayedDeliveryTracker unable to recover [pulsar]

2024-05-17 Thread via GitHub
coderzc commented on code in PR #22735: URL: https://github.com/apache/pulsar/pull/22735#discussion_r1604467408 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/DelayedDeliveryTrackerDisable.java: ## @@ -0,0 +1,72 @@ +/* + * Licensed to the Apache Software Foundat

(pulsar) branch master updated (23d5e123d3b -> 528c4b0fdb0)

2024-05-17 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git from 23d5e123d3b [improve] [broker] Do not call cursor.isCursorDataFullyPersistable if disabled dispatcherPauseOnAckStat

  1   2   >