[GitHub] helix issue #48: helix-ui improvements

2016-08-31 Thread kishoreg
Github user kishoreg commented on the issue: https://github.com/apache/helix/pull/48 LGTM --- 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

[GitHub] helix issue #48: helix-ui improvements

2016-08-25 Thread brandtg
Github user brandtg commented on the issue: https://github.com/apache/helix/pull/48 0.6.x because the beta label was a hard sell 😄 --- 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

[GitHub] helix issue #48: helix-ui improvements

2016-08-25 Thread kishoreg
Github user kishoreg commented on the issue: https://github.com/apache/helix/pull/48 Got it. Will review this. Are you guys using 0.6.x or 0.7.x --- 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

[GitHub] helix issue #48: helix-ui improvements

2016-08-25 Thread brandtg
Github user brandtg commented on the issue: https://github.com/apache/helix/pull/48 Ah here is the JIRA https://issues.apache.org/jira/browse/HELIX-627 --- 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

[GitHub] helix issue #48: helix-ui improvements

2016-08-25 Thread brandtg
Github user brandtg commented on the issue: https://github.com/apache/helix/pull/48 @kishoreg Sorry I should have been more clear - the FULL_AUTO bug was a problem in which the UI wouldn't show the table containing partition / ideal state / external view, as there is no state map

[GitHub] helix issue #48: helix-ui improvements

2016-08-25 Thread kishoreg
Github user kishoreg commented on the issue: https://github.com/apache/helix/pull/48 @lei-xia can you also review this change. Subbu had a test case https://issues.apache.org/jira/browse/HELIX-631. Will be good to see if this change fixes that as well. --- If your project is set up

[GitHub] helix issue #48: helix-ui improvements

2016-08-25 Thread kishoreg
Github user kishoreg commented on the issue: https://github.com/apache/helix/pull/48 this is awesome Greg, can you please create a separate PR for FULL_AUTO bug. I will review the changes tonight. --- If your project is set up for it, you can reply to this email and have your reply