Re: [PR] [feat] added a slog wrapper of the logger interface [pulsar-client-go]

2024-07-05 Thread via GitHub
ivan-penchev commented on PR #1234: URL: https://github.com/apache/pulsar-client-go/pull/1234#issuecomment-2210350936 If everyone is OK with the suggested changes, would someone be able to merge this in? cc @nodece @crossoverJie @RobertIndie -- This is an automated message from the A

Re: [PR] [fix][broker] Replication stuck when partitions count between two clusters is not the same [pulsar]

2024-07-05 Thread via GitHub
poorbarcode commented on PR #22983: URL: https://github.com/apache/pulsar/pull/22983#issuecomment-2210358547 /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 sp

Re: [PR] [fix][admin] Fix half deletion when attempt to topic with a incorrect API [pulsar]

2024-07-05 Thread via GitHub
poorbarcode commented on PR #23002: URL: https://github.com/apache/pulsar/pull/23002#issuecomment-2210359336 /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 sp

Re: [PR] [feat][misc] PIP-264: Copy OpenTelemetry resource attributes to Prometheus labels [pulsar]

2024-07-05 Thread via GitHub
merlimat merged PR #23005: URL: https://github.com/apache/pulsar/pull/23005 -- 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) branch master updated: [feat][misc] PIP-264: Copy OpenTelemetry resource attributes to Prometheus labels (#23005)

2024-07-05 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 dd1b57944b1 [feat][misc] PIP-264: Copy OpenTelem

Re: [PR] (WIP) Build deb packages via vcpkg [pulsar-client-cpp]

2024-07-05 Thread via GitHub
BewareMyPower closed pull request #375: (WIP) Build deb packages via vcpkg URL: https://github.com/apache/pulsar-client-cpp/pull/375 -- 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] [fix][broker] Replication stuck when partitions count between two clusters is not the same [pulsar]

2024-07-05 Thread via GitHub
Demogorgon314 commented on code in PR #22983: URL: https://github.com/apache/pulsar/pull/22983#discussion_r1666513004 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/service/OneWayReplicatorTest.java: ## @@ -1034,4 +1040,88 @@ public void testConfigReplicationStartAt() t

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
Demogorgon314 commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r1650555346 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/AvgShedder.java: ## @@ -0,0 +1,316 @@ +/** + * Licensed to the Apache Software Foundatio

[PR] Fix transaction coordinator client cann't reconnect to the broker [pulsar-client-go]

2024-07-05 Thread via GitHub
RobertIndie opened a new pull request, #1237: URL: https://github.com/apache/pulsar-client-go/pull/1237 *(If this PR fixes a github issue, please add `Fixes #`.)* Fixes # *(or if this PR is one task of a github issue, please add `Master Issue: #` to link to the master i

Re: [PR] [feat][broker] PIP-264: Add OpenTelemetry managed cursor metrics [pulsar]

2024-07-05 Thread via GitHub
merlimat merged PR #23000: URL: https://github.com/apache/pulsar/pull/23000 -- 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) branch master updated: [feat][broker] PIP-264: Add OpenTelemetry managed cursor metrics (#23000)

2024-07-05 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 8351c079d8e [feat][broker] PIP-264: Add OpenTele

Re: [PR] [feat] added a slog wrapper of the logger interface [pulsar-client-go]

2024-07-05 Thread via GitHub
RobertIndie commented on PR #1234: URL: https://github.com/apache/pulsar-client-go/pull/1234#issuecomment-2210572269 > but I noticed that the [v0.13.0](https://github.com/apache/pulsar-client-go/milestone/15) milestone has a deadline in 2 days. > And was wondering, due to the nature of

Re: [PR] [feat] added a slog wrapper of the logger interface [pulsar-client-go]

2024-07-05 Thread via GitHub
RobertIndie merged PR #1234: URL: https://github.com/apache/pulsar-client-go/pull/1234 -- 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-go) branch master updated: [feat] added a slog wrapper of the logger interface (#1234)

2024-07-05 Thread zike
This is an automated email from the ASF dual-hosted git repository. zike pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 1152cfc2 [feat] added a slog wrapper of

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
thetumbled commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r118448 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/AvgShedder.java: ## @@ -0,0 +1,312 @@ +/* + * Licensed to the Apache Software Foundation (A

(pulsar) branch branch-3.0 updated: [fix] Make operations on `individualDeletedMessages` in lock scope (#22966)

2024-07-05 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 e01e90f0bf7 [fix] Make operations on `i

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
Demogorgon314 commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r134368 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java: ## @@ -270,7 +270,12 @@ public void initialize(final Pulsar

(pulsar) branch branch-3.0 updated: [fix][ci] Fix OWASP Dependency Check download by using NVD API key (#22999)

2024-07-05 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 29d7150f6c2 [fix][ci] Fix OWASP Depende

(pulsar) branch branch-3.1 updated: [fix][ci] Fix OWASP Dependency Check download by using NVD API key (#22999)

2024-07-05 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 1b4b92082fa [fix][ci] Fix OWASP Depende

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
thetumbled commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r1666757425 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java: ## @@ -270,7 +270,12 @@ public void initialize(final PulsarSer

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
Demogorgon314 commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r1666766200 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java: ## @@ -270,7 +270,12 @@ public void initialize(final Pulsar

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
thetumbled commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r1666768514 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java: ## @@ -270,7 +270,12 @@ public void initialize(final PulsarSer

(pulsar) branch branch-3.2 updated: [fix] Make operations on `individualDeletedMessages` in lock scope (#22966)

2024-07-05 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 9e5255cec72 [fix] Make operations on `i

(pulsar) branch branch-3.3 updated: [fix] Make operations on `individualDeletedMessages` in lock scope (#22966)

2024-07-05 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 2e99b706a99 [fix] Make operations on `i

Re: [PR] [improve][broker] log exception in MessageDeduplication#takeSnapshot [pulsar]

2024-07-05 Thread via GitHub
lhotari closed pull request #22994: [improve][broker] log exception in MessageDeduplication#takeSnapshot URL: https://github.com/apache/pulsar/pull/22994 -- 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

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
Demogorgon314 commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r1666868940 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java: ## @@ -270,7 +270,12 @@ public void initialize(final Pulsar

Re: [PR] [improve][broker] log exception in MessageDeduplication#takeSnapshot [pulsar]

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

Re: [PR] Add a note about increases in backlog when replicated subscription is activated [pulsar-site]

2024-07-05 Thread via GitHub
AlvaroStream commented on PR #916: URL: https://github.com/apache/pulsar-site/pull/916#issuecomment-2210999728 Sorry I missed the ping. Updated. -- 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 th

Re: [PR] [improve] [pip] PIP-364: Introduce a new load balance algorithm AvgShedder [pulsar]

2024-07-05 Thread via GitHub
thetumbled commented on code in PR #22949: URL: https://github.com/apache/pulsar/pull/22949#discussion_r1666934066 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java: ## @@ -270,7 +270,12 @@ public void initialize(final PulsarSer

Re: [PR] [fix][admin] Fix half deletion when attempt to topic with a incorrect API [pulsar]

2024-07-05 Thread via GitHub
nodece commented on code in PR #23002: URL: https://github.com/apache/pulsar/pull/23002#discussion_r1666959049 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java: ## @@ -742,7 +742,17 @@ protected void internalDeletePartitionedTopic(Asy

Re: [PR] [fix][admin] Fix half deletion when attempt to topic with a incorrect API [pulsar]

2024-07-05 Thread via GitHub
nodece commented on code in PR #23002: URL: https://github.com/apache/pulsar/pull/23002#discussion_r1666960439 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java: ## @@ -742,7 +742,17 @@ protected void internalDeletePartitionedTopic(Asy

Re: [PR] [fix][admin] Fix half deletion when attempt to topic with a incorrect API [pulsar]

2024-07-05 Thread via GitHub
nodece commented on code in PR #23002: URL: https://github.com/apache/pulsar/pull/23002#discussion_r1666960652 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/PersistentTopics.java: ## @@ -1167,7 +1167,17 @@ public void deleteTopic( @ApiParam(value

Re: [PR] [fix][admin] Fix half deletion when attempt to topic with a incorrect API [pulsar]

2024-07-05 Thread via GitHub
nodece commented on code in PR #23002: URL: https://github.com/apache/pulsar/pull/23002#discussion_r1666963241 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminTopicApiTest.java: ## @@ -71,6 +75,62 @@ protected void cleanup() throws Exception { super.i

Re: [PR] [fix][admin] Fix half deletion when attempt to topic with a incorrect API [pulsar]

2024-07-05 Thread via GitHub
nodece commented on code in PR #23002: URL: https://github.com/apache/pulsar/pull/23002#discussion_r1666965046 ## pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminTopicApiTest.java: ## @@ -71,6 +75,62 @@ protected void cleanup() throws Exception { super.i

Re: [PR] [fix][broker] Replication stuck when partitions count between two clusters is not the same [pulsar]

2024-07-05 Thread via GitHub
nodece commented on code in PR #22983: URL: https://github.com/apache/pulsar/pull/22983#discussion_r1666970494 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractReplicator.java: ## @@ -184,8 +187,42 @@ public void startProducer() { } log

Re: [PR] [fix][broker] Fix MessageDeduplication replay timeout cause topic loading stuck [pulsar]

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

[I] [Bug] Segmentation Fault when using Pulsar Consumer batchReceive [pulsar-client-node]

2024-07-05 Thread via GitHub
rafael-polakiewicz opened a new issue, #390: URL: https://github.com/apache/pulsar-client-node/issues/390 # Description I encountered a segmentation fault using pulsar consumer batch receive method. The consumer process terminates unexpectedly with a segmentation fault. # Node and

[PR] [feat][doc] PIP-264: Update OpenTelemetry deployment documentation [pulsar-site]

2024-07-05 Thread via GitHub
dragosvictor opened a new pull request, #934: URL: https://github.com/apache/pulsar-site/pull/934 ### ✅ Contribution Checklist - [x] I read the [contribution guide](https://pulsar.apache.org/contribute/document-contribution/) - [x] I updated the [versioned docs](https://pulsar.apa

Re: [PR] Add a note about increases in backlog when replicated subscription is activated [pulsar-site]

2024-07-05 Thread via GitHub
visortelle merged PR #916: URL: https://github.com/apache/pulsar-site/pull/916 -- 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...@pulsa

(pulsar-site) branch main updated: Add a note about increases in backlog when replicated subscription is activated (#916)

2024-07-05 Thread visortelle
This is an automated email from the ASF dual-hosted git repository. visortelle 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 8a9492403c6b Add a note about increases in

Re: [PR] [fix][broker] Replication stuck when partitions count between two clusters is not the same [pulsar]

2024-07-05 Thread via GitHub
poorbarcode commented on code in PR #22983: URL: https://github.com/apache/pulsar/pull/22983#discussion_r1667074885 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ProducerConfigurationData.java: ## @@ -204,6 +204,8 @@ public class ProducerConfigurationData imp

Re: [PR] [fix][broker] Replication stuck when partitions count between two clusters is not the same [pulsar]

2024-07-05 Thread via GitHub
poorbarcode commented on code in PR #22983: URL: https://github.com/apache/pulsar/pull/22983#discussion_r1667078640 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractReplicator.java: ## @@ -184,8 +187,42 @@ public void startProducer() { }

Re: [PR] [improve][broker] Use RoaringBitmap in tracking individual acks to reduce memory usage [pulsar]

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

Re: [PR] [fix][broker] Fix MessageDeduplication replay timeout cause topic loading stuck [pulsar]

2024-07-05 Thread via GitHub
dao-jun merged PR #23004: URL: https://github.com/apache/pulsar/pull/23004 -- 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][broker] MessageDeduplication replay timeout would cause topic loading stuck and become unavailable [pulsar]

2024-07-05 Thread via GitHub
dao-jun closed issue #23003: [Bug][broker] MessageDeduplication replay timeout would cause topic loading stuck and become unavailable URL: https://github.com/apache/pulsar/issues/23003 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

(pulsar) branch master updated: [fix][broker] Fix MessageDeduplication replay timeout cause topic loading stuck (#23004)

2024-07-05 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 41ef3f6fb1c [fix][broker] Fix MessageDeduplicati

[PR] Bump certifi from 2023.7.22 to 2024.7.4 in /tools/pytools [pulsar-site]

2024-07-05 Thread via GitHub
dependabot[bot] opened a new pull request, #935: URL: https://github.com/apache/pulsar-site/pull/935 Bumps [certifi](https://github.com/certifi/python-certifi) from 2023.7.22 to 2024.7.4. Commits https://github.com/certifi/python-certifi/commit/bd8153872e9c6fc98f4023df9c2deaffe

(pulsar-site) branch dependabot/pip/tools/pytools/certifi-2024.7.4 created (now f44ec32def54)

2024-07-05 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/pip/tools/pytools/certifi-2024.7.4 in repository https://gitbox.apache.org/repos/asf/pulsar-site.git at f44ec32def54 Bump certifi from 2023.7.22 to 2024.7.4 in /tools/pytool

Re: [PR] Bump certifi from 2023.7.22 to 2024.7.4 in /tools/pytools [pulsar-site]

2024-07-05 Thread via GitHub
visortelle merged PR #935: URL: https://github.com/apache/pulsar-site/pull/935 -- 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...@pulsa

(pulsar-site) branch main updated: Bump certifi from 2023.7.22 to 2024.7.4 in /tools/pytools (#935)

2024-07-05 Thread visortelle
This is an automated email from the ASF dual-hosted git repository. visortelle 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 937e875390ac Bump certifi from 2023.7.22 to

(pulsar-site) branch dependabot/pip/tools/pytools/certifi-2024.7.4 deleted (was f44ec32def54)

2024-07-05 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/pip/tools/pytools/certifi-2024.7.4 in repository https://gitbox.apache.org/repos/asf/pulsar-site.git was f44ec32def54 Bump certifi from 2023.7.22 to 2024.7.4 in /tools/pytool