[GitHub] [pulsar] equanz commented on pull request #20179: [fix][broker] Fix recentlyJoinedConsumers to address the out-of-order issue

equanz commented on PR #20179: URL: https://github.com/apache/pulsar/pull/20179#issuecomment-1543445903 @eolivelli Thank you for your comments. I'll check it. By the way, do you have any comments about the following concern from @poorbarcode ? > By the way, if there are a

[GitHub] [pulsar] equanz commented on pull request #20179: [fix][broker] Fix recentlyJoinedConsumers to address the out-of-order issue

equanz commented on PR #20179: URL: https://github.com/apache/pulsar/pull/20179#issuecomment-1539314184 @poorbarcode * [Q-1] Yes. * [Q-2] When I created this PR, I introduced Map structure because it was necessary. For more details, please see the description("2. Keep the last

[GitHub] [pulsar] equanz commented on pull request #20179: [fix][broker] Fix recentlyJoinedConsumers to address the out-of-order issue

equanz commented on PR #20179: URL: https://github.com/apache/pulsar/pull/20179#issuecomment-1523037906 @poorbarcode Thank you for your clarification. * [1] Yes. * [2] Partially yes. However, I think the main point of this PR is the change in the timing of getting the position.