[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-15 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1167717510 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -527,17 +528,22 @@ protected long

[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-15 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1167590266 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -527,17 +528,22 @@ protected long

[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-15 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1167590083 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -527,17 +528,22 @@ protected long

[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-15 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1167579969 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -527,17 +528,22 @@ protected long

[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-14 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1166912389 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -567,26 +574,19 @@ public synchronized

[GitHub] [pulsar] lifepuzzlefun commented on a diff in pull request #20086: [fix][broker] Ensure previous delayed index be removed from snapshotSegmentLastIndexTable & Make load operate asynchronous

2023-04-14 Thread via GitHub
lifepuzzlefun commented on code in PR #20086: URL: https://github.com/apache/pulsar/pull/20086#discussion_r1166912389 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/bucket/BucketDelayedDeliveryTracker.java: ## @@ -567,26 +574,19 @@ public synchronized