Skip to content

Zeppelin interpreter documentation and interpreter REST API documentation#256

Closed
eranwitkon wants to merge 5 commits into
apache:gh-pagesfrom
eranwitkon:gh-pages
Closed

Zeppelin interpreter documentation and interpreter REST API documentation#256
eranwitkon wants to merge 5 commits into
apache:gh-pagesfrom
eranwitkon:gh-pages

Conversation

@eranwitkon
Copy link
Copy Markdown

Add documentation about the role of Interpreters, interpreters group and interpreters settings in Zeppelin and docs about interpreter REST API.

Please review verify that
https://github.com/eranwitkon/incubator-zeppelin/blob/gh-pages/docs/manual/interpreters.md
is correct...

eranwitkon added 2 commits August 25, 2015 15:19
Comment thread index.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor: "support" -> "supports"

@Leemoonsoo
Copy link
Copy Markdown
Member

Looks great. +1

Comment thread docs/manual/interpreters.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to keep if for now, or you need some kind of input to resolve it?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't sure that the description of what is a registered interpreter,
setting and group mean.
If it is correct I will remove the TODO and the url parameter as commented.
BTW, Internally, I think the doing a create setting which is actualky
calling add group is a bed naming and hard to follow.
Has no effect on documentation for now...

בתאריך יום ו׳, 28 באוג׳ 2015, 09:05 מאת Alexander <notifications@github.com

:

In docs/manual/interpreters.md
#256 (comment)
:

+
+
+### What is zeppelin interpreter setting?
+
+Zeppelin interpreter setting is the configuration of a given interpreter on zeppelin server. For example, the properties requried for hive JDBC interpreter to connect to the Hive server.
+
+
+### What is zeppelin interpreter group?
+
+Every Interpreter belongs to an InterpreterGroup. InterpreterGroup is a unit of start/stop interpreter.
+By default, every interpreter belong to a single group but the group might contain more interpreters. For example, spark interpreter group include spark support, pySpark,
+SparkSQL and the dependency loader.
+
+Technically, Zeppelin interpreters from the same group are running in the same JVM.
+
+TODO: Validate this with the community ...

Do we want to keep if for now, or you need some kind of input to resolve
it?


Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-zeppelin/pull/256/files#r38173203.

@bzz
Copy link
Copy Markdown
Member

bzz commented Aug 28, 2015

Thank you very much for the contribution! Docs are very valuable asserts indeed.
For the REST API doc - I wish we could use swagger... Does anybody remember, why did we got rid of it?

@Leemoonsoo
Copy link
Copy Markdown
Member

@bzz recent discussion related to swagger #226

@eranwitkon
Copy link
Copy Markdown
Author

Docs fixed according to comments.
Ready for merge.

@eranwitkon
Copy link
Copy Markdown
Author

Docs fixed abd ready for merge.

בתאריך יום ו׳, 28 באוג׳ 2015, 09:21 מאת Lee moon soo <
notifications@github.com>:

@bzz https://github.com/bzz recent discussion related to swagger #226
#226


Reply to this email directly or view it on GitHub
#256 (comment)
.

@Leemoonsoo
Copy link
Copy Markdown
Member

LGTM

@bzz
Copy link
Copy Markdown
Member

bzz commented Aug 31, 2015

@Leemoonsoo thank you for pointing out the history!

Looks great to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants