LGTM. BTW, I found the docs of 0.7.0-SNAPSHOT is broken, is it a known issue ?
e.g.
https://zeppelin.apache.org/install/install.html

@zjffdu Ah it's wired. In 0.7.0-SNAPSHOT, all the link in the dropdown menu are broken. It should be https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/install/install.html not https://zeppelin.apache.org/install/install.html.

@AhyoungRyu LGTM with the dead link in README.md

Also, shouldn't we need to fix the broken image in shell interpreter doc?

@kavinkumarks
The shell interpreter docs in github is just a source not a real docs page. The real one is deployed at https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/shell.html.

The below tag is for the broken image.

<img src="{{BASE_PATH}}/assets/themes/zeppelin/img/docs-img/shell-example.png" />

Since it contains {{BASE_PATH}}, github can't recognize this variable. That's the reason why the image is broken. Zeppelin is using Jekyll to build the documentation site. (See here for more details). Only Jekyll can recognize the variable.
Just FYI, that kind of variables(e.g. BASE_PATH, ASSET_PATH) are defined in docs/_config.yml :)

@AhyoungRyu thanks for the information but we can use the path https://github.com/apache/zeppelin/blob/master/docs/assets/themes/zeppelin/img/docs-img/shell-example.png to access the image as used in other interpreters.There is no need to use BASE_PATH variable.What do you say?

Thanks,
Kavin
MailTo: [email protected]

@kavinkumarks Ah right. Although all docs/*.md are just source for the real documentation website, someone might be confused like you. After checking all docs pages under /docs, I found some link and image path contains {{BASE_PATH}}. So I removed them and changed to the actual path. Could you please check this again? :)

@AhyoungRyu the changes look good!

Thanks,
Kavin
MailTo: [email protected]

@zjffdu Just FYI

I found the docs of 0.7.0-SNAPSHOT is broken, is it a known issue ?

This issue was resolved today. docs/0.7.0-SNAPSHOT site works well like before :)
@kavinkumarks Thanks!

Merging to master if there is no further discussion

Thanks @AhyoungRyu

Tested, Merging it