Marcosrico opened a new pull request, #2290: URL: https://github.com/apache/helix/pull/2290
To test implementation of FederatedZkClient Multi, tests must be written. Mainly rearranged current test base for cleaner code and added specific test for FederatedZkClient multi support. Issues N.A. Tests Solely rearranged tests under /multizk and combined similar code to clean up the tests. Wrote separate test to test future implementation of FederatedZkClient ZkMulti in testMultiInMultiZk. Currently FederatedZkClient ZkMulti is not supported. Current test failure is known issue: https://github.com/apache/helix/issues/2286 Code Quality My diff has been formatted using helix-style.xml (helix-style-intellij.xml if IntelliJ IDE is used) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
