virajjasani commented on PR #5266: URL: https://github.com/apache/hbase/pull/5266#issuecomment-1582936997
> especially that if you pass the list of server names in and get a List of CompletableFuture back what if we have one more argument `boolean runOnAllServers`? well, that's the usecase we have in reality anyways. if runOnAllServers is true, then List<ServerName> gets ignored and we use `FutureUtils#addListener` to supply list of live regionservers first, and then in the callback we execute the regionserver coproc function (similar to what you suggested). does this sound good? our usecase is anyways to be able to run on all servers but just to make it look more flexible for any other client usecases, we thought of using list of servers. -- 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]
