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

2016-07-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1170 CI build failed due to networking issues ``` [ERROR] Failed to execute goal org.apache.maven.plugins:maven-remote-resources-plugin:1.4:process (default) on project zeppelin: Error

[GitHub] zeppelin issue #1194: [ZEPPELIN-1193] Update Node JS related dependencies to...

2016-07-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1194 So what do you guys suggest we do @corneadoug @lresende ? --- 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

[GitHub] zeppelin pull request #1233: [ZEPPELIN-1235] ignore rules for files generate...

2016-07-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1233 --- 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 #1240: [ZEPPELIN-1246] In JDBCInterpreter.getScheduler...

2016-07-28 Thread prasadwagle
GitHub user prasadwagle reopened a pull request: https://github.com/apache/zeppelin/pull/1240 [ZEPPELIN-1246] In JDBCInterpreter.getScheduler, use getMaxConcurrentConnection instead of hardcoding maxConcurrency to 10 ### What is this PR for? In JDBCInterpreter.getScheduler, use

[GitHub] zeppelin pull request #1240: [ZEPPELIN-1246] In JDBCInterpreter.getScheduler...

2016-07-28 Thread prasadwagle
Github user prasadwagle closed the pull request at: https://github.com/apache/zeppelin/pull/1240 --- 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-28 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1210 @mwpenny Sorry for the delay, I got a bit busy. I will take a look this week-end. What you can do until then is to rebase with master since there is some conflicts right now. --- If your

[GitHub] zeppelin issue #1218: [Zeppelin-1213] Customize editor configuration

2016-07-28 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1218 @bzz @corneadoug Okay, Thank you very much --- 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

[jira] [Created] (ZEPPELIN-1249) Set Scala 2.11 to default scala version

2016-07-28 Thread Mina Lee (JIRA)
Mina Lee created ZEPPELIN-1249: -- Summary: Set Scala 2.11 to default scala version Key: ZEPPELIN-1249 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1249 Project: Zeppelin Issue Type: Task

[jira] [Created] (ZEPPELIN-1248) DAG paragraph of paragraphs

2016-07-28 Thread jacquesqiao (JIRA)
jacquesqiao created ZEPPELIN-1248: - Summary: DAG paragraph of paragraphs Key: ZEPPELIN-1248 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1248 Project: Zeppelin Issue Type: New Feature

[GitHub] zeppelin issue #1221: [MINOR] Change url in pom.xml files

2016-07-28 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1221 I removed URLs from all child pom. Merge if there is no further 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 your

[GitHub] zeppelin issue #1220: [MINOR] Make scala version definition consistent in Tr...

2016-07-28 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1220 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 have this feature

[jira] [Created] (ZEPPELIN-1247) JdbcInterpreter is always chosen as the default interpreter.

2016-07-28 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1247: Summary: JdbcInterpreter is always chosen as the default interpreter. Key: ZEPPELIN-1247 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1247 Project: Zeppelin

[GitHub] zeppelin issue #1225: [Zeppelin-1229] Add cache buster for zeppelin-web buil...

2016-07-28 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1225 @Leemoonsoo in 0.6 or 0.6.1? --- 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

[GitHub] zeppelin issue #1232: ZEPPELIN-1197. Should print output directly without in...

2016-07-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1232 I also compare it with native python repl, the second case is consistent. So I think this behvior is fine, although it is different from jyputer.

[GitHub] zeppelin issue #1232: ZEPPELIN-1197. Should print output directly without in...

2016-07-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1232 @Leemoonsoo Thanks for the careful checking. I compare it with jupyter, only the second case is different. Let me investigate how to fix it. --- If your project is set up for it, you can reply

[GitHub] zeppelin pull request #1240: [ZEPPELIN-1246] In JDBCInterpreter.getScheduler...

2016-07-28 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/zeppelin/pull/1240 [ZEPPELIN-1246] In JDBCInterpreter.getScheduler, use getMaxConcurrentConnection instead of hardcoding maxConcurrency to 10 ### What is this PR for? In JDBCInterpreter.getScheduler, use

[jira] [Created] (ZEPPELIN-1246) In JDBCInterpreter.getScheduler, use getMaxConcurrentConnection instead of hardcoding maxConcurrency to 10

2016-07-28 Thread Prasad Wagle (JIRA)
Prasad Wagle created ZEPPELIN-1246: -- Summary: In JDBCInterpreter.getScheduler, use getMaxConcurrentConnection instead of hardcoding maxConcurrency to 10 Key: ZEPPELIN-1246 URL:

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-07-28 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz only difference between this and last was squash of commit. all fails are related to downloading dependencies still. --- If your project is set up for it, you can reply to this email

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

2016-07-28 Thread babupe
Github user babupe commented on the issue: https://github.com/apache/zeppelin/pull/1170 Awesome! Merged this. --- 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

[GitHub] zeppelin issue #1201: [MINOR] Enable pyspark test in local mode

2016-07-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1201 LGTM. CI failure is flaky one. i've seen it couple of times. --- 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 pull request #1201: [MINOR] Enable pyspark test in local mode

2016-07-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1201 --- 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 #1236: [ZEPPELIN-1237] Auto-suggestion of notebook permission...

2016-07-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1236 @r-kamath have implement your feedback. --- 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 #1201: [MINOR] Enable pyspark test in local mode

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1201 Thanks. I'll merge it --- 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

[GitHub] zeppelin issue #1233: [ZEPPELIN-1235] ignore rules for files generated by ze...

2016-07-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1233 Merging this if no more discussion. CI fails for unrelated issue. --- 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 #1234: [ZEPPELIN-1234] Fix for bug ZEPPELIN-1234

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1234 @SachinJanani Could you please add test case for this bug? It would help understand the situation. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1226: Fix logger class name to correct one

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1226 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 #1201: [MINOR] Enable pyspark test in local mode

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1201 I've seen it for the first time. Do you have any hint to deal with? --- 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 #1233: [ZEPPELIN-1235] ignore rules for files generated by ze...

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1233 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 #1239: [HOTFIX][ZEPPELIN-1240] Removed interpreter properties...

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1239 I'll trigger the CI one more, and merge it --- 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 pull request #1239: [HOTFIX][ZEPPELIN-1240] Removed interpreter pro...

2016-07-28 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/1239 --- 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 issue #1232: ZEPPELIN-1197. Should print output directly without in...

2016-07-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1232 Thanks @zjffdu for great improvement. I have tested bit and i could see some inconsistent behavior.

[GitHub] zeppelin issue #1239: [HOTFIX][ZEPPELIN-1240] Removed interpreter properties...

2016-07-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1239 Thanks for the fix. Tested and 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

[GitHub] zeppelin issue #928: [ZEPPELIN-116] Add Apache Mahout Interpreter

2016-07-28 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/928 @bzz and @jongyoul OK, so two things I think have happened. (In case some thing like this happens again). When I rebased, some version of ZeppelinClusterTest.java that had the

[GitHub] zeppelin issue #1239: [HOTFIX][ZEPPELIN-1240] Removed interpreter properties...

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1239 @Leemoonsoo I fixed it. Review it, please. --- 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 #1168: [MINOR] Add formatter

2016-07-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1168 @bzz Sure, I agree. it's kind of separate issue. I think enforcing code format at first supporting formatter later would be a good plan. How about you? --- If your project is set up for it, you

[jira] [Created] (ZEPPELIN-1245) Focus first paragraph after notebook creation

2016-07-28 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1245: -- Summary: Focus first paragraph after notebook creation Key: ZEPPELIN-1245 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1245 Project: Zeppelin

[GitHub] zeppelin issue #1233: [ZEPPELIN-1235] ignore rules for files generated by ze...

2016-07-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1233 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 #1215: ZEPPELIN-1225. Errors before the last shell command ar...

2016-07-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1215 Thanks @zjffdu for the contribution. LGTM. Tested, CI failure irrelevant. Merge it into master and branch-0.6 --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1225: [Zeppelin-1229] Add cache buster for zeppelin-web buil...

2016-07-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1225 LGTM. Can we merge it into branch-0.6 as well? --- 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

[GitHub] zeppelin issue #1228: ZEPPELIN-1175. AM log is not available for yarn-client...

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

[GitHub] zeppelin pull request #1238: ZEPPELIN-1115: add correct %python.sql interpre...

2016-07-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1238 --- 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 #1238: ZEPPELIN-1115: add correct %python.sql interpreter nam...

2016-07-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1238 Thanks for the prompt review, 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

[GitHub] zeppelin issue #1238: ZEPPELIN-1115: add correct %python.sql interpreter nam...

2016-07-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1238 Merging it ASAP as a hotfix --- 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

[GitHub] zeppelin pull request #:

2016-07-28 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/zeppelin/commit/5a4aacef25b0b54d151cfc7a3ea81cc312f6f655#commitcomment-18428447 In jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java: In

[GitHub] zeppelin issue #1168: [MINOR] Add formatter

2016-07-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1168 Do you think we should merge it as is and address eclipse formatter in the separate issue? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #1218: [Zeppelin-1213] Customize editor configuration

2016-07-28 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1218 @cloverhearts Don't hesitate to ping me when you start making the small PRs --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #1235: ZEPPELIN-1227 change spark-2.0.0-preview to spark-2.0....

2016-07-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1235 Looks great to me, modulo CI failure --- 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

[GitHub] zeppelin issue #1237: ZEPPELIN-1242. Should set property SPARK_YARN_MODE and...

2016-07-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1237 Build fails due to network issue. ``` [INFO] [ERROR] Failed to execute goal

[GitHub] zeppelin issue #1206: ZEPPELIN-1199. Need to login using keytab and principa...

2016-07-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1206 Close it and merge it to #1237 --- 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

[GitHub] zeppelin issue #1237: ZEPPELIN-1242. Should set property SPARK_YARN_MODE and...

2016-07-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1237 @prabhjyotsingh I close #1206, and merge it here. Please help review, thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If