[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-04-15 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-210745240 @Leemoonsoo Basically, the idea of Helium is very good and promising, and I also think my implementation is a little bit over-sized patch for solving proble

[GitHub] incubator-zeppelin pull request: Use bootstrap dialog in place of ...

2016-04-15 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/837#issuecomment-210742857 you need to fix it https://github.com/apache/incubator-zeppelin/pull/837/files#diff-a7fb72a3220c2a1f756a04eaa5cf1dc6R245 as well --- If your project

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-15 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/839#issuecomment-210742560 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 thi

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-95] Make environment va...

2016-04-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/789 --- 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 fea

[GitHub] incubator-zeppelin pull request: Add tooltip for notes reload butt...

2016-04-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/813 --- 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 fea

[GitHub] incubator-zeppelin pull request: Add tooltip for notes reload butt...

2016-04-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/813#issuecomment-210741303 LGTM merging it into master. --- 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 proje

Re: Running zeppelin as a webapp within another server

2016-04-15 Thread moon soo Lee
Please take a look https://developer.mozilla.org/en-US/docs/Web/API/Location. it might help. Thanks, moon On Fri, Apr 8, 2016 at 11:08 AM Minudika Malshan wrote: > + what does "location" in this code secment refer to? > > var port = Number(*location*.port); > > return wsProtocol + '//' + *loc

Re: [ZEPPELIN-519] Get interpreter id via REST API

2016-04-15 Thread moon soo Lee
Hi, Preciously, InterpreterSetting has 'id' field, but Interpreter doesn't. Identification of Interpreter can be with either 'className' or combination of 'group' and 'name' from interpreter registration information[1]. [1] https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-interp

[GitHub] incubator-zeppelin pull request: Pluggable resource pools

2016-04-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/816#issuecomment-210729996 Somehow, the PR includes commits from other contributions. @fireboy1919 could you manage it to not includes them? --- If your project is set up for it, y

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-705]Search should aware...

2016-04-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/833#issuecomment-210728757 @ravicodder Thanks for the patch. Would it be difficult to make corresponding rest api test? --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-04-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-210727637 Thanks @jongyoul for taking care of the issue. I think this subject is related to [ZEPPELIN-598](https://issues.apache.org/jira/browse/ZEPPELIN-59

[GitHub] incubator-zeppelin pull request: [WIP] [ZEPPELIN-732] Helium Appli...

2016-04-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/836#issuecomment-210724384 @jongyoul Thanks for the interest. Helium just runs jar library (which extends `Application`). Which may include some output and the output is dis

[GitHub] incubator-zeppelin pull request: Use bootstrap dialog in place of ...

2016-04-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/837#issuecomment-210722868 Thanks for the improvement. There're a [test case](https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-server/src/test/java/org/apac

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/839#issuecomment-210721338 Nice improvements. 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 do

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-15 Thread cyhsutw
GitHub user cyhsutw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/839 Improve README.md ### What is this PR for? Update README.md - Unify code block styles (fenced code blocks). - Add syntax highlighting in some code blocks. -

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread cyhsutw
Github user cyhsutw commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/838#issuecomment-210702820 @felixcheung Nice idea, I'll close this one and open another instead. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread cyhsutw
Github user cyhsutw closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/838 --- 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 fe

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/838#issuecomment-210690599 would be better to put `-Pmapr[version]` in `? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin pull request: update VM readme and VM install d...

2016-04-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/826 --- 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 fea

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-741] Re-use existing pa...

2016-04-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/832 --- 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 fea

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-788] Update Lens interp...

2016-04-15 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/811#issuecomment-210688863 LGTM merging if no more comment --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-788] Update Lens interp...

2016-04-15 Thread lresende
Github user lresende commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/811#issuecomment-210680735 @bzz @felixcheung Ok, all tests are passing, ready for review --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/838#issuecomment-210677282 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 feat

[jira] [Created] (ZEPPELIN-807) unable to create multiple spark interpretors

2016-04-15 Thread Bhanuprakash Prathap (JIRA)
Bhanuprakash Prathap created ZEPPELIN-807: - Summary: unable to create multiple spark interpretors Key: ZEPPELIN-807 URL: https://issues.apache.org/jira/browse/ZEPPELIN-807 Project: Zeppelin

[GitHub] incubator-zeppelin pull request: Pluggable resource pools

2016-04-15 Thread fireboy1919
Github user fireboy1919 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/816#issuecomment-210565802 The implementation that I suggested has been implemented. I did it by actually including all of the things that are in zeppelinconfig that start with ze

[GitHub] incubator-zeppelin pull request: Fix unwanted link in README.md

2016-04-15 Thread cyhsutw
GitHub user cyhsutw opened a pull request: https://github.com/apache/incubator-zeppelin/pull/838 Fix unwanted link in README.md ### What is this PR for? Fix an unwanted link in `README.md` by escaping the link syntax. ### What type of PR is it? Documentation.

[GitHub] incubator-zeppelin pull request: Pluggable resource pools

2016-04-15 Thread fireboy1919
Github user fireboy1919 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/816#issuecomment-210477759 Great ideas! Will add. --- 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 d

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-672] Add feature to exp...

2016-04-15 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-210409274 @ankurmitujjain 👍 --- 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] incubator-zeppelin pull request: [ZEPPELIN-672] Add feature to exp...

2016-04-15 Thread ankurmitujjain
Github user ankurmitujjain commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-210406135 @darionyaphet . If you want to save data as csv on server you can use below code... [spark-csv](https://github.com/databricks/spark-csv)

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-672] Add feature to exp...

2016-04-15 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-210367054 hi @ankurjainyash could support a option about save the CSV/TSV to server ? --- If your project is set up for it, you can reply to this email and have y