(pulsar) branch dependabot/maven/pulsar-io/solr/org.apache.solr-solr-core-8.11.3 deleted (was 1929569fb6a)

2024-02-09 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/solr/org.apache.solr-solr-core-8.11.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git was 1929569fb6a Bump org.apache.solr:solr-core from 8.11.1

(pulsar) branch master updated: [fix] Bump org.apache.solr:solr-core from 8.11.1 to 8.11.3 in /pulsar-io/solr (#22047)

2024-02-09 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 7a90426253e [fix] Bump org.apache.solr:solr-core

Re: [PR] [fix] Bump org.apache.solr:solr-core from 8.11.1 to 8.11.3 in /pulsar-io/solr [pulsar]

2024-02-09 Thread via GitHub
codecov-commenter commented on PR #22047: URL: https://github.com/apache/pulsar/pull/22047#issuecomment-1936783061 ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/22047?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] [fix] Bump org.apache.solr:solr-core from 8.11.1 to 8.11.3 in /pulsar-io/solr [pulsar]

2024-02-09 Thread via GitHub
merlimat merged PR #22047: URL: https://github.com/apache/pulsar/pull/22047 -- 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-320: Add OpenTelemetry scaffolding (#22010)

2024-02-09 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 8957e353ded [feat][misc] PIP-320: Add OpenTeleme

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
merlimat merged PR #22010: URL: https://github.com/apache/pulsar/pull/22010 -- 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

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
dragosvictor commented on code in PR #22010: URL: https://github.com/apache/pulsar/pull/22010#discussion_r1484884025 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java: ## @@ -897,6 +902,7 @@ public void start() throws PulsarServerException {

Re: [PR] Bump org.apache.solr:solr-core from 8.11.1 to 8.11.3 in /pulsar-io/solr [pulsar]

2024-02-09 Thread via GitHub
github-actions[bot] commented on PR #22047: URL: https://github.com/apache/pulsar/pull/22047#issuecomment-1936620319 @dependabot[bot] Please add the following content to your PR description and select a checkbox: ``` - [ ] `doc` - [ ] `doc-required` - [ ] `doc-not-needed` -

(pulsar) branch dependabot/maven/pulsar-io/solr/org.apache.solr-solr-core-8.11.3 created (now 1929569fb6a)

2024-02-09 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/solr/org.apache.solr-solr-core-8.11.3 in repository https://gitbox.apache.org/repos/asf/pulsar.git at 1929569fb6a Bump org.apache.solr:solr-core from 8.11.1

[PR] Bump org.apache.solr:solr-core from 8.11.1 to 8.11.3 in /pulsar-io/solr [pulsar]

2024-02-09 Thread via GitHub
dependabot[bot] opened a new pull request, #22047: URL: https://github.com/apache/pulsar/pull/22047 Bumps org.apache.solr:solr-core from 8.11.1 to 8.11.3. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apac

[I] [Bug] Reference docs is 404'd [pulsar]

2024-02-09 Thread via GitHub
withinboredom opened a new issue, #22046: URL: https://github.com/apache/pulsar/issues/22046 ### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Version https://pulsar.apache.org/docs/3.2.x

Re: [PR] [fix][broker][branch-3.1] Avoid PublishRateLimiter use an already closed RateLimiter [pulsar]

2024-02-09 Thread via GitHub
frankjkelly commented on PR #22011: URL: https://github.com/apache/pulsar/pull/22011#issuecomment-1936543593 > > > > Thanks for the clarification so does that mean the client will retry and if so is that within milliseconds, or seconds or something else? > > > > > > > > > @frankjk

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
codecov-commenter commented on PR #22010: URL: https://github.com/apache/pulsar/pull/22010#issuecomment-1936447675 ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/22010?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] [improve] PIP-335: Pulsar with Oxia integration test [pulsar]

2024-02-09 Thread via GitHub
merlimat merged PR #22045: URL: https://github.com/apache/pulsar/pull/22045 -- 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: [improve] PIP-335: Pulsar with Oxia integration test (#22045)

2024-02-09 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 2b75ca0e02c [improve] PIP-335: Pulsar with Oxia

Re: [PR] [improve] PIP-335: Pulsar with Oxia integration test [pulsar]

2024-02-09 Thread via GitHub
codecov-commenter commented on PR #22045: URL: https://github.com/apache/pulsar/pull/22045#issuecomment-1936438602 ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/22045?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] [fix][broker] Delete current ledger when it is closed [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on PR #22034: URL: https://github.com/apache/pulsar/pull/22034#issuecomment-1936364633 There's a scheduled job https://github.com/apache/pulsar/blob/5df97b41a431a714a448c198c312e98aa4a084e8/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java#L653

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
dragosvictor commented on code in PR #22010: URL: https://github.com/apache/pulsar/pull/22010#discussion_r1484638484 ## pulsar-opentelemetry/src/test/java/org/apache/pulsar/opentelemetry/OpenTelemetryServiceTest.java: ## @@ -0,0 +1,199 @@ +/* + * Licensed to the Apache Software

Re: [PR] [fix][broker] Delete current ledger when it is closed [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on PR #22034: URL: https://github.com/apache/pulsar/pull/22034#issuecomment-1936321236 It seems that the problem might be very different. It would be necessary to explain the use case. The description doesn't explain the use case. If I have understood correctly, the use

Re: [PR] [fix][broker] Delete current ledger when it is closed [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on PR #22034: URL: https://github.com/apache/pulsar/pull/22034#issuecomment-1936293921 Would it make sense to rename "Delete current ledger when it is closed" to "Trim current ledger when it is closed" if that's what is the expectation? -- This is an automated message f

Re: [PR] [improve] PIP-335: Pulsar with Oxia integration test [pulsar]

2024-02-09 Thread via GitHub
merlimat commented on code in PR #22045: URL: https://github.com/apache/pulsar/pull/22045#discussion_r1484578470 ## tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/PulsarInitMetadataContainer.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache

Re: [PR] [improve] PIP-335: Pulsar with Oxia integration test [pulsar]

2024-02-09 Thread via GitHub
heesung-sn commented on code in PR #22045: URL: https://github.com/apache/pulsar/pull/22045#discussion_r1484569644 ## tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/PulsarInitMetadataContainer.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apac

Re: [I] Improve the internal Jersey Client to use a Connection Pool [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on issue #22003: URL: https://github.com/apache/pulsar/issues/22003#issuecomment-1936259756 This issue seems to be a duplicate of #22041. The AsyncHttpClient is used as a Jersey client connector in pulsar-client-admin. I'll close this ticket since #22041 contains a better

Re: [I] Improve the internal Jersey Client to use a Connection Pool [pulsar]

2024-02-09 Thread via GitHub
lhotari closed issue #22003: Improve the internal Jersey Client to use a Connection Pool URL: https://github.com/apache/pulsar/issues/22003 -- 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 specif

Re: [PR] [improve] PIP-335: Pulsar with Oxia integration test [pulsar]

2024-02-09 Thread via GitHub
heesung-sn commented on code in PR #22045: URL: https://github.com/apache/pulsar/pull/22045#discussion_r1484569644 ## tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/PulsarInitMetadataContainer.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apac

Re: [I] Improve the internal Jersey Client to use a Connection Pool [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on issue #22003: URL: https://github.com/apache/pulsar/issues/22003#issuecomment-1936252205 pulsar-client-admin uses AsyncHttpClient , I created #22041 for addressing that. Jetty client is used in the Pulsar Proxy [for proxying /admin endpoint](https://github.com/ap

Re: [PR] [fix] [broker] [branch-3.0] Fast fix infinite HTTP call createSubscriptions caused by wrong topicName [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on PR #21997: URL: https://github.com/apache/pulsar/pull/21997#issuecomment-1936239338 It was also noticed that pulsar-client-admin doesn't have http connection pooling configured resulting in "Connection Limit(2048) reached for " messages in the log. There is #22041 to ad

[PR] [improve] PIP-335: Pulsar with Oxia integration test [pulsar]

2024-02-09 Thread via GitHub
merlimat opened a new pull request, #22045: URL: https://github.com/apache/pulsar/pull/22045 PIP: [335](https://github.com/apache/pulsar/blob/master/pip/pip-335%20Oxia%20metadata%20plugin.md) ### Motivation Added integration test for Pulsar & BookKeeper cluster with Oxia metada

Re: [PR] [fix][broker][branch-3.1] Avoid PublishRateLimiter use an already closed RateLimiter [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on PR #22011: URL: https://github.com/apache/pulsar/pull/22011#issuecomment-1936188709 > > > Thanks for the clarification so does that mean the client will retry and if so is that within milliseconds, or seconds or something else? > > > > > > @frankjkelly I think

Re: [I] [Bug] Bookkeeper autorecovery ReplicationWorker not rereplicating under replicated ledgers. [pulsar]

2024-02-09 Thread via GitHub
horizonzy commented on issue #21987: URL: https://github.com/apache/pulsar/issues/21987#issuecomment-1936146542 Hi, Thomas. Sorry for the late reply. I check the recovery jvm stack, I found that the ReplicationWorker didn't find any underreplication ledger, it is waiting for the incoming

Re: [PR] [fix][proxy] Fix outboundChannel close [pulsar]

2024-02-09 Thread via GitHub
dao-jun commented on PR #22015: URL: https://github.com/apache/pulsar/pull/22015#issuecomment-1936080318 seems netty's issue, but I've checked netty's source code, this is just a WARN log. but if we want to fix it, maybe we can change the [line](https://github.com/netty/netty/blob/4.1/tr

(pulsar-site) branch main updated: Fix missing docker cmd (#785)

2024-02-09 Thread technoboy
This is an automated email from the ASF dual-hosted git repository. technoboy 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 44a3cd53b927 Fix missing docker cmd (#785) 4

Re: [I] [Doc] The pulsar docker deployment command is incorrect [pulsar]

2024-02-09 Thread via GitHub
Technoboy- closed issue #22027: [Doc] The pulsar docker deployment command is incorrect URL: https://github.com/apache/pulsar/issues/22027 -- 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 missing docker cmd [pulsar-site]

2024-02-09 Thread via GitHub
Technoboy- merged PR #785: URL: https://github.com/apache/pulsar-site/pull/785 -- 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

Re: [PR] [feat][io] New component. AzureDataExplorer/Kusto sink for pulsar (https://learn.microsoft.com/en-us/azure/data-explorer/) [pulsar]

2024-02-09 Thread via GitHub
asaharn commented on code in PR #22006: URL: https://github.com/apache/pulsar/pull/22006#discussion_r1484293622 ## pulsar-io/azure-data-explorer/src/main/java/org/apache/pulsar/io/azuredataexplorer/ADXSinkConfig.java: ## @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

Re: [PR] [feat][io] New component. AzureDataExplorer/Kusto sink for pulsar (https://learn.microsoft.com/en-us/azure/data-explorer/) [pulsar]

2024-02-09 Thread via GitHub
asaharn commented on code in PR #22006: URL: https://github.com/apache/pulsar/pull/22006#discussion_r1484292510 ## pulsar-io/azure-data-explorer/src/main/java/org/apache/pulsar/io/azuredataexplorer/ADXSinkConfig.java: ## @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

Re: [PR] [fix][broker][branch-3.1] Avoid PublishRateLimiter use an already closed RateLimiter [pulsar]

2024-02-09 Thread via GitHub
frankjkelly commented on PR #22011: URL: https://github.com/apache/pulsar/pull/22011#issuecomment-1935889856 > > Thanks for the clarification so does that mean the client will retry and if so is that within milliseconds, or seconds or something else? > > @frankjkelly I think the clien

Re: [PR] [feat][io] New component. AzureDataExplorer/Kusto sink for pulsar (https://learn.microsoft.com/en-us/azure/data-explorer/) [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on code in PR #22006: URL: https://github.com/apache/pulsar/pull/22006#discussion_r1484216970 ## pulsar-io/azure-data-explorer/src/main/java/org/apache/pulsar/io/azuredataexplorer/ADXSinkConfig.java: ## @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

Re: [PR] [feat][io] New component. AzureDataExplorer/Kusto sink for pulsar (https://learn.microsoft.com/en-us/azure/data-explorer/) [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on code in PR #22006: URL: https://github.com/apache/pulsar/pull/22006#discussion_r1484208319 ## pulsar-io/azure-data-explorer/src/main/java/org/apache/pulsar/io/azuredataexplorer/ADXSinkConfig.java: ## @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on code in PR #22010: URL: https://github.com/apache/pulsar/pull/22010#discussion_r1484124406 ## pulsar-opentelemetry/src/test/java/org/apache/pulsar/opentelemetry/OpenTelemetryServiceTest.java: ## @@ -0,0 +1,199 @@ +/* + * Licensed to the Apache Software Found

Re: [PR] [fix][broker][branch-3.1] Avoid PublishRateLimiter use an already closed RateLimiter [pulsar]

2024-02-09 Thread via GitHub
coderzc commented on PR #22011: URL: https://github.com/apache/pulsar/pull/22011#issuecomment-1935656478 > Thanks for the clarification so does that mean the client will retry and if so is that within milliseconds, or seconds or something else? @frankjkelly I think the client does not

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on code in PR #22010: URL: https://github.com/apache/pulsar/pull/22010#discussion_r1484076947 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java: ## @@ -897,6 +902,7 @@ public void start() throws PulsarServerException { }

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
lhotari commented on code in PR #22010: URL: https://github.com/apache/pulsar/pull/22010#discussion_r1484071300 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java: ## @@ -897,6 +902,7 @@ public void start() throws PulsarServerException { }

Re: [PR] [feat][misc] PIP-320: Add OpenTelemetry scaffolding [pulsar]

2024-02-09 Thread via GitHub
dragosvictor commented on code in PR #22010: URL: https://github.com/apache/pulsar/pull/22010#discussion_r1484067271 ## pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/stats/PulsarWorkerOpenTelemetry.java: ## @@ -0,0 +1,42 @@ +/* + * Licensed to the

(pulsar) branch branch-2.11 updated: [fix][broker] Sanitize values before logging in apply-config-from-env.py script (#22044)

2024-02-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.11 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.11 by this push: new 4d320e780b7 [fix][broker] Sanitize va

(pulsar) branch branch-3.1 updated: [fix][broker] Sanitize values before logging in apply-config-from-env.py script (#22044)

2024-02-09 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 9a8761e204f [fix][broker] Sanitize valu

(pulsar) branch branch-3.2 updated: [fix][broker] Sanitize values before logging in apply-config-from-env.py script (#22044)

2024-02-09 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 14342058803 [fix][broker] Sanitize valu

(pulsar) branch branch-3.0 updated: [fix][broker] Sanitize values before logging in apply-config-from-env.py script (#22044)

2024-02-09 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 a88f592a759 [fix][broker] Sanitize valu

(pulsar) branch branch-2.10 updated: [fix][broker] Sanitize values before logging in apply-config-from-env.py script (#22044)

2024-02-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 f5922bc2730 [fix][broker] Sanitize va