Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
Thanks! Merge into master if there are no more 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 project does
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
Thanks @AhyoungRyu , I rename the note, please check the updated screenshot
in PR description.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
Sorry for my late response. It looks nice! Except one minor suggestion,
LGTM ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
ping @AhyoungRyu , please check the updated screenshot
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
@zjffdu Nice!
In the past `Spark Tutorial` is the only tutorial note in Zeppelin included
by default. But after this PR, 5 tutorial notes will be shown to Zeppelin new
commers. Since
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1830
@felixcheung @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
