Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2021
CI was passing here.
merged to master and branch-0.7
---
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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2021
@prabhjyotsingh @felixcheung Thanks for review, the comments are addressed.
---
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2021
I think that's a fair ask - @zjffdu could you update and we will 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
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2021
Ran this couple of time on my local, works a expected.
---
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
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2021
LGTM
Waiting for CI and
merging if no more comment
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2021
@prabhjyotsingh @felixcheung Please help review, CI failure is not
relevant.
```
TypeError: namedtuple() missing 3 required keyword-only arguments:
'verbose', 'rename', and 'module'