Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2077
Rebased to resolve conflict.
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2077
Merge into master if there are no more comments!
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2077
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2077
@AhyoungRyu Thanks for cool review ð
---
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2077
Tested `505c7db` commit only with your package, and it works well as
expected.
![screen shot 2017-02-28 at 6 29 59
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2077
CI failed, but irrelevant.
- build profile: https://travis-ci.org/1ambda/zeppelin/builds/206002571
- failed log1:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/206002573/log.txt