Github user anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1595
@Leemoonsoo thank you for the feedback, please fix the fix under PR #1604
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1595
Looks like zeppelin-web dev mode (`grunt serve`) has some problem after the
patch.
@anthonycorbacho Could you take a look?
---
If your project is set up for it, you can reply to this email
Github user anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1595
Hell yeah!!!
I allow you to 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 ha
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1595
Finally CI is green!!! I am merging 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
e
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1595
Tested and it works well. 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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1595
Test fails, can you fix 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 feature
enabl
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1595
tested and changes 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 an