michaeljmarshall opened a new pull request, #19180: URL: https://github.com/apache/pulsar/pull/19180
### Motivation With the acceptance of PIP 149, we've made a clear move to remove blocking calls in the Pulsar code base. The `AuthorizationService` has some unused methods that synchronous wrappers for async methods. Since these methods are unused, I propose we deprecate them in 2.12 and remove them in 2.13. ### Modifications * Add `@Deprecated` annotation and an updated javadoc. ### Verifying this change I relied on IDE's usage finder to indicate that these methods are unused. ### Does this pull request potentially affect one of the following parts: - [x] The public API This will eventually affect the public API for the `AuthorizationService`, which only impacts users that are running custom code inside the Pulsar Broker. I will send an email to the mailing list to make sure this change does not go unnoticed. ### Documentation - [x] `doc-not-needed` We do not document the `AuthorizationService` API anywhere but the source code, so I see no need to document this change. ### Matching PR in forked repository Skipping the forked PR because it is a trivial update. -- 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]
