[GitHub] zeppelin issue #1214: ZEPPELIN-1224: Fix typo in method name

2016-07-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1214 LGTM. CI failure irrelevant. Merge it into master and branch-0.6 if there're no more discussions. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1214: ZEPPELIN-1224: Fix typo in method name

2016-07-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1214 @karup1990 Thanks! Looks good 👍 --- 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

[GitHub] zeppelin issue #1214: ZEPPELIN-1224: Fix typo in method name

2016-07-22 Thread karup1990
Github user karup1990 commented on the issue: https://github.com/apache/zeppelin/pull/1214 I was thinking to keep changes minimal. Should I change all variable names ? @AhyoungRyu --- If your project is set up for it, you can reply to this email and have your reply appear on