Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1879
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 if the
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@cloverhearts update looks good to me.
---
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 cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1879
fixed
---
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 t
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@minahlee Okay, I will take look.
---
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
en
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@cloverhearts Can you also look into
https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js#L764
this line?
I see another error re
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
Merge if there is no more discussion
---
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
ena
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
Tested and it works well :+1:
---
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
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@cloverhearts Thank you :+1:
Let me test it out and I'll comment this.
---
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
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@soralee
Sure :)
rebase is 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 th
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
@cloverhearts I'd like to test this. could you resolve some conflict file?
---
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 p
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1879
Tested, 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 i
11 matches
Mail list logo