lifepuzzlefun commented on PR #20456: URL: https://github.com/apache/pulsar/pull/20456#issuecomment-1572304621
> > I have the same question as jizhai if the default method is return CompletableFuture with Exception. maybe we need cover the exception in the outer admin api handler to avoid earlier version of user-define AuthorizationProvider fail. > > I may not get what you mean. Earlier version won't fail Sorry for my misunderstanding comments. I mean that if previous version user-define AuthorizationProvider didn't override the default method. when user call the admin api and execute the default method. will it be failed ? like `org.apache.pulsar.broker.authorization.MockAuthorizationProvider`. -- 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]
