zhangmeng916 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-601939430
> 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.
This one is tested in TestZkCallbackHandlerLeak.
> 2. Admin change to create the new paths.
This one is tested in TestZkHelixAdmin.
> 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.
Tested in TestComputeAndCleanupCustomizedView.
The provider has unit test checked in, and the end to end integration test
will cover from provider to router, which will be checked in by @mgao0.
> 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.
Added test cases in TestComputeAndCleanupCustomizedView
>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