Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1239
Tested, looks good to me. I have one question. Seems like the issue exists
both in `master` and `branch-0.6` branch, will this patch be applicable to
branch-0.6 or shall we create another PR
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1239
@Leemoonsoo @minahlee Could you please test it?
---
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 user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1239
Looks great to me.
CI failure:
```
[INFO] Zeppelin: web Application .. FAILURE [02:13
min]
...
[INFO] Running 'grunt build --no-color' in
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1239
Rebased it
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1239
Confirm that bug is gone. @jongyoul Could you rebase before merge?
---
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 user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1239
I'll trigger the CI one more, and merge it
---
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 user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1239
Thanks for the fix. Tested and 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
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1239
@Leemoonsoo I fixed it. Review it, please.
---
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
