Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1773
Looks good! Thanks @zjffdu
Merge into master if there are no more comments on this!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1773
@zjffdu Do you have any chances to look into the last comment by
@felixcheung ?
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1773
@zjffdu All CI build jobs are red now. Could you check again?
---
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 do
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1773
Looks good :)
---
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, o
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1773
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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1773
Push another commit to fix the double checking lock for singleton
---
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 doe
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1773
@AhyoungRyu Please help 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 feature
enabled an