Hi guys,

I have added all of the review feedback to the API. Here are the changes

* Removed the rebalancer configuration into configuration.
* Added Provisioner and TargetProvider configurations.
* All constructors using ZNRecord is deprecated
* Added clusterid to memberid and resourceid
* Removed apis to create generic members from admin
* Changed all other non-participant APIs to be final and throws
UnsupportedException as discussed.

I have been unable to move the ZNRecord and ZNRecordDelta and other
classes from model primarily because that means HelixConfig needs to
be moved which is currently the base class from most configurations.
Ideally we have the ZNRecord abstracted through SPI and not in API.

As usual feedback is welcome.

Thanks,

Sandeep

Reply via email to