virajjasani commented on PR #5295: URL: https://github.com/apache/hbase/pull/5295#issuecomment-1595665636
oh i see, but even same is fine with AsyncAdmin also since it is already IA#Public? earlier what i meant for IA#Private is that FutureUtils methods are all private so ideally downstreamers should avoid using `FutureUtils#addListener` method to combine multiple listeners (as much as possible). but otherwise for this particular method, we are good with default implementation on AsyncAdmin as well right? we don't need AdminUtil because AsyncAdmin is anyways IA#Public, correct? -- 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]
