[GitHub] zeppelin issue #1211: [ZEPPELIN-910] New interpreter registration mechanism ...

2016-07-21 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1211 @rerngvit thank you for contributing! Please check if I have correctly assigned JIRA id for this issue to keep attribution. --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin pull request #1211: [ZEPPELIN-910] New interpreter registration mec...

2016-07-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1211 --- 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 feature is

[GitHub] zeppelin issue #1197: [ZEPPELIN-1196] Fix for bug ZEPPELIN-1196

2016-07-21 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/1197 @jongyoul This changes are only for branch-0.6 as master does not have this files (even this code) anymore due to helium changes.Also i think issue might not occur for zeppelin-0.7.So this

Revert ZEPPELIN-1196 from branch-0.6

2016-07-21 Thread Jongyoul Lee
I've reverted that issue because that occurs merge conflict while merging to master. Sorry for confusion. -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

[GitHub] zeppelin issue #1207: [DOC][ZEPPELIN-1209] Remove a useless sentence about d...

2016-07-21 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1207 Thanks @AhyoungRyu. Merging if there's no more discussion --- 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] zeppelin pull request #1214: ZEPPELIN-1224: Fix typo in method name

2016-07-21 Thread karup1990
GitHub user karup1990 opened a pull request: https://github.com/apache/zeppelin/pull/1214 ZEPPELIN-1224: Fix typo in method name ### What is this PR for? Get the correct method to invoke, for autocompletion ### What type of PR is it? Bug Fix ### What is

[jira] [Created] (ZEPPELIN-1224) NoSuchMethodError when trying to invoke autocompletion for Spark

2016-07-21 Thread Karuppayya (JIRA)
Karuppayya created ZEPPELIN-1224: Summary: NoSuchMethodError when trying to invoke autocompletion for Spark Key: ZEPPELIN-1224 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1224 Project:

[GitHub] zeppelin pull request #:

2016-07-21 Thread karup1990
Github user karup1990 commented on the pull request: https://github.com/apache/zeppelin/commit/bd714c2b96d28b9b6e1b2c71431ace99e5e963ec#commitcomment-18349512 In spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java: In

[GitHub] zeppelin pull request #:

2016-07-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/zeppelin/commit/bd714c2b96d28b9b6e1b2c71431ace99e5e963ec#commitcomment-18349468 In spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java: In

[GitHub] zeppelin pull request #:

2016-07-21 Thread karup1990
Github user karup1990 commented on the pull request: https://github.com/apache/zeppelin/commit/bd714c2b96d28b9b6e1b2c71431ace99e5e963ec#commitcomment-18349421 In spark/src/main/java/org/apache/zeppelin/spark/DepInterpreter.java: In

[GitHub] zeppelin pull request #:

2016-07-21 Thread karup1990
Github user karup1990 commented on the pull request: https://github.com/apache/zeppelin/commit/bd714c2b96d28b9b6e1b2c71431ace99e5e963ec#commitcomment-18349417 In spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java: In

[GitHub] zeppelin issue #1163: [ZEPPELIN-1149] %sh interpreter kerberos support

2016-07-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1163 Sorry, I mistake it as hdfs interpreter. For shell interpreter, do we must have kerberos support ? I think here kinit is just for the hadoop command, so actually it is for running hdfs command in

[GitHub] zeppelin pull request #:

2016-07-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/zeppelin/commit/bd714c2b96d28b9b6e1b2c71431ace99e5e963ec#commitcomment-18349378 In spark/src/main/java/org/apache/zeppelin/spark/DepInterpreter.java: In

[GitHub] zeppelin issue #1163: [ZEPPELIN-1149] %sh interpreter kerberos support

2016-07-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1163 agree with @felixcheung, use the zeppelin service keytab/principal might bring some other potential security problems. I think user A would expect to run shell script as himself, but actually it

[GitHub] zeppelin issue #1117: [ZEPPELIN-1086] Auto Completion for Interpreter.

2016-07-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1117 Ping --- 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

[GitHub] zeppelin issue #1212: ZEPPELIN-1222. ClassNotFoundException of SparkJLineCom...

2016-07-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1212 I'm merging it into master and branch-0.6 --- 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

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-21 Thread babupe
Github user babupe commented on the issue: https://github.com/apache/zeppelin/pull/1170 Awesome! Thank you! I have added that to the documentation. --- 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

Re: Interpreter password security

2016-07-21 Thread Vinay Shukla
Turns out there is already https://issues.apache.org/jira/browse/ZEPPELIN-530 to track this issue. On Thu, Jul 21, 2016 at 8:19 AM, Vinay Shukla wrote: > Hi Eros, > > Not sure if there is an active plan for this. But we should look at Hadoop > Credential provider

[GitHub] zeppelin issue #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

2016-07-21 Thread mwpenny
Github user mwpenny commented on the issue: https://github.com/apache/zeppelin/pull/1210 @corneadoug, The Esri angular module is used to download and lazy load the JS API and require its various classes. These classes are then used to display the dynamically downloaded map tiles and

[GitHub] zeppelin issue #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

2016-07-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1155 Have uploaded the GIF again, I think it would have failed last time. --- 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

[GitHub] zeppelin issue #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

2016-07-21 Thread karup1990
Github user karup1990 commented on the issue: https://github.com/apache/zeppelin/pull/1155 Thanks for addressing the feedbacks @prabhjyotsingh .. I cannot see the GIF In the description of the PR . --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1211: [ZEPPELIN-910] New interpreter registration mechanism ...

2016-07-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1211 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] zeppelin issue #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

2016-07-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1155 @karup1990 have tried to implement the behaviour that you were asking about, can you take a look. Have added a GIF in PR description. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1211: [ZEPPELIN-910] New interpreter registration mechanism ...

2016-07-21 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1211 Looks good to me. CI failure is due to flaky R test and is treked under [ZEPPELIN-862](https://issues.apache.org/jira/browse/ZEPPELIN-862) Merging if there is no further discussion

[GitHub] zeppelin issue #1212: ZEPPELIN-1222. ClassNotFoundException of SparkJLineCom...

2016-07-21 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1212 Looks great to me. Let's merge if there is no further discussion --- 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] zeppelin pull request #1163: [ZEPPELIN-1149] %sh interpreter kerberos suppor...

2016-07-21 Thread rconline
Github user rconline closed the pull request at: https://github.com/apache/zeppelin/pull/1163 --- 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 feature

[GitHub] zeppelin pull request #1163: [ZEPPELIN-1149] %sh interpreter kerberos suppor...

2016-07-21 Thread rconline
Github user rconline closed the pull request at: https://github.com/apache/zeppelin/pull/1163 --- 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 feature

[GitHub] zeppelin pull request #1163: [ZEPPELIN-1149] %sh interpreter kerberos suppor...

2016-07-21 Thread rconline
GitHub user rconline reopened a pull request: https://github.com/apache/zeppelin/pull/1163 [ZEPPELIN-1149] %sh interpreter kerberos support ### What is this PR for? Zeppelin Shell interpreter should work in a Kerberos environment ### What type of PR is it?

[GitHub] zeppelin issue #1205: [ZEPPELIN-1212] User impersonation support in JDBC

2016-07-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1205 Merging this if no more discussion. --- 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

[GitHub] zeppelin issue #1160: [Zeppelin - 1152] Listing note revision history

2016-07-21 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1160 Looks good to me, merging if there is no further discussion --- 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] zeppelin pull request #1202: [ZEPPELIN-1208] fixed invaild position user con...

2016-07-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1202 --- 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 feature is

[GitHub] zeppelin pull request #1204: [Zeppelin-1089] Replace JsHint with Eslint

2016-07-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1204 --- 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 feature is

[GitHub] zeppelin issue #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

2016-07-21 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1210 @mwpenny Thanks for the PR There was already some attempts to include Leaflet before, however I think it is not yet to be finished. (No pivot to choose columns at least) I think the

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-21 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1170 Great, thank you for understanding, we are almost there. I have marked above ones that I was able to verify, could you please double-check it? On the

[GitHub] zeppelin issue #1173: [ZEPPELIN-1164] ZeppelinHub Realm

2016-07-21 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1173 @khalidhuseynov i think it make sense to do it in this PR. --- 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

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-21 Thread babupe
Github user babupe commented on the issue: https://github.com/apache/zeppelin/pull/1170 No problem. Pushed --- 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,