You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/09/15 03:53:01 UTC
[GitHub] [pulsar-site] Anonymitaet commented on a diff in pull request #209: Pulsar 2.8.4 release notes
Anonymitaet commented on code in PR #209:
URL: https://github.com/apache/pulsar-site/pull/209#discussion_r971497129
##########
site2/website-next/release-notes/versioned/pulsar-2.8.4.md:
##########
@@ -0,0 +1,179 @@
+---
+id: pulsar-2.8.4
+title: Apache Pulsar 2.8.4
+sidebar_label: Apache Pulsar 2.8.4
+---
+
+#### Broker
+- [improve][broker] Do not print the log that the topic does not exist [13535](https://github.com/apache/pulsar/pull/13535)
+- [fix][broker] Fix NPE when nextValidLedger is null [13975](https://github.com/apache/pulsar/pull/13975)
+- [fix][broker] Fix NPE when `ReplicatedSubscriptionsController` sends marker message with deduplication enabled [14017](https://github.com/apache/pulsar/pull/14017)
+- [fix][broker] Fix avg-messagePerEntry metrics for consumer [14330](https://github.com/apache/pulsar/pull/14330)
+- [improve][broker] Validate rack name when setting the bookie rack info [14336](https://github.com/apache/pulsar/pull/14336)
+- [improve][broker] Optimize find nics process. [14340](https://github.com/apache/pulsar/pull/14340)
+- [fix][broker] Fix NPE when subscription is already removed [14363](https://github.com/apache/pulsar/pull/14363)
+- [fix][broker] Fix Broker HealthCheck Endpoint Exposes Race Conditions [14367](https://github.com/apache/pulsar/pull/14367)
+- [improve][broker] Optimize RawReader#create when using Compactor [14447](https://github.com/apache/pulsar/pull/14447)
+- [fix][broker] Fix the latest message of the compacted topic cannot be read [14449](https://github.com/apache/pulsar/pull/14449)
+- [fix][broker] Fix producerFuture not completed in ServerCnx#handleProducer [14467](https://github.com/apache/pulsar/pull/14467)
+- [fix][broker] Fix NPW in ManagedLedgerImpl [14481](https://github.com/apache/pulsar/pull/14481)
+- [improve][broker] Support shrink in ConcurrentLongHashMap [14497](https://github.com/apache/pulsar/pull/14497)
+- [fix][broker] Fixed wrong behaviour caused by not cleaning up topic policy service state. [14503](https://github.com/apache/pulsar/pull/14503)
+- [improve][broker] Optimize memory usage: support to shrink for pendingAcks map [14515](https://github.com/apache/pulsar/pull/14515)
+- [fix][broker] Cancel offload tasks when managed ledger closed [14744](https://github.com/apache/pulsar/pull/14744)
+- [improve][broker] Reduce unnecessary expansions for ConcurrentLong map and set [14562](https://github.com/apache/pulsar/pull/14562)
+- [fix][broker] Fix metadata store deadlock when checking BacklogQuota [14634](https://github.com/apache/pulsar/pull/14634)
+- [improve][broker] Support shrink for map or set [14663](https://github.com/apache/pulsar/pull/14663)
+- [fix][broker] Fix lost message issue due to ledger rollover [14664](https://github.com/apache/pulsar/pull/14664)
+- [fix][broker] Set splitNamespaceBundle with `readonly=false` [14680](https://github.com/apache/pulsar/pull/14680)
+- [fix][broker] Fixed duplicated delayed messages when all consumers disconnect [14740](https://github.com/apache/pulsar/pull/14740)
+- [fix][broker] Fix wrong state for non-durable cursor [14869](https://github.com/apache/pulsar/pull/14869)
+- [fix][broker] Improve error logging for topic not found [14892](https://github.com/apache/pulsar/pull/14892)
+- [fix][broker] Fix topic policy reader close bug [14897](https://github.com/apache/pulsar/pull/14897)
+- [fix][broker] Return if reset in progress [14978](https://github.com/apache/pulsar/pull/14978)
+- [fix][broker] Fix potential NPE in Replicator [15003](https://github.com/apache/pulsar/pull/15003)
+- [fix][broker] Fix race condition in updating lastMarkDeleteEntry field [15031](https://github.com/apache/pulsar/pull/15031)
+- [fix][broker] Fix potential to add duplicated consumer [16826](https://github.com/apache/pulsar/pull/16826)
+- [fix][broker] Follow up on race condition fixes in ManagedCursorImpl #15031 [15067](https://github.com/apache/pulsar/pull/15067)
+- [cleanup] [broker] Remove useless code to avoid confusion in OpReadEntry#checkReadCompletion [15104](https://github.com/apache/pulsar/pull/15104)
+- [fix][broker] Cancel `fencedTopicMonitoringTask` when topic closed normally. [15202](https://github.com/apache/pulsar/pull/15202)
+- [fix][broker] Fix MessageDeduplication#inactiveProducers may not be persistence correctly [15206](https://github.com/apache/pulsar/pull/15206)
+- [fix][broker] Fix race condition between timeout and completion in `OpAddEntry` [15233](https://github.com/apache/pulsar/pull/15233)
+- [fix][broker] Fix problem at RateLimiter#tryAcquire [15306](https://github.com/apache/pulsar/pull/15306)
+- [improve][broker] Use shrink map for message redelivery. [15342](https://github.com/apache/pulsar/pull/15342)
+- [improve][broker] Support shrink for ConcurrentSortedLongPairSet [15354](https://github.com/apache/pulsar/pull/15354)
+- [improve][broker] Close connection if a ping or pong message cannot be sent [15382](https://github.com/apache/pulsar/pull/15382)
+- [cleanup][broker] Fix typo in enum name and handle closing of the channel properly since writeAndFlush is asynchronous [15384](https://github.com/apache/pulsar/pull/15384)
+- [fix][broker] Avoid heartbeat topic to offload. [15393](https://github.com/apache/pulsar/pull/15393)
+- [fix][broker] Fix deadlock in broker after race condition in topic creation failure [15570](https://github.com/apache/pulsar/pull/15570)
+- [fix][broker] fix calculation in getNumberOfEntriesInStorage [15627](https://github.com/apache/pulsar/pull/15627)
+- [fix][broker] Fix NPE when set `AutoTopicCreationOverride` [15653](https://github.com/apache/pulsar/pull/15653)
+- [fix][broker] Fix creating producer failure when set backlog quota [15663](https://github.com/apache/pulsar/pull/15663)
+- [fix][broker] Fix NPE when put value to `RangeCache` [15707](https://github.com/apache/pulsar/pull/15707)
+- [fix][broker] Fix can not enable system topic if `AutoUpdateSchemaEnabled=false` [15759](https://github.com/apache/pulsar/pull/15759)
+- [fix][broker] Fix wrong unit of NIC speed on linux [15770](https://github.com/apache/pulsar/pull/15770)
+- [fix][broker] Fix NPE in MessageDeduplication [15820](https://github.com/apache/pulsar/pull/15820)
+- [fix][broker] Fix NPE when ledger id not found in `OpReadEntry` [15837](https://github.com/apache/pulsar/pull/15837)
+- [fix][broker] When skipping updating mark delete position, execute callback with executor to prevent deadlock [15971](https://github.com/apache/pulsar/pull/15971)
+- [fix][broker] Fix NPE when drop backlog for time limit. [16235](https://github.com/apache/pulsar/pull/16235)
+- [fix][broker]Fix getInternalStats occasional lack of LeaderInfo again [16238](https://github.com/apache/pulsar/pull/16238)
+- [fix][broker] Fix RawReader out of order [16390](https://github.com/apache/pulsar/pull/16390)
+- [fix][broker] Skip reading more entries for a pending read with no more entries [16400](https://github.com/apache/pulsar/pull/16400)
+- [fix][broker] Fix No such ledger exception [16420](https://github.com/apache/pulsar/pull/16420)
+- [fix][broker] Fix RawReader hasMessageAvailable returns true when no messages [16443](https://github.com/apache/pulsar/pull/16443)
+- [fix][broker] Fix stats-internal with option -m cause active ledger recover then close [16662](https://github.com/apache/pulsar/pull/16662)
+- [fix][broker] Fix passing incorrect authentication data [16840](https://github.com/apache/pulsar/pull/16840)
+- [fix][broker] ManagedCursor: mark delete no callback when create meta-ledger fail [16841](https://github.com/apache/pulsar/pull/16841)
+- [improve][broker] Avoid reconnection when a partitioned topic was created concurrently [16867](https://github.com/apache/pulsar/pull/16867)
+- [improve][broker] Recycle OpReadEntry in some corner cases [16869](https://github.com/apache/pulsar/pull/16869)
+- [fix][broker] Fix compaction subscription acknowledge Marker msg issue. [16918](https://github.com/apache/pulsar/pull/16918)
+
+#### Security
+- [fix][sec] Use provider's canLookupAsync method for AuthorizationService [11777](https://github.com/apache/pulsar/pull/11777)
+- [improve][sec] Optimize the logic of allowing namespace operation [13090](https://github.com/apache/pulsar/pull/13090)
+- [improve][sec] Full-support set ssl provider, ciphers and protocols [13740](https://github.com/apache/pulsar/pull/13740)
Review Comment:
```suggestion
- [improve][sec] Full-support set SSL provider, ciphers and protocols [13740](https://github.com/apache/pulsar/pull/13740)
```
##########
site2/website-next/release-notes/versioned/pulsar-2.8.4.md:
##########
@@ -0,0 +1,179 @@
+---
+id: pulsar-2.8.4
+title: Apache Pulsar 2.8.4
+sidebar_label: Apache Pulsar 2.8.4
+---
+
+#### Broker
+- [improve][broker] Do not print the log that the topic does not exist [13535](https://github.com/apache/pulsar/pull/13535)
+- [fix][broker] Fix NPE when nextValidLedger is null [13975](https://github.com/apache/pulsar/pull/13975)
+- [fix][broker] Fix NPE when `ReplicatedSubscriptionsController` sends marker message with deduplication enabled [14017](https://github.com/apache/pulsar/pull/14017)
+- [fix][broker] Fix avg-messagePerEntry metrics for consumer [14330](https://github.com/apache/pulsar/pull/14330)
+- [improve][broker] Validate rack name when setting the bookie rack info [14336](https://github.com/apache/pulsar/pull/14336)
+- [improve][broker] Optimize find nics process. [14340](https://github.com/apache/pulsar/pull/14340)
+- [fix][broker] Fix NPE when subscription is already removed [14363](https://github.com/apache/pulsar/pull/14363)
+- [fix][broker] Fix Broker HealthCheck Endpoint Exposes Race Conditions [14367](https://github.com/apache/pulsar/pull/14367)
+- [improve][broker] Optimize RawReader#create when using Compactor [14447](https://github.com/apache/pulsar/pull/14447)
+- [fix][broker] Fix the latest message of the compacted topic cannot be read [14449](https://github.com/apache/pulsar/pull/14449)
+- [fix][broker] Fix producerFuture not completed in ServerCnx#handleProducer [14467](https://github.com/apache/pulsar/pull/14467)
+- [fix][broker] Fix NPW in ManagedLedgerImpl [14481](https://github.com/apache/pulsar/pull/14481)
+- [improve][broker] Support shrink in ConcurrentLongHashMap [14497](https://github.com/apache/pulsar/pull/14497)
+- [fix][broker] Fixed wrong behaviour caused by not cleaning up topic policy service state. [14503](https://github.com/apache/pulsar/pull/14503)
+- [improve][broker] Optimize memory usage: support to shrink for pendingAcks map [14515](https://github.com/apache/pulsar/pull/14515)
+- [fix][broker] Cancel offload tasks when managed ledger closed [14744](https://github.com/apache/pulsar/pull/14744)
+- [improve][broker] Reduce unnecessary expansions for ConcurrentLong map and set [14562](https://github.com/apache/pulsar/pull/14562)
+- [fix][broker] Fix metadata store deadlock when checking BacklogQuota [14634](https://github.com/apache/pulsar/pull/14634)
+- [improve][broker] Support shrink for map or set [14663](https://github.com/apache/pulsar/pull/14663)
+- [fix][broker] Fix lost message issue due to ledger rollover [14664](https://github.com/apache/pulsar/pull/14664)
+- [fix][broker] Set splitNamespaceBundle with `readonly=false` [14680](https://github.com/apache/pulsar/pull/14680)
+- [fix][broker] Fixed duplicated delayed messages when all consumers disconnect [14740](https://github.com/apache/pulsar/pull/14740)
+- [fix][broker] Fix wrong state for non-durable cursor [14869](https://github.com/apache/pulsar/pull/14869)
+- [fix][broker] Improve error logging for topic not found [14892](https://github.com/apache/pulsar/pull/14892)
+- [fix][broker] Fix topic policy reader close bug [14897](https://github.com/apache/pulsar/pull/14897)
+- [fix][broker] Return if reset in progress [14978](https://github.com/apache/pulsar/pull/14978)
+- [fix][broker] Fix potential NPE in Replicator [15003](https://github.com/apache/pulsar/pull/15003)
+- [fix][broker] Fix race condition in updating lastMarkDeleteEntry field [15031](https://github.com/apache/pulsar/pull/15031)
+- [fix][broker] Fix potential to add duplicated consumer [16826](https://github.com/apache/pulsar/pull/16826)
+- [fix][broker] Follow up on race condition fixes in ManagedCursorImpl #15031 [15067](https://github.com/apache/pulsar/pull/15067)
+- [cleanup] [broker] Remove useless code to avoid confusion in OpReadEntry#checkReadCompletion [15104](https://github.com/apache/pulsar/pull/15104)
+- [fix][broker] Cancel `fencedTopicMonitoringTask` when topic closed normally. [15202](https://github.com/apache/pulsar/pull/15202)
+- [fix][broker] Fix MessageDeduplication#inactiveProducers may not be persistence correctly [15206](https://github.com/apache/pulsar/pull/15206)
+- [fix][broker] Fix race condition between timeout and completion in `OpAddEntry` [15233](https://github.com/apache/pulsar/pull/15233)
+- [fix][broker] Fix problem at RateLimiter#tryAcquire [15306](https://github.com/apache/pulsar/pull/15306)
+- [improve][broker] Use shrink map for message redelivery. [15342](https://github.com/apache/pulsar/pull/15342)
+- [improve][broker] Support shrink for ConcurrentSortedLongPairSet [15354](https://github.com/apache/pulsar/pull/15354)
+- [improve][broker] Close connection if a ping or pong message cannot be sent [15382](https://github.com/apache/pulsar/pull/15382)
+- [cleanup][broker] Fix typo in enum name and handle closing of the channel properly since writeAndFlush is asynchronous [15384](https://github.com/apache/pulsar/pull/15384)
+- [fix][broker] Avoid heartbeat topic to offload. [15393](https://github.com/apache/pulsar/pull/15393)
+- [fix][broker] Fix deadlock in broker after race condition in topic creation failure [15570](https://github.com/apache/pulsar/pull/15570)
+- [fix][broker] fix calculation in getNumberOfEntriesInStorage [15627](https://github.com/apache/pulsar/pull/15627)
+- [fix][broker] Fix NPE when set `AutoTopicCreationOverride` [15653](https://github.com/apache/pulsar/pull/15653)
+- [fix][broker] Fix creating producer failure when set backlog quota [15663](https://github.com/apache/pulsar/pull/15663)
+- [fix][broker] Fix NPE when put value to `RangeCache` [15707](https://github.com/apache/pulsar/pull/15707)
+- [fix][broker] Fix can not enable system topic if `AutoUpdateSchemaEnabled=false` [15759](https://github.com/apache/pulsar/pull/15759)
+- [fix][broker] Fix wrong unit of NIC speed on linux [15770](https://github.com/apache/pulsar/pull/15770)
+- [fix][broker] Fix NPE in MessageDeduplication [15820](https://github.com/apache/pulsar/pull/15820)
+- [fix][broker] Fix NPE when ledger id not found in `OpReadEntry` [15837](https://github.com/apache/pulsar/pull/15837)
+- [fix][broker] When skipping updating mark delete position, execute callback with executor to prevent deadlock [15971](https://github.com/apache/pulsar/pull/15971)
+- [fix][broker] Fix NPE when drop backlog for time limit. [16235](https://github.com/apache/pulsar/pull/16235)
+- [fix][broker]Fix getInternalStats occasional lack of LeaderInfo again [16238](https://github.com/apache/pulsar/pull/16238)
+- [fix][broker] Fix RawReader out of order [16390](https://github.com/apache/pulsar/pull/16390)
+- [fix][broker] Skip reading more entries for a pending read with no more entries [16400](https://github.com/apache/pulsar/pull/16400)
+- [fix][broker] Fix No such ledger exception [16420](https://github.com/apache/pulsar/pull/16420)
+- [fix][broker] Fix RawReader hasMessageAvailable returns true when no messages [16443](https://github.com/apache/pulsar/pull/16443)
+- [fix][broker] Fix stats-internal with option -m cause active ledger recover then close [16662](https://github.com/apache/pulsar/pull/16662)
+- [fix][broker] Fix passing incorrect authentication data [16840](https://github.com/apache/pulsar/pull/16840)
+- [fix][broker] ManagedCursor: mark delete no callback when create meta-ledger fail [16841](https://github.com/apache/pulsar/pull/16841)
+- [improve][broker] Avoid reconnection when a partitioned topic was created concurrently [16867](https://github.com/apache/pulsar/pull/16867)
+- [improve][broker] Recycle OpReadEntry in some corner cases [16869](https://github.com/apache/pulsar/pull/16869)
+- [fix][broker] Fix compaction subscription acknowledge Marker msg issue. [16918](https://github.com/apache/pulsar/pull/16918)
+
+#### Security
+- [fix][sec] Use provider's canLookupAsync method for AuthorizationService [11777](https://github.com/apache/pulsar/pull/11777)
+- [improve][sec] Optimize the logic of allowing namespace operation [13090](https://github.com/apache/pulsar/pull/13090)
+- [improve][sec] Full-support set ssl provider, ciphers and protocols [13740](https://github.com/apache/pulsar/pull/13740)
+- [fix][sec] Role with namespace produce authz can also get topics [13773](https://github.com/apache/pulsar/pull/13773)
+- [improve][sec] Full-support ssl provider, ciphers and protocols for broker service and proxy service [15034](https://github.com/apache/pulsar/pull/15034)
+- [improve][sec] Add load multiple certificates support [14798](https://github.com/apache/pulsar/pull/14798)
+- [fix][sec] Fix handling single role and non-jwt-token in MultiRolesTokenAuthorizationProvider [14857](https://github.com/apache/pulsar/pull/14857)
+- [fix][sec] Use tlsCertRefreshCheckDurationSec instead of 0 for refresh value [15075](https://github.com/apache/pulsar/pull/15075)
+- [improve][sec] Skip unnecessary DNS resolution when creating AuthenticationDataHttp instance [15221](https://github.com/apache/pulsar/pull/15221)
+- [improve][sec] Improve skipping of DNS resolution when creating AuthenticationDataHttp instance [15228](https://github.com/apache/pulsar/pull/15228)
+- [fix][sec] Fix MultiRolesTokenAuthorizationProvider `authorize` issue [15454](https://github.com/apache/pulsar/pull/15454)
+- [fix][sec] Fix grant all permissions but can't list topic [15501](https://github.com/apache/pulsar/pull/15501)
+- [improve][sec] Switch to rely on Netty for Hostname Verification [15824](https://github.com/apache/pulsar/pull/15824)
+- [fix][sec] Fix create client with TLS config [16014](https://github.com/apache/pulsar/pull/16014)
+- [fix][sec] Avoid AuthenticationDataSource mutation for subscription name [16065](https://github.com/apache/pulsar/pull/16065)
+- [improve][sec] Improve get the basic authentication config [16526](https://github.com/apache/pulsar/pull/16526)
+- [fix][sec] Fix multiple roles authorization [16645](https://github.com/apache/pulsar/pull/16645)
+- [fix][sec] Add timeout of sync methods and avoid call sync method for AuthoriationService (#15694) [16831](https://github.com/apache/pulsar/pull/16831)
+- [improve][sec] Support for get token from HTTP params [16871](https://github.com/apache/pulsar/pull/16871)
+- [improve][sec] Adapt basic authentication configuration with prefix [16935](https://github.com/apache/pulsar/pull/16935)
+
+#### Admin
+- [improve][admin] Add corresponding get command for namespace [12322](https://github.com/apache/pulsar/pull/12322)
+- [fix][admin] Fix deleting namespace will fail when system topics exist [15040](https://github.com/apache/pulsar/pull/15040)
+- [fix][admin] Fix validateGlobalNamespaceOwnership wrap exception issue [14612](https://github.com/apache/pulsar/pull/14612)
+- [fix][admin] Fix wrong prompt exception when get non-persistent topic list without GET_BUDNLE permission [14638](https://github.com/apache/pulsar/pull/14638)
+- [fix][admin] Fixed 404 error msg not returned correctly using http lookup [14677](https://github.com/apache/pulsar/pull/14677)
+- [improve][admin] Provide an accurate error message when set `autoTopicCreation` [14684](https://github.com/apache/pulsar/pull/14684)
+- [fix][admin] Fix cannot delete namespace with system topic [14730](https://github.com/apache/pulsar/pull/14730)
+- [fix][admin] Fix NPE in PulsarAdminBuilder when the service is not set [14769](https://github.com/apache/pulsar/pull/14769)
+- [fix][admin] Fix missing response type in swagger definitions [16022](https://github.com/apache/pulsar/pull/16022)
+
+#### Function
+- [fix][fn] Fixing get functions for output topic and serde classname [14103](https://github.com/apache/pulsar/pull/14103)
+- [improve][fn] Pass configured metricsPort to k8s runtime [14502](https://github.com/apache/pulsar/pull/14502)
+- [fix][fn] fix some IOExceptions when create functions from package URL [14553](https://github.com/apache/pulsar/pull/14553)
+- [fix][fn] handle NPE when `getLeader` returns null [15058](https://github.com/apache/pulsar/pull/15058)
+- [improve][fn] Add KeyStore support in WebSocket, Function Worker HTTPS Servers [15084](https://github.com/apache/pulsar/pull/15084)
+- [fix][function] Check executor null when close the FileSource [15247](https://github.com/apache/pulsar/pull/15247)
+- [improve][fn] Provide default error handler for function log appender [15728](https://github.com/apache/pulsar/pull/15728)
+- [fix][function] Fix error when user starts with the pulsar functions local runner [16565](https://github.com/apache/pulsar/pull/16565)
+- [fix][function] Fix python instance not process zip file correctly [16697](https://github.com/apache/pulsar/pull/16697)
+
+#### IO
+- [fix][io] throw exceptions when kafka offset backing store failed to start [14491](https://github.com/apache/pulsar/pull/14491)
+- [fix][io] Handle kafka sinks that return immutable maps as configs [14780](https://github.com/apache/pulsar/pull/14780)
+
+#### SQL
+- [fix][sql] Fix PulsarRecordCursor deserialize issue. [14615](https://github.com/apache/pulsar/pull/14615)
+- [fix][sql] Fix `messageQueue` release message issue. [16155](https://github.com/apache/pulsar/pull/16155)
+
+#### Offloader
+- [fix][offloader] Fix incorrect entryId in warn log when reading entry from tiered storage [14685](https://github.com/apache/pulsar/pull/14685)
+- [improve] TieredStorage: add debug information [14907](https://github.com/apache/pulsar/pull/14907)
+- [feat][offloader] Add pure S3 provider for the offloader [15710](https://github.com/apache/pulsar/pull/15710)
+- [fix][offloader] Fix setManagedLedgerOffloadedReadPriority not work. [16436](https://github.com/apache/pulsar/pull/16436)
+
+#### Proxy
+- [improve][proxy] Log warning when opening connection to broker fails [14710](https://github.com/apache/pulsar/pull/14710)
+- [refactor][proxy] Refactor Proxy code and fix connection stalling by switching to auto read mode [14713](https://github.com/apache/pulsar/pull/14713)
+- [improve][proxy] Configure Netty DNS resolver to match JDK DNS caching setting, share DNS resolver instance in Proxy [15219](https://github.com/apache/pulsar/pull/15219)
+- [fix][proxy] Fix proxy connection leak when inbound connection closes while connecting is in progress [15366](https://github.com/apache/pulsar/pull/15366)
+- [fix][proxy] Fix DNS server denial-of-service issue when DNS entry expires [15403](https://github.com/apache/pulsar/pull/15403)
+- [improve][proxy] Remove unnecessary blocking DNS lookup in LookupProxyHandler [15415](https://github.com/apache/pulsar/pull/15415)
+- [fix][proxy] Do not preserve host when forwarding admin requests. [16342](https://github.com/apache/pulsar/pull/16342)
+- [fix][proxy] Fix client service url [16834](https://github.com/apache/pulsar/pull/16834)
Review Comment:
```suggestion
- [fix][proxy] Fix client service URL [16834](https://github.com/apache/pulsar/pull/16834)
```
--
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-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org