You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "coderzc (via GitHub)" <gi...@apache.org> on 2023/04/23 08:40:05 UTC
[GitHub] [pulsar-site] coderzc opened a new pull request, #534: Release 2.9.5
coderzc opened a new pull request, #534:
URL: https://github.com/apache/pulsar-site/pull/534
### Documentation
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [x] `doc` <!-- Your PR contains doc changes. Please attach the local preview screenshots (run `./preview.sh` at root path) to your PR description, or else your PR might not get merged. -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->
--
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
[GitHub] [pulsar-site] Anonymitaet commented on a diff in pull request #534: Release 2.9.5
Posted by "Anonymitaet (via GitHub)" <gi...@apache.org>.
Anonymitaet commented on code in PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#discussion_r1174702254
##########
release-notes/versioned/client-java-2.9.5.md:
##########
@@ -0,0 +1,15 @@
+---
+id: client-java-2.9.5
+title: Client Java 2.9.5
+sidebar_label: Client Java 2.9.5
+---
+
+* [fix][client] Fix reader listener can't auto ack with pooled message. (#19354)
+* [fix][client] Prevent DNS reverse lookup when physical address is an IP address (#19028)
+* [fix][client] Set fields earlier for correct ClientCnx initialization (#19327)
+* [improve][client] Change the get lastMessageId to debug level (#18421)
+* [fix][client] For exclusive subscriptions, if two consumers are created repeatedly, the second consumer will block #18633
+* [fix][client] Fix async completion in ConsumerImpl#processPossibleToDLQ (#19392)
+* [fix][client] Broker address resolution wrong if connect through a multi-dns names proxy (#19597)
+* [fix] [client] fix memory leak if enabled pooled messages (#19585)
+* [fix][client] Fix IllegalThreadStateException when using newThread in ExecutorProvider.ExtendedThreadFactory (#18268)`
Review Comment:
```suggestion
* [fix][client] Fix IllegalThreadStateException when using newThread in ExecutorProvider.ExtendedThreadFactory (#18268)
```
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
Review Comment:
```suggestion
* [fix] [ml] Fix the incorrect total size if using ML interceptor (#19404)
```
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
+* [improve][broker] Use shrink map for trackerCache (#19534)
+* [fix] [broker] Incorrect service name selection logic (#19505)
+* [fix] [broker] Make the service name resolver cache of PulsarWebResource expire after access (#19532)
+* [fix][branch-2.9] Support zookeeper read-only config (#19693)
+* [fix][branch-2.9] Fix validatePersistencePolicies that Namespace/Topic persistent policies cannot set to < 0 #19687
Review Comment:
```suggestion
* [fix][branch-2.9] Fix validatePersistencePolicies that Namespace/Topic persistent policies cannot be set to < 0 #19687
```
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
Review Comment:
```suggestion
* [fix][broker]Fix multi invocation for ledger createComplete (#18975)
```
Please keep consistent on the first letter of a sentence and check all occurrences
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
+* [improve][broker] Use shrink map for trackerCache (#19534)
+* [fix] [broker] Incorrect service name selection logic (#19505)
+* [fix] [broker] Make the service name resolver cache of PulsarWebResource expire after access (#19532)
+* [fix][branch-2.9] Support zookeeper read-only config (#19693)
+* [fix][branch-2.9] Fix validatePersistencePolicies that Namespace/Topic persistent policies cannot set to < 0 #19687
+* [fix][branch-2.9] fix txn coordinator recover handle committing and aborting txn race condition. #19699
+* [fix][branch-2.9] Fix race condition while updating partition number (#19691)
+* [improve] upgrade the bookkeeper version to 4.14.7 (#19178)
+* [improve] Upgrade lombok to 1.8.26 (#19426)
+* [fix][cli][branch-2.9] Fix release tool (#19712)
+* [fix] [broker] Topic close failure leaves subscription in a permanent fence state (#19692)
+* [fix][meta] Fix deadlock causes session notification not to work (#19754)
+* [improve][misc] Upgrade Netty to 4.1.87.Final (#19417)
+* [fix][broker] Fix potential exception cause the policy service init fail. (#19746)
Review Comment:
```suggestion
* [fix][broker] Fix potential exception causing the policy service init to fail. (#19746)
```
--
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
[GitHub] [pulsar-site] momo-jun commented on a diff in pull request #534: Release 2.9.5
Posted by "momo-jun (via GitHub)" <gi...@apache.org>.
momo-jun commented on code in PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#discussion_r1174712170
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
+* [improve][broker] Use shrink map for trackerCache (#19534)
+* [fix] [broker] Incorrect service name selection logic (#19505)
+* [fix] [broker] Make the service name resolver cache of PulsarWebResource expire after access (#19532)
+* [fix][branch-2.9] Support zookeeper read-only config (#19693)
+* [fix][branch-2.9] Fix validatePersistencePolicies that Namespace/Topic persistent policies cannot set to < 0 #19687
+* [fix][branch-2.9] fix txn coordinator recover handle committing and aborting txn race condition. #19699
+* [fix][branch-2.9] Fix race condition while updating partition number (#19691)
+* [improve] upgrade the bookkeeper version to 4.14.7 (#19178)
+* [improve] Upgrade lombok to 1.8.26 (#19426)
+* [fix][cli][branch-2.9] Fix release tool (#19712)
+* [fix] [broker] Topic close failure leaves subscription in a permanent fence state (#19692)
+* [fix][meta] Fix deadlock causes session notification not to work (#19754)
+* [improve][misc] Upgrade Netty to 4.1.87.Final (#19417)
+* [fix][broker] Fix potential exception cause the policy service init fail. (#19746)
+* [improve][broker] Authorize originalPrincipal when provided (#19830)
+* [fix] [broker] Counter of pending send messages in Replicator incorrect if schema future not complete #19242
+* [branch-2.9][fix][broker] Fix index generator is not rollback after entries are failed added #19978
+* [fix][broker] Only validate superuser access if authz enabled #19989
+
+
+### Proxy
+* [fix][proxy] Only go to connecting state once (#19331)
+
+### Transaction
+
+* [fix][txn] Transaction buffer recover blocked by readNext #18970
+* [fix][txn] transaction pending ack store future not completely problem (#18943)
+* [fix][txn] Fix PendingAckHandleImpl when `pendingAckStoreProvider.checkInitializedBefore` failed (#18859)
+* [fix][broker] Close transactionBuffer after MessageDeduplication#checkStatus failed (#19157)
+* [fix][txn] Correct the prompt message (#17009)
+* [improve][txn] Handle changeToReadyState failure correctly in TC client (#19308)
+
+### Pulsar IO and Pulsar Functions
+
+* [fix][io] Only bundle kafka schema registry client (#18931)
+* [fix][fn] Typo in method name (#18844)
+* [fix][io] Update Elasticsearch sink idle cnx timeout to 30s (#19377)
+* [fix][fn] Fix k8s merge runtime opts bug (#19481)
+
+### Tiered Storage
+
+* [fix][offload] Fix numerical overflow bug while reading data from tiered storage (#18595)
+* [fix][offload] Fix memory leak while Offloading ledgers (#18500)
+
+### Pulsar SQL
+
+* [fix][sql] Fix message without schema issue. (#18745)
+
+### CLI
+
+* [fix][cli] Fix CLI client produce don't able to use multiple -m send multiple messages (#18238)
+* [fix][cli] Check numMessages after incrementing counter #17826
+
+## Admin
+
+* [improve][admin] Unset namespace policy to improve deleting namespace. (#17033)
+* [fix] [admin] Make response code to 400 instead of 500 when delete topic fails due to enabled geo-replication #19879
Review Comment:
```suggestion
* [fix] [admin] Make response code 400 instead of 500 when deleting topic fails due to enabled geo-replication #19879
```
--
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
[GitHub] [pulsar-site] coderzc commented on a diff in pull request #534: Release 2.9.5
Posted by "coderzc (via GitHub)" <gi...@apache.org>.
coderzc commented on code in PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#discussion_r1175973086
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
Review Comment:
@Anonymitaet I check all occurrences and fixed them, please take a look again.
--
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
[GitHub] [pulsar-site] coderzc merged pull request #534: Release 2.9.5
Posted by "coderzc (via GitHub)" <gi...@apache.org>.
coderzc merged PR #534:
URL: https://github.com/apache/pulsar-site/pull/534
--
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
[GitHub] [pulsar-site] momo-jun commented on a diff in pull request #534: Release 2.9.5
Posted by "momo-jun (via GitHub)" <gi...@apache.org>.
momo-jun commented on code in PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#discussion_r1174707866
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
+* [improve][broker] Use shrink map for trackerCache (#19534)
+* [fix] [broker] Incorrect service name selection logic (#19505)
+* [fix] [broker] Make the service name resolver cache of PulsarWebResource expire after access (#19532)
+* [fix][branch-2.9] Support zookeeper read-only config (#19693)
+* [fix][branch-2.9] Fix validatePersistencePolicies that Namespace/Topic persistent policies cannot set to < 0 #19687
+* [fix][branch-2.9] fix txn coordinator recover handle committing and aborting txn race condition. #19699
+* [fix][branch-2.9] Fix race condition while updating partition number (#19691)
+* [improve] upgrade the bookkeeper version to 4.14.7 (#19178)
+* [improve] Upgrade lombok to 1.8.26 (#19426)
+* [fix][cli][branch-2.9] Fix release tool (#19712)
+* [fix] [broker] Topic close failure leaves subscription in a permanent fence state (#19692)
+* [fix][meta] Fix deadlock causes session notification not to work (#19754)
+* [improve][misc] Upgrade Netty to 4.1.87.Final (#19417)
+* [fix][broker] Fix potential exception cause the policy service init fail. (#19746)
+* [improve][broker] Authorize originalPrincipal when provided (#19830)
+* [fix] [broker] Counter of pending send messages in Replicator incorrect if schema future not complete #19242
+* [branch-2.9][fix][broker] Fix index generator is not rollback after entries are failed added #19978
+* [fix][broker] Only validate superuser access if authz enabled #19989
+
+
+### Proxy
+* [fix][proxy] Only go to connecting state once (#19331)
+
+### Transaction
+
+* [fix][txn] Transaction buffer recover blocked by readNext #18970
+* [fix][txn] transaction pending ack store future not completely problem (#18943)
+* [fix][txn] Fix PendingAckHandleImpl when `pendingAckStoreProvider.checkInitializedBefore` failed (#18859)
+* [fix][broker] Close transactionBuffer after MessageDeduplication#checkStatus failed (#19157)
+* [fix][txn] Correct the prompt message (#17009)
+* [improve][txn] Handle changeToReadyState failure correctly in TC client (#19308)
+
+### Pulsar IO and Pulsar Functions
+
+* [fix][io] Only bundle kafka schema registry client (#18931)
+* [fix][fn] Typo in method name (#18844)
+* [fix][io] Update Elasticsearch sink idle cnx timeout to 30s (#19377)
+* [fix][fn] Fix k8s merge runtime opts bug (#19481)
+
+### Tiered Storage
+
+* [fix][offload] Fix numerical overflow bug while reading data from tiered storage (#18595)
+* [fix][offload] Fix memory leak while Offloading ledgers (#18500)
+
+### Pulsar SQL
+
+* [fix][sql] Fix message without schema issue. (#18745)
+
+### CLI
+
+* [fix][cli] Fix CLI client produce don't able to use multiple -m send multiple messages (#18238)
+* [fix][cli] Check numMessages after incrementing counter #17826
+
+## Admin
+
+* [improve][admin] Unset namespace policy to improve deleting namespace. (#17033)
+* [fix] [admin] Make response code to 400 instead of 500 when delete topic fails due to enabled geo-replication #19879
+
+### Security
+
+* [fix][sec] Upgrade scala-library to get rid of CVE-2022-36944 (#18021)
+* [fix][sec] Upgrade jettison to 1.5.3 (#19038)
+* [fix][sec] Upgrade woodstox to 5.4.0 (#19041)
+* [feat][broker] OneStageAuth State: move authn out of constructor #19295
+* [improve][broker] Require authRole is proxyRole to set originalPrincipal (#19455)
+* [fix][broker] Make authentication refresh threadsafe (#19506)
+* [fix][broker] Allow proxy to pass same role for authRole and originalRole (#19557)
+* [improve][sec] Suppress false positive OWASP reports (#19105)
+* [improve] Upgrade wildfly-eytron (used by debezium) to fix CVE-2022-3143 (#19333)
Review Comment:
```suggestion
* [improve] Upgrade wildfly-elytron (used by debezium) to fix CVE-2022-3143 (#19333)
```
--
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
[GitHub] [pulsar-site] coderzc commented on pull request #534: Release 2.9.5
Posted by "coderzc (via GitHub)" <gi...@apache.org>.
coderzc commented on PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#issuecomment-1524500034
/cc @Anonymitaet
--
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
[GitHub] [pulsar-site] Anonymitaet commented on a diff in pull request #534: Release 2.9.5
Posted by "Anonymitaet (via GitHub)" <gi...@apache.org>.
Anonymitaet commented on code in PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#discussion_r1175301328
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
Review Comment:
This comment still needs to be resolved. Please keep consistent on the first letter of a sentence and check all occurrences, then ask for a review. Thanks.
--
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
[GitHub] [pulsar-site] tisonkun commented on a diff in pull request #534: Release 2.9.5
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on code in PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#discussion_r1174541863
##########
data/release-pulsar.js:
##########
@@ -1,4 +1,13 @@
module.exports = [
+ {
Review Comment:
Although IIRC it's not functionally different, may you do a sort by semver?
--
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
[GitHub] [pulsar-site] coderzc commented on a diff in pull request #534: Release 2.9.5
Posted by "coderzc (via GitHub)" <gi...@apache.org>.
coderzc commented on code in PR #534:
URL: https://github.com/apache/pulsar-site/pull/534#discussion_r1175971794
##########
release-notes/versioned/pulsar-2.9.5.md:
##########
@@ -0,0 +1,123 @@
+---
+id: pulsar-2.9.5
+title: Apache Pulsar2.9.5
+sidebar_label: Apache Pulsar 2.9.5
+---
+
+#### 2023-4-23
+
+### Broker
+
+* [improve][broker] Omit making a copy of CommandAck when there are no broker interceptors [#18997](https://github.com/apache/pulsar/pull/18997)
+* [fix][broker] Copy proto command fields into final variables in ServerCnx [#18987](https://github.com/apache/pulsar/pull/18987)
+* [improve][broker] Add logs for why namespace bundle been split (#19003)
+* [fix][broker] Fix deadlock in PendingAckHandleImpl (#18989)
+* [fix][broker]Update interceptor handler exception (#18940)
+* [improve][broker] Make Consumer#equals more effective (#18662)
+* [fix][broker]add test case for deleting namespace fail when has partitioned system topic (#17338)
+* [fix][broker] Fix open cursor with null-initialPosition result with earliest position (#18416)
+* [improve][broker] Add ref count for sticky hash to optimize the performance of Key_Shared subscription (#19167)
+* [fix][broker] Support deleting partitioned topics with the keyword -partition- (#19230)
+* [improve][broker] Follow up #19230 to tighten the validation scope (#19234)
+* [branch-2.9] Fix compile error caused by cherry-pick #19167. (#19241)
+* [improve][websocket][branch-2.9] Add ping support (#19245)
+* [fix][ml] Topics stats shows msgBacklog but there reality no backlog (#19275)
+* [fix][websocket][branch-2.9] Fix webSocketPingDurationSeconds config (#19293)
+* [fix] [ml] Fix the incorrect total size if use ML intercepto (#19404)
+* [fix][broker] Expect msgs after server initiated CloseProducer (#19446)
+* [cleanup][broker] Validate originalPrincipal earlier in ServerCnx (#19270)
+* [fix][Branch-2.9] Fix bookie JVM param BOOKIE_MEM, BOOKIE_GC miss problem. #19441
+* [improve][broker] ServerCnx: go to Failed state when auth fails (#19312)
+* [fix][broker][branch-2.9] Fix geo-replication admin (#19614)
+* [fix] [ml] messagesConsumedCounter of NonDurableCursor was initialized incorrectly (#19355)
+* [fix][ml] Fix potential NPE cause future never complete. (#19415)
+* [fix] [ml] topic load fail by ledger lost (#19444)
+* [fix] [broker] getLastMessageId returns a wrong batch index of last message if enabled read compacted (#18877)
+* [fix][broker] Remove timestamp from broker metrics (#17419)
+* [fix][broker]fix multi invocation for ledger createComplete (#18975)
Review Comment:
Ok
--
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