narendly commented on issue #357: [WIP] Add getWorkflows(long timeout) to TaskDriver. URL: https://github.com/apache/helix/pull/357#issuecomment-514106795 And once you've done that, in your unit test, create a mock of getWorkflow () function, and make it take a long time (to simulate ZK disconnect - or just literally disconnect the HelixManager), and make sure the new API behaves as expected. I apologize for making these comments since I am not able to log in to GitHub on my phone. - Hunter On Jul 23, 2019, 10:02, at 10:02, pkuwm <[email protected]> wrote: >Wonder what the unit test requirement is. Can you please suggest? > >-- >You are receiving this because you are subscribed to this thread. >Reply to this email directly or view it on GitHub: >https://github.com/apache/helix/pull/357#issuecomment-514099797
---------------------------------------------------------------- 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
