jiajunwang commented on issue #851: Modify Helix generic controller to include new stage for customized view aggregation URL: https://github.com/apache/helix/pull/851#issuecomment-600905403 One more thing, the added test cases are not enough. Please add tests for the following features: 1. We need to add a case in the test listener leakage for the new listener introduced. 2. Admin change to create the new paths. 3. In the integration test, please test adjusting the view config and see if the view changed accordingly. Moreover, we should use the customized view provider to test. 4. Try to cover some error case, for example, instances are not reporting the configured type. Or 2 instances are reporting different sets of types. Please try to cover all the use cases that we mentioned in the design doc.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@helix.apache.org For additional commands, e-mail: reviews-h...@helix.apache.org