[GitHub] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2144 Merge into master if no more discussions. --- 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] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2144 Cool! Tested again and it works well as expected. 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

[GitHub] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2144 @AhyoungRyu I resolved your all comments. Thanks! --- 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] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2144 Cool improvement! LGTM and merge master if no further discussions. --- 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