Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Thank you @felixcheung. Finally merged !
---
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 featur
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/746
Done!
---
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 user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/746
I've glanced at this PR. I think this PR never break current behaviour and
codes looks good. If it doesn't have more issues, it's enough to merge this PR.
---
If your project is set up for it, yo
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Ping @Leemoonsoo , can we merge this long-standing PR ? CI failures are
irrelevant as mentioned above
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Ok, there are 3 unrelated errors:
1) https://s3.amazonaws.com/archive.travis-ci.org/jobs/155941982/log.txt
```
[32mAngularElem[0m
[31m- should provide onclick method **
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Rebased from master
---
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
