Looking great. Feedback/comments

-- HelixAdministratorClient methods are not symmetric. addXXX, removeXXX,
updateXXX
-- Its hard to know the subclasses of command, how to set the configuration
for a member, partition, resource, cluster. how to register a statemachine
-- we need service for each member type. ParticipantService,
ControllerService etc, how do these get created and join the cluster.
-- What about the event listeners. I guess that will be part of
message/events
-- so the user does not need spi package to write his code ?
-- I think i got the answer for this, but will be nice to know the
dependency tree between the packages and how will it be once we move have
specific implementations for zk, database, hazelcast/infinispan etc
-- I am worried about the commands, that it might lead to an explosion. we
should look at existing helixadmin class and see what would it mean to
support all those methods.


I will try to write a recipe using these api's and that will help me give
better feedback.






On Tue, Apr 29, 2014 at 11:10 PM, Sandeep Nayak <[email protected]> wrote:

> Hey guys,
>
> I finally got around incorporating the review feedback and all changes
> are checked in. Where should I look to next? I think there is more to
> do with the API but wanted to pause and ask what should I focus on
> next.
>
> Here are the open items on the top of the list
>
> * Rebalancer API
> * Message and/or Events
>
> Let me know what should be next and I will tackle it accordingly.
>
> Thanks,
>
> Sandeep
>

Reply via email to