Github user kanakb commented on the pull request:
https://github.com/apache/helix/pull/1#issuecomment-87272749
This generally looks good, but the following are necessary before it can be
merged:
- All new files need an Apache license header.
- This code should be rebased off of the latest Helix master so that the
gray icon above becomes green.
- The 90+ commits should be squashed into a single commit that describes
this work.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
