[GitHub] zeppelin issue #2466: [ZEPPELIN-2731] GetUserList with JDBCRealm should read...

2017-07-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2466 Thanks @reminia. Merge to master if no further discussions. --- 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 pull request #2471: [ZEPPELIN-2744] [minor] Increase error handling...

2017-07-06 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/2471 [ZEPPELIN-2744] [minor] Increase error handling in JDBC interpreter ### What is this PR for? At times running a phoenix sql query throws "ResultSet is closed." ``` java.sql

[GitHub] zeppelin pull request #2443: [ZEPPELIN-2698] Apply KerberosInterpreter to JD...

2017-07-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2443 --- 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-2744) Increase error handling in JDBC interpreter

2017-07-06 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-2744: - Summary: Increase error handling in JDBC interpreter Key: ZEPPELIN-2744 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2744 Project: Zeppelin

[GitHub] zeppelin issue #2466: [ZEPPELIN-2731] GetUserList with JDBCRealm should read...

2017-07-06 Thread reminia
Github user reminia commented on the issue: https://github.com/apache/zeppelin/pull/2466 restart failed job, CI looks good now. --- 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 en

[GitHub] zeppelin issue #2449: [Zeppelin-2676] recent notes list

2017-07-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2449 I have a suggestion. - Place `Recent` before `Notebook` (more upper side) - Limit the maximum number of recent notebooks. That's because users are accessing recent notebook mor

[jira] [Created] (ZEPPELIN-2743) java.sql.SQLException: [Teradata][Presto](100200) Connection string is invalid: driver is not recognized.

2017-07-06 Thread Rajesh Patel (JIRA)
Rajesh Patel created ZEPPELIN-2743: -- Summary: java.sql.SQLException: [Teradata][Presto](100200) Connection string is invalid: driver is not recognized. Key: ZEPPELIN-2743 URL: https://issues.apache.org/jira/brows

[GitHub] zeppelin pull request #2464: [ZEPPELIN-2722]. Use Note.toJson and fromJson i...

2017-07-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2464 --- 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 #2466: [ZEPPELIN-2731] GetUserList with JDBCRealm should read...

2017-07-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2466 Thanks @reminia for the fix. It looks good to me, and ci failure does not look related to this PR. but just in case, could you restart failed jobs https://travis-ci.org/reminia/zeppelin

[GitHub] zeppelin issue #2468: [ZEPPELIN-2733] Remove System Information Leak in Auth...

2017-07-06 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2468 @yanboliang Thanks for taking care of this. Yeah, It makes sense. I guess this PR works well, because you just removed log. But it would be great to make sure that CI pass. Could you c

[GitHub] zeppelin issue #2467: [ZEPPELIN-2730] compare with maxLength after getting a...

2017-07-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2467 @gaoyang143 Thanks for the fix. Could you try setup [CI](http://zeppelin.apache.org/contribution/contributions.html#continuous-integration) and see it passes the CI? --- If your project i

[GitHub] zeppelin issue #2468: [ZEPPELIN-2733] Remove System Information Leak in Auth...

2017-07-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2468 Thanks @yanboliang for the fix. LGTM and merge to master if 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

[GitHub] zeppelin issue #2470: [ZEPPELIN-2648] Fix spark module build problem with sc...

2017-07-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2470 Thanks @reminia for the fix. LGTM \cc @jongyoul @zjffdu --- 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 projec

[GitHub] zeppelin pull request #2465: [ZEPPELIN-2724] Move NotebookSocket.send() out ...

2017-07-06 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/zeppelin/pull/2465 [ZEPPELIN-2724] Move NotebookSocket.send() out of synchronized block ### What is this PR for? This PR tries to address https://issues.apache.org/jira/browse/ZEPPELIN-2724. ### W

[GitHub] zeppelin pull request #2465: [ZEPPELIN-2724] Move NotebookSocket.send() out ...

2017-07-06 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/2465 --- 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 #2470: [ZEPPELIN-2648] Fix spark module build problem ...

2017-07-06 Thread reminia
GitHub user reminia opened a pull request: https://github.com/apache/zeppelin/pull/2470 [ZEPPELIN-2648] Fix spark module build problem with scala-2.10 profile * remove scala version in spark2 profile , we should compose scala profile and spark profile to address scala and spark ver

[jira] [Created] (ZEPPELIN-2742) Unify Zeppelin's Storage Layer

2017-07-06 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-2742: Summary: Unify Zeppelin's Storage Layer Key: ZEPPELIN-2742 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2742 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin pull request #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-06 Thread jerryshao
Github user jerryshao closed the pull request at: https://github.com/apache/zeppelin/pull/2459 --- 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 i

[GitHub] zeppelin pull request #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-06 Thread jerryshao
GitHub user jerryshao reopened a pull request: https://github.com/apache/zeppelin/pull/2459 [ZEPPELIN-2716] Change the default value of zeppelin.livy.displayAppInfo to true ### What is this PR for? Since it is quite useful to expose the application info for user to monitor

[GitHub] zeppelin issue #2459: [ZEPPELIN-2716] Change the default value of zeppelin.l...

2017-07-06 Thread jerryshao
Github user jerryshao commented on the issue: https://github.com/apache/zeppelin/pull/2459 Thanks guys for your review, seems there's always some UT failures. I will reopen this PR to try again. --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] zeppelin issue #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2329 This is where you use custom ClassLoader. Seems YarnClient didn't use the custom class loader. ``` this.configuration.setClassLoader(classLoader); this.yarnClient = YarnClient.createYa

[GitHub] zeppelin pull request #:

2017-07-06 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/zeppelin/commit/0c08dc1277bd7349111e1f8f30703fe0b9fb4098#commitcomment-22963139 See this commit, please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] zeppelin issue #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 I changed my code of passing custom classloader to clustermanager and it would be passed into Configuration of hadoop --- If your project is set up for it, you can reply to this email and have y

[GitHub] zeppelin issue #2429: [ZEPPELIN-2675] Distributing Jars when using an extern...

2017-07-06 Thread marcotagliabue
Github user marcotagliabue commented on the issue: https://github.com/apache/zeppelin/pull/2429 I restarted build in Travis and now is ok. The problem seems to be flaky tests. @1ambda https://travis-ci.org/radicalbit/zeppelin --- If your project is set up for it, you can

[GitHub] zeppelin pull request #2456: [ZEPPELIN-2714] Soft-code Spark UI button visua...

2017-07-06 Thread necosta
GitHub user necosta reopened a pull request: https://github.com/apache/zeppelin/pull/2456 [ZEPPELIN-2714] Soft-code Spark UI button visualization ### What is this PR for? When "spark.ui.enabled" property is set to "false" we should not show the Spark UI button. We keep the s

[GitHub] zeppelin pull request #2456: [ZEPPELIN-2714] Soft-code Spark UI button visua...

2017-07-06 Thread necosta
Github user necosta closed the pull request at: https://github.com/apache/zeppelin/pull/2456 --- 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 #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2329 @jongyoul unfortunately, I still have this issue. And the only place you use custom classloader is in `ClusterManagerFactory.java`. But I only see you use it to load the ClusterManager class (like

[GitHub] zeppelin issue #2443: [ZEPPELIN-2698] Apply KerberosInterpreter to JDBCInter...

2017-07-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2443 @zjffdu Thank you for the review. CI is green, will merge 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

[GitHub] zeppelin pull request #2443: [ZEPPELIN-2698] Apply KerberosInterpreter to JD...

2017-07-06 Thread prabhjyotsingh
GitHub user prabhjyotsingh reopened a pull request: https://github.com/apache/zeppelin/pull/2443 [ZEPPELIN-2698] Apply KerberosInterpreter to JDBCInterpreter ### What is this PR for? This is to apply new KerberosInterpreter mechanism to JDBCInterpreter for relogin from keytab, i

[GitHub] zeppelin pull request #2443: [ZEPPELIN-2698] Apply KerberosInterpreter to JD...

2017-07-06 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/2443 --- 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 feat

[GitHub] zeppelin issue #2429: [ZEPPELIN-2675] Distributing Jars when using an extern...

2017-07-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2429 Thanks for the quick reply. There are 2 failed jobs. We can restart or need to fix them. https://travis-ci.org/radicalbit/zeppelin/builds/250680464 --- If your project is set up

[GitHub] zeppelin pull request #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-06 Thread jerryshao
Github user jerryshao closed the pull request at: https://github.com/apache/zeppelin/pull/2459 --- 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 i

[GitHub] zeppelin pull request #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-06 Thread jerryshao
GitHub user jerryshao reopened a pull request: https://github.com/apache/zeppelin/pull/2459 [ZEPPELIN-2716] Change the default value of zeppelin.livy.displayAppInfo to true ### What is this PR for? Since it is quite useful to expose the application info for user to monitor

[jira] [Created] (ZEPPELIN-2741) [FRONT-BUILD] Apply angular eslint rule

2017-07-06 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2741: --- Summary: [FRONT-BUILD] Apply angular eslint rule Key: ZEPPELIN-2741 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2741 Project: Zeppelin Issue Type: Sub-

[GitHub] zeppelin issue #2469: [ZEPPELIN-2726] Move angular related pkgs from bower t...

2017-07-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2469 Thanks for the review. @soralee @sravan-s we are using annotation for angular, but two angular conflicts. (bower side required by angular-websocket) Thus, I will change this PR a

[GitHub] zeppelin issue #2446: [ZEPPELIN-2699] Helium REST API bug-fixes. Also update...

2017-07-06 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2446 Hi @1ambda , one more attempt :) can you have a look at this latest version? Thanks for checking the UI side, my bad on only running the unit tests. I cannot guarantee 100% but to me everything lo

[GitHub] zeppelin issue #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-07-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2418 For that purpose, you can use zeppelin's rest api or note's cron scheduler https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/usage/rest_api/notebook.html --- If your project is set up for it, you

[GitHub] zeppelin issue #2467: [ZEPPELIN-2730] compare with maxLength after getting a...

2017-07-06 Thread gaoyang143
Github user gaoyang143 commented on the issue: https://github.com/apache/zeppelin/pull/2467 @Leemoonsoo please reivew 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

[jira] [Created] (ZEPPELIN-2740) Pyspark not working error thrown after installing Zeppelin

2017-07-06 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2740: Summary: Pyspark not working error thrown after installing Zeppelin Key: ZEPPELIN-2740 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2740 Project: Zeppelin Iss

[GitHub] zeppelin issue #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu Can you check it again? I've reproduced and fixed 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

[GitHub] zeppelin issue #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-07-06 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/2418 @zjffdu I am thinking to automation (using the notes in an scheduled or batch process). Does it make sense to you? --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #2456: [ZEPPELIN-2714] Soft-code Spark UI button visualizatio...

2017-07-06 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2456 @Leemoonsoo , @felixcheung , one minor commit added. Fixed merge conflicts. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] zeppelin issue #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-07-06 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2418 @echarles Thanks for the feedback. Currently Interpreter Process can only communicate with Zeppelin Server process, and it is a internal communication channel which is not exposed to users and othe

[GitHub] zeppelin issue #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-07-06 Thread cacti77
Github user cacti77 commented on the issue: https://github.com/apache/zeppelin/pull/2437 Thanks @1ambda . So, looking at #2371, it looks like a user would simply click on the dropdown and then Interpreter > Interpreter Binding Mode to access the new web page. Is that correct? Basicall

[GitHub] zeppelin issue #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-07-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2437 @cacti77 it's provided as of 0.8.0-SNAPSHOT. - http://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/usage/interpreter/interpreter_binding_mode.html here are the background - we

[GitHub] zeppelin issue #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-07-06 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/2418 @jongyoul sorry, I was not clear enough. With frontend, I was meaning the web layer (so jetty). Jetty could communicate via a variety of protocols to the interpreters. Now going further in that d

[jira] [Created] (ZEPPELIN-2739) [FRONT-BUILD] Upgrade to webpack2

2017-07-06 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2739: --- Summary: [FRONT-BUILD] Upgrade to webpack2 Key: ZEPPELIN-2739 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2739 Project: Zeppelin Issue Type: Sub-task

[jira] [Created] (ZEPPELIN-2738) Zeppelin Interpreter crashing

2017-07-06 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2738: Summary: Zeppelin Interpreter crashing Key: ZEPPELIN-2738 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2738 Project: Zeppelin Issue Type: Bug Repo

[GitHub] zeppelin issue #2429: [ZEPPELIN-2675] Distributing Jars when using an extern...

2017-07-06 Thread marcotagliabue
Github user marcotagliabue commented on the issue: https://github.com/apache/zeppelin/pull/2429 Hi! I've just updated branch. CI Should be ok. Thanks for the review! @1ambda --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] zeppelin pull request #2469: [ZEPPELIN-2725] Move angular related pkgs from ...

2017-07-06 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2469 --- 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 #2469: [ZEPPELIN-2725] Move angular related pkgs from ...

2017-07-06 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2469 [ZEPPELIN-2725] Move angular related pkgs from bower to npm (zeppelin-web) ### What is this PR for? Move angular related packages from bower to npm. Because bower team recommends using

[jira] [Created] (ZEPPELIN-2737) Unable to run pyspark in Zeppelin 0.7.2

2017-07-06 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2737: Summary: Unable to run pyspark in Zeppelin 0.7.2 Key: ZEPPELIN-2737 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2737 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-07-06 Thread cacti77
Github user cacti77 commented on the issue: https://github.com/apache/zeppelin/pull/2437 @1ambda One question please: Do these docs constitute a new web page in Zeppelin's docs? I'm just wondering how a user would navigate to this new information on binding modes from the Docs home pa

[GitHub] zeppelin issue #2460: [gh-pages] fix hover event in technologies section on ...

2017-07-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2460 checked that works well, merge 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 t

[GitHub] zeppelin issue #2467: [ZEPPELIN-2730] compare with maxLength after getting a...

2017-07-06 Thread gaoyang143
Github user gaoyang143 commented on the issue: https://github.com/apache/zeppelin/pull/2467 @felixcheung already added test --- 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 enable

[jira] [Created] (ZEPPELIN-2736) Trigger code completion when use type tab

2017-07-06 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-2736: Summary: Trigger code completion when use type tab Key: ZEPPELIN-2736 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2736 Project: Zeppelin Issue Type:

[GitHub] zeppelin issue #2460: [gh-pages] fix hover event in technologies section on ...

2017-07-06 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2460 Simple fix, I just changed font-size from 24 to 23 when browser size is small. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y