BewareMyPower commented on pull request #14320: URL: https://github.com/apache/pulsar/pull/14320#issuecomment-1042801722
> Instead, you can say that a lot of PRs with[ "make async"](https://github.com/apache/pulsar/pulls?q=is%3Apr+make+async) have been pushed and merged recently. Yeah, I noticed these PRs recently as well. But are these PRs blockers for 2.10.0 release? IMO, **they should not be blockers as well.** I've thought they are intended to be included in Pulsar 2.11.0. > I haven't requested to block the release. Sorry, I might missed some context. I just saw this PR in the 2.10.0 release email list. This PR should focus on the fix itself, but the previous discussion might go far for the release issue. > [My valid questions were never answered by the contributors of the "make async" changes](https://github.com/apache/pulsar/issues/14013#issuecomment-1033528348). It's a pity to see the lack of communication. AFAIK, @Technoboy- is also preparing for a PIP to make admin APIs async. I think you should have a discussion about: 1. Whether this PR solve the root problem? 2. Based on this PR, is making admin APIs async meaningful? -- 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]
