[GitHub] zeppelin issue #2034: [ZEPPELIN-2133] All interpreters sometimes throw rando...

2017-02-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2034 Thanks @prabhjyotsingh, Is it possible to keep the behavior consistent between notebook and interpreter setting page ? Because I am afraid it would confuse users when we introduce different

[GitHub] zeppelin issue #2034: [ZEPPELIN-2133] All interpreters sometimes throw rando...

2017-02-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2034 @zjffdu have made some changes to honour ZEPPELIN-1770, but this will be only when user restarts from notebook. But if a restart, edit or delete is made to an interpreter from interpreter

[GitHub] zeppelin pull request #1946: [ZEPPELIN-921] Apply new mechanism to KnitR and...

2017-02-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1946 --- 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 #2054: [ZEPPELIN-2113] Remove border and margin of Link this ...

2017-02-21 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2054 @1ambda Thanks for taking a look this! Regarding text layout, I don't consider left-pending because it already fixed #1983. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #2053: [ZEPPELIN-1964] Layout info is lost after refresh

2017-02-21 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2053 I will review and give you feedback soon. Thanks for fixing bug @astroshim! --- 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 #2054: [ZEPPELIN-2113] Remove border and margin of Link this ...

2017-02-21 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2054 I will review and comment soon. Thanks for fixing bug @soralee! --- 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 #2042: [ZEPPELIN-1588]: bumping nvd3 to 1.8.5

2017-02-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2042 Looks nice! Better than before I think. @Remilito The tooltip of scatter chart itself is good as you changed, but how about keeping these focusing marked on each axis?

[GitHub] zeppelin issue #2043: [ZEPPELIN-2114] adding reload endpoint

2017-02-21 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2043 >If you think this endpoint belongs to another API (NotebookRestApi?), let me know. I think `notebook-repositories` is right place for this as you did. @Leemoonsoo It would

[GitHub] zeppelin pull request #2053: [ZEPPELIN-1964] Layout info is lost after refre...

2017-02-21 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/2053 [ZEPPELIN-1964] Layout info is lost after refresh ### What is this PR for? This PR fixes layout info is lost after refreshing issue. ### What type of PR is it? Bug Fix

[GitHub] zeppelin issue #2036: [ZEPPELIN-2135] Don't re-license jdbc/src/main/java/or...

2017-02-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2036 merge to master and branch-0.7 if no more 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 does not

[GitHub] zeppelin issue #2052: [ZEPPELIN-2048] Can't run first paragraph when persona...

2017-02-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2052 LGTM. merge to master and branch-0.7 if no further comments --- 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 #2052: [ZEPPELIN-2048] Can't run first paragraph when persona...

2017-02-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/2052 CI has passed. Thank you @Leemoonsoo --- 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 #2052: [ZEPPELIN-2048] Can't run first paragraph when ...

2017-02-21 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/2052 [ZEPPELIN-2048] Can't run first paragraph when personalize mode on. ### What is this PR for? Problem of ZEPPELIN-2048 was because not set the user id when inserting new paragraph.

[GitHub] zeppelin pull request #2052: [ZEPPELIN-2048] Can't run first paragraph when ...

2017-02-21 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/2052 --- 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

[jira] [Created] (ZEPPELIN-2149) Each interpreter should have a seperate log file

2017-02-21 Thread Yesha Vora (JIRA)
Yesha Vora created ZEPPELIN-2149: Summary: Each interpreter should have a seperate log file Key: ZEPPELIN-2149 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2149 Project: Zeppelin

[jira] [Created] (ZEPPELIN-2148) On creation of Bar graph zeppelin UI shows it as mini graph

2017-02-21 Thread Yesha Vora (JIRA)
Yesha Vora created ZEPPELIN-2148: Summary: On creation of Bar graph zeppelin UI shows it as mini graph Key: ZEPPELIN-2148 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2148 Project: Zeppelin

[jira] [Created] (ZEPPELIN-2147) zeppelin should redirect to login page after browser session expires

2017-02-21 Thread Yesha Vora (JIRA)
Yesha Vora created ZEPPELIN-2147: Summary: zeppelin should redirect to login page after browser session expires Key: ZEPPELIN-2147 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2147 Project:

[jira] [Created] (ZEPPELIN-2146) Enhance the error message when "top" or similar commands are executed via zeppelin notebook.

2017-02-21 Thread Yesha Vora (JIRA)
Yesha Vora created ZEPPELIN-2146: Summary: Enhance the error message when "top" or similar commands are executed via zeppelin notebook. Key: ZEPPELIN-2146 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2146

[jira] [Created] (ZEPPELIN-2145) create notebook should have a file browser capability to select or create a folder

2017-02-21 Thread Yesha Vora (JIRA)
Yesha Vora created ZEPPELIN-2145: Summary: create notebook should have a file browser capability to select or create a folder Key: ZEPPELIN-2145 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2145

[GitHub] zeppelin issue #2015: [ZEPPELIN-2103] Unnecessary read to Helium registry

2017-02-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2015 Thanks @1ambda @felixcheung for review. Merge to master if 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

[GitHub] zeppelin issue #2043: [ZEPPELIN-2114] adding reload endpoint

2017-02-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2043 Thanks for the contribution. Do you mind add a documentation [here](https://github.com/apache/zeppelin/blob/master/docs/rest-api/rest-notebook.md) ? --- If your project is set up for it,

[GitHub] zeppelin issue #2052: [ZEPPELIN-2048] Can't run first paragraph when persona...

2017-02-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2052 CI failure doesn't look related, but to make sure, could you try restart failed job https://travis-ci.org/astroshim/zeppelin/jobs/203861608 again? Once failed job becomes green, you

[jira] [Created] (ZEPPELIN-2144) Python Interpreter although shows jobs are in running state , jobs are dependent on each other to finish

2017-02-21 Thread vamsi (JIRA)
vamsi created ZEPPELIN-2144: --- Summary: Python Interpreter although shows jobs are in running state , jobs are dependent on each other to finish Key: ZEPPELIN-2144 URL:

[GitHub] zeppelin pull request #2052: [ZEPPELIN-2048] Can't run first paragraph when ...

2017-02-21 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/2052 [ZEPPELIN-2048] Can't run first paragraph when personalize mode on. ### What is this PR for? Problem of ZEPPELIN-2048 was because not set the user id when inserting new paragraph.

[GitHub] zeppelin issue #2041: ZEPPELIN-2137 Changed "show line chart with focus" to ...

2017-02-21 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2041 ah, yes, I didn't realize this is based of branch-0.7 - we should create PR off master instead. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin pull request #2051: [ZEPPELIN-1968] Added property to disable hive ...

2017-02-21 Thread gfalcone
GitHub user gfalcone opened a pull request: https://github.com/apache/zeppelin/pull/2051 [ZEPPELIN-1968] Added property to disable hive user impersonation ### What is this PR for? Added new property "hive.proxy.user" to disable hive impersonation (on some clusters, this

[GitHub] zeppelin issue #1946: [ZEPPELIN-921] Apply new mechanism to KnitR and RRepl

2017-02-21 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1946 Thanks for checking it. As you mentioned, we removed those kind of tests but we should take care of it because we couldn't break to install any of modules. Your comment makes sense to me and

[GitHub] zeppelin issue #2049: [ZEPPELIN-1586][MINOR] add new line char before "[urls...

2017-02-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2049 @anthonycorbacho help review this. I think "[urls]" should be in new line. --- 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 #2015: [ZEPPELIN-2103] Unnecessary read to Helium registry

2017-02-21 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2015 Looks good to me! --- 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,

[GitHub] zeppelin pull request #2050: ZEPPELIN-2137 Changed "show line chart with foc...

2017-02-21 Thread samthebest
GitHub user samthebest opened a pull request: https://github.com/apache/zeppelin/pull/2050 ZEPPELIN-2137 Changed "show line chart with focus" to "zoom" https://issues.apache.org/jira/browse/ZEPPELIN-2137 Improvement ### How should this be tested? Run it, look

[GitHub] zeppelin issue #2034: [ZEPPELIN-2133] All interpreters sometimes throw rando...

2017-02-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2034 @FireArrow Yes, agreed, I too believe that there are few minimum expectation as an Admin, and as a end user that Zeppelin should work in desired way. So, if we all can come to a

[jira] [Created] (ZEPPELIN-2143) Unable to upload JSON files greater than 4MB

2017-02-21 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2143: Summary: Unable to upload JSON files greater than 4MB Key: ZEPPELIN-2143 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2143 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin pull request #1917: [ZEPPELIN-1988] precode execution in JDBCInterp...

2017-02-21 Thread dwhsys
Github user dwhsys closed the pull request at: https://github.com/apache/zeppelin/pull/1917 --- 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 #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-21 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1982 [ZEPPELIN-2069] Helium Package Configuration ### What is this PR for? Supporting helium package configurations. I attached screenshots. Implementation details. In

[GitHub] zeppelin pull request #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-21 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1982 --- 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 #2034: [ZEPPELIN-2133] All interpreters sometimes throw rando...

2017-02-21 Thread FireArrow
Github user FireArrow commented on the issue: https://github.com/apache/zeppelin/pull/2034 There should probably be a jira to decide the effect of restarting in different locations. For example: As an admin I expect the restart button in Interpreter to force a restart of all

[GitHub] zeppelin issue #2034: [ZEPPELIN-2133] All interpreters sometimes throw rando...

2017-02-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2034 Yes, I understand this will violate that [ZEPPELIN-1770](https://issues.apache.org/jira/browse/ZEPPELIN-1770). But right now I see following problem/challenges, and that is why I would

[GitHub] zeppelin issue #2042: [ZEPPELIN-1588]: bumping nvd3 to 1.8.5

2017-02-21 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2042 thanks! I'd say the tooltip for pie is nicer in your change and perhaps tooltip for scatter the original one might be nicer. Since this is a visible change I'll leave it open for others to

[GitHub] zeppelin issue #2034: [ZEPPELIN-2133] All interpreters sometimes throw rando...

2017-02-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2034 @prabhjyotsingh Would this violate the behavior of [ZEPPELIN-1770](https://issues.apache.org/jira/browse/ZEPPELIN-1770) ? --- If your project is set up for it, you can reply to this email and

[jira] [Created] (ZEPPELIN-2142) Get Author of COMMIT from COMMIT (Jenkins job: zeppelin-pull-request)

2017-02-21 Thread Maks (JIRA)
Maks created ZEPPELIN-2142: -- Summary: Get Author of COMMIT from COMMIT (Jenkins job: zeppelin-pull-request) Key: ZEPPELIN-2142 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2142 Project: Zeppelin

[GitHub] zeppelin issue #1582: [ZEPPELIN-1604] Add Neo4j interpreter and Network visu...

2017-02-21 Thread conker84
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/1582 If i can do anything to simplify the review process, for me there is no problem to split this PR into more activities; just one thing, for me the (c) and (b) are one piece becouse the (c) expose