Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1559
I'll 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 feature
enabled and wishes so,
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1559
Tested.
Looks good to me!
Very good and clear code!
+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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1559
@jongyoul 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
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1559
Tested on local LGTM.
+1 for adding more test cases!
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1559
\cc @jongyoul @cloverhearts @kjhong for review
---
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