Re: [PR] [fix][broker] Fix multiple race conditions in topic unloading and loading [pulsar]

2024-01-10 Thread via GitHub
lhotari commented on code in PR #20540: URL: https://github.com/apache/pulsar/pull/20540#discussion_r1447078219 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java: ## @@ -2264,18 +2315,8 @@ private Optional>> findTopicFutureInCache(Topi

Re: [PR] [fix][broker] Fix multiple race conditions in topic unloading and loading [pulsar]

2024-01-10 Thread via GitHub
lhotari commented on code in PR #20540: URL: https://github.com/apache/pulsar/pull/20540#discussion_r144707 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractTopic.java: ## @@ -1310,6 +1312,23 @@ public HierarchyTopicPolicies getHierarchyTopicPolicies(

Re: [PR] [fix][broker] Fix multiple race conditions in topic unloading and loading [pulsar]

2023-12-27 Thread via GitHub
lhotari commented on PR #20540: URL: https://github.com/apache/pulsar/pull/20540#issuecomment-1870160236 This test app ends up with a topic being fenced: https://github.com/lhotari/pulsar-playground/blob/master/src/main/java/com/github/lhotari/pulsar/playground/TestScenarioUnloading.java ,

Re: [PR] [fix][broker] Fix multiple race conditions in topic unloading and loading [pulsar]

2023-12-06 Thread via GitHub
graysonzeng commented on PR #20540: URL: https://github.com/apache/pulsar/pull/20540#issuecomment-1844281116 Thanks for the fix, I tried using this PR but still encountered this problem on rolling restart bookies and was unable to consume until I restarted all the broker. I will try to trou

Re: [PR] [fix][broker] Fix multiple race conditions in topic unloading and loading [pulsar]

2023-10-22 Thread via GitHub
github-actions[bot] commented on PR #20540: URL: https://github.com/apache/pulsar/pull/20540#issuecomment-1774307990 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an