Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2512
Thank you everyone, for the review. Will merge this if 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.
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2512
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2512
@1ambda thanks for the review, have handled your review comments.
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2512
I missed the notification for a while. Sorry for the late response. Let me
check and reply ASAP.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2512
@felixcheung, @AhyoungRyu any views on this one?
---
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
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2512
Ping
---
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
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2512
CI Green, Ready for review.
@Leemoonsoo @1ambda @zjffdu Please help review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub