[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 `com.google.apis:google-api-services-bigqu

[jira] [Created] (ZEPPELIN-1223) Saving jdbc interpreter setting takes a long time

2016-07-21 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1223: Summary: Saving jdbc interpreter setting takes a long time Key: ZEPPELIN-1223 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1223 Project: Zeppelin Issu

[GitHub] zeppelin issue #1208: [gh-pages][ZEPPELIN-1216] Add a matrix table for "Zepp...

2016-07-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1208 @minahlee Yeah it makes sense. >How about move recent release to left? It would be better since this table becomes wider. I addressed it. And also `Geode` :) --- If your project

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

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

[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 comm

[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 e

[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 e

[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 hav

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

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

[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 featur

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

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

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

2016-07-21 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1163 Re-trigger CI --- 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

[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 is

[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? [Feat

[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? [Feat

[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 is

[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 no

[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 #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 to

[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 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 r

Interpreter password security

2016-07-21 Thread Eros Espínola González
Hi all, so I was wondering if there is any plans or current work about the password security in interpreters, right now passwords are exposed, wondering if there is any approach right now for masking/encrypting passwords. Tx, Eros.

[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 #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 r

Re: Interpreter password security

2016-07-21 Thread Vinay Shukla
Hi Eros, Not sure if there is an active plan for this. But we should look at Hadoop Credential provider (https://issues.apache.org/jira/browse/HADOOP-11031) and see if that can be re-used in Zeppelin. The downside is brining Hadoop as a dependency but perhaps Credential Provider is refactored out

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

2016-07-21 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1211 I've checked if the tests of FileInterpreter have passed. 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

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 (https://issues.apache.org/jir

[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

[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 feat

[GitHub] zeppelin pull request #1213: [ZEPPELIN-1206] fix "name 'z' is not defined" w...

2016-07-21 Thread sixmen
GitHub user sixmen opened a pull request: https://github.com/apache/zeppelin/pull/1213 [ZEPPELIN-1206] fix "name 'z' is not defined" with python3 ### What is this PR for? PythonInterpreter can not use dynamic form with python3. Fix this problem. ### What type of PR i

[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 f

[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 ru

[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 spark/src/main/java/org/apache/zeppelin/spark/Dep

[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 s

[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 spark/src/main/java/org/apache/zeppelin/spark/Sp

[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 spark/src/main/java/org/apache/zeppelin/spark/DepI

[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 spark/src/main/java/org/apache/zeppelin/spark/S

[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 spark/src/main/java/org/apache/zeppelin/spark/Sp

[GitHub] zeppelin pull request #1212: ZEPPELIN-1222. ClassNotFoundException of SparkJ...

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

[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: Zeppeli

[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 the

[GitHub] zeppelin issue #1186: [ZEPPELIN-1179] Append scala version to maven artifact...

2016-07-21 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1186 @minahlee I have included the missing modules, and also updated the build to change the artifact version to _2.11 when performing the build based on Scala 2.11. The build is still failin

[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 issue #1197: [ZEPPELIN-1196] Fix for bug ZEPPELIN-1196

2016-07-21 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1197 @SachinJanani I tried to merge it but this is based on branch-0.6. Could you please rebase from current master? I've tried to do it but I occurs error about merge conflict. Can you handle it? And

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 #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 is

[jira] [Created] (ZEPPELIN-1225) Errors before the last shell command is ignored

2016-07-21 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1225: Summary: Errors before the last shell command is ignored Key: ZEPPELIN-1225 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1225 Project: Zeppelin Issue

[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 e

[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 issue #1214: ZEPPELIN-1224: Fix typo in method name

2016-07-21 Thread karup1990
Github user karup1990 commented on the issue: https://github.com/apache/zeppelin/pull/1214 I was thinking to keep changes minimal. Should I change all variable names ? @AhyoungRyu --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] zeppelin pull request #1215: ZEPPELIN-1225. Errors before the last shell com...

2016-07-21 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1215 ZEPPELIN-1225. Errors before the last shell command are ignored ### What is this PR for? The problem is that command "bash -c " will always return 0 as long as the last line of shell script ru

[GitHub] zeppelin issue #1215: ZEPPELIN-1225. Errors before the last shell command ar...

2016-07-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1215 Find ShellInterpreterTest has some format issue (it is windows format), I have changed it to unix format in this PR as well. --- If your project is set up for it, you can reply to this email and

[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 @AhyoungRyu 5a27871f5a9e436653c8862e6d2ecc8913fdd496 for documentation --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] zeppelin pull request #:

2016-07-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/zeppelin/commit/5a27871f5a9e436653c8862e6d2ecc8913fdd496#commitcomment-18350624 In docs/security/shiroauthentication.md: In docs/security/shiroauthentication.md on line 121: Here is a typo. `easely`

[GitHub] zeppelin pull request #:

2016-07-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/zeppelin/commit/5a27871f5a9e436653c8862e6d2ecc8913fdd496#commitcomment-18350630 In docs/security/shiroauthentication.md: In docs/security/shiroauthentication.md on line 110: It's a nit.. But could y

[GitHub] zeppelin pull request #:

2016-07-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/zeppelin/commit/5a27871f5a9e436653c8862e6d2ecc8913fdd496#commitcomment-18350641 In docs/security/shiroauthentication.md: In docs/security/shiroauthentication.md on line 128: Could you add a new line

[GitHub] zeppelin issue #1215: ZEPPELIN-1225. Errors before the last shell command ar...

2016-07-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1215 After checking ShellInterpreter.java, I found other 2 issues. * Should remove entry from executors after the shell execution is done otherwise it would accumulate more and more. * executor

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

2016-07-21 Thread rerngvit
Github user rerngvit commented on the issue: https://github.com/apache/zeppelin/pull/1211 @bzz No problem. The JIRA id is correct. --- 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

[GitHub] zeppelin pull request #1216: [ZEPPELIN-919] Apply new mechanism to Markdown

2016-07-21 Thread rerngvit
GitHub user rerngvit opened a pull request: https://github.com/apache/zeppelin/pull/1216 [ZEPPELIN-919] Apply new mechanism to Markdown ### What is this PR for? This PR applies the new interpreter registration mechanism to Markdown ### What type of PR is it? Improvem

[GitHub] zeppelin issue #1186: [ZEPPELIN-1179] Append scala version to maven artifact...

2016-07-21 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1186 Looks good to me. Merge if there is 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 hav

[GitHub] zeppelin issue #1214: ZEPPELIN-1224: Fix typo in method name

2016-07-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1214 I tried to find all `completor` from the root dir. That I found was all. Since it's just 8 part in 2 files (`SparkInterpreter.java` & `DepInterpreter.java`), we don't need to remain typos i thi

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

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