pkuwm commented on issue #357: [WIP] Add getWorkflows(long timeout) to TaskDriver. URL: https://github.com/apache/helix/pull/357#issuecomment-517060382 This would really help as you guys know the use case better and how this api is used . I believe we should optimize it and trade off base on the how it is used.. Thanks! We can sync up offline and complete it later. On Wed, Jul 31, 2019 at 4:30 PM Hunter Lee <[email protected]> wrote: > @pkuwm <https://github.com/pkuwm> Good progress, but I have alternative > thoughts on how this should be implemented. This is a simple list call in > the Helix client, and we do not want the users to bear the burden of having > to maintain a thread pool. Since it's not urgent, let us sync offline to > get on the same page on why this is needed and how it could be simplified :) > > — > You are receiving this because you were mentioned. > > > Reply to this email directly, view it on GitHub > <https://github.com/apache/helix/pull/357?email_source=notifications&email_token=ABHSRCKQSDGMZQS5CVCUBT3QCIOBFA5CNFSM4IGBGEJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3I3ISI#issuecomment-517059657>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/ABHSRCNILBXIKNPYNOANQETQCIOBFANCNFSM4IGBGEJA> > . >
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
