shibd commented on PR #1310: URL: https://github.com/apache/pulsar-client-go/pull/1310#issuecomment-2499898134
I have revert #1283 and copy unit test from this PR: https://github.com/apache/pulsar-client-go/pull/1311 This is an alternative solution to the issue. The current PR offers a method similar to an `unbounded` channel. The PR #1311 maintains the initial behavior, using `availablePermits` to ensure the channel is never filled. We can compare which approach is more user-friendly. From my perspective, #1311 is simpler and the code is more readable. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
