Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2594
merging if no more comment
---
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
Ready to merge! ð
---
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
Oops! I was looking at wrong place ð¢
Thanks @felixcheung !
I restarted failed jobs.
After that, should I re-open this PR?
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2594
see some test failure here though
https://travis-ci.org/tae-jun/zeppelin/builds/277760188
---
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
Hi! How can I make Jenkins green?
All tests passed on my Travis:
https://travis-ci.org/tae-jun/zeppelin/builds/277762975
---
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
@zjffdu Thanks for the review! ð
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2594
Thanks @tae-jun LGTM
---
