[GitHub] [helix] dasahcc opened a new pull request #336: #334 Fix invoke rebalance by "touching" IdealState/ResourceConfig

[GitBox <gi...@apache.org>]

dasahcc opened a new pull request #336: #334 Fix invoke rebalance by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336
 
 
   **Issues**
   (fixes #334   )
   
   **Description**
   Current HelixDataAccesor updateProperty uses ZNRecordUpdater. It's merge logic just simply adding all elements when do a merge for ZNRecord. That could cause lot of duplication of listFields.
   This impact the invokeRebalanceForResourceConfig. The fix will be implementing a customized updater.
   
   In this commit:
   1. Fix invoke rebalance with customized updater.
   2. Add comments for ZNRecord merge.
   3. Add checks in TaskUtil to only trigger Workflow Config "touch" when purge job.
   4. Add a test for RebalanceScheduler.
   
   **Tests**
   Test result is pending.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services