michaeljmarshall opened a new pull request, #19182: URL: https://github.com/apache/pulsar/pull/19182
### Motivation We have many methods in the `AuthorizationProvider` interface that have been annotated with `@Deprecated` for since the 2.7.0 release. The PR that deprecated these methods was merged August 2020: #7788. These methods are not used by the Pulsar code base, but may technically be used by third party extensions, which is why it is important to be careful when removing them. That being said, enough time has passed. Removing these methods should make the `AuthorizationProvider` easier to read and easier to use. ### Modifications * Remove `allowTenantOperationAsync`, `allowTenantOperation`, `allowNamespaceOperationAsync`, `allowNamespaceOperation`, `allowNamespacePolicyOperationAsync`, `allowNamespacePolicyOperation`, `allowTopicOperationAsync`, `allowTopicOperation` (all deprecated by #7788) ### Verifying this change I reviewed the git history and made sure the project compiles. ### Does this pull request potentially affect one of the following parts: - [x] The public API This is technically a breaking change. I will send an email to the mailing list. ### Documentation - [x] `doc-not-needed` We allow the `AuthorizationProvider` interface itself to be the documentation, so I do not see any need to update docs. ### Matching PR in forked repository Since I already tested the compilation of the project, I see no reason to build in my own fork. -- 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]
