[GitHub] zeppelin issue #2158: [ZEPPELIN-2279] excluded comments from SQL

2017-03-22 Thread masyan
Github user masyan commented on the issue: https://github.com/apache/zeppelin/pull/2158 @felixcheung falls because ";" is delimiter query and after split generated two queries ("/* ", "*/ select ...") I relied on the standards for SQL89, 92 ... if you're talking about

[GitHub] zeppelin issue #2158: [ZEPPELIN-2279] excluded comments from SQL

2017-03-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2158 @felixcheung falls because ";" is delimiter query and after split generated two queries ("/* ", "*/ select ...") I relied on the standards for SQL89, 92 ... if you're talking

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-03-22 Thread del680202
Github user del680202 commented on the issue: https://github.com/apache/zeppelin/pull/1976 @AhyoungRyu Thanks, I will check 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

[GitHub] zeppelin issue #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2174 @1ambda Just noticed that `orderBy: publishedDate` doesn't work well properly now. Let me updated again and will let you know when it's resolved. (Actually same problem is in

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1976 @del680202 >When I enable authentication with shiro or switch interpreter mode to Per user & isolated, Restart does not work and it is possible to make zombie process. This issue

[GitHub] zeppelin issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2119 Thanks @1ambda Merge into master and branch-0.7 if there are no more comments. --- 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 #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2144 Merge into master 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 have this

[GitHub] zeppelin issue #2141: [ZEPPELIN-1720] Adding tests to verify behaviour of dy...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2141 Merge into master and branch-0.7 if there are no more comments on 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

[GitHub] zeppelin issue #2133: [ZEPPELIN-1357][Umbrella] Text overlap in the MultiBar...

2017-03-22 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2133 Thanks @Leemoonsoo! I did it and CI is green 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

Zeppelin Spark Sql Interpreter CompileException

2017-03-22 Thread Chaoran Yu
Hello,   I’m running Zeppelin 0.7.0 with Mesos.  I tried to run the tutorial on this page https://zeppelin.apache.org/docs/0.7.0/quickstart/tutorial.html   The regular Spark code in Scala worked fine but I got a org.codehaus.commons.compiler.CompileException when running the Spark Sql code.

Re: Zeppelin Spark Sql Interpreter CompileException

2017-03-22 Thread Jeff Zhang
I suspect it is due to spark issue. Can you run it in spark shell local mode to verify that ? Chaoran Yu 于2017年3月23日周四 上午10:19写道: > Hello, > >I’m running Zeppelin 0.7.0 with Mesos. > > I tried to run the tutorial on this page >

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2098 @1ambda Looks awesome! Let me test it and i'll get back to you soon. --- 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 #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 @FRosner I see and agree with that we need more realtime notification on paragraph editing. LGTM --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2177: [ZEPPELIN-2301] DON'T overwrite editor text when note ...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2177 @FRosner I updated GIF. What #2176 and #2177 is trying to prevent local change from overwritten. Overwriting occurs in many cases. - #2176 is only about **when other browsers run

[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2176 @FRosner I updated GIF. - The basic motivation is, prevent overwriting my local changes when other browsers execute paragraphs. > From the user perspective, this is a critical

[GitHub] zeppelin pull request #2150: [ZEPPELIN-2278] Env variable to configure Npm r...

2017-03-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2150 --- 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 #2167: [ZEPPELIN-2230] .travis.yml protecting against ...

2017-03-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2167 --- 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 #2158: [ZEPPELIN-2279] excluded comments from SQL

2017-03-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2158 LGTM Merge to master 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 not have

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 Looking at the discussion and the different issues I see what you mean @1ambda. We are actually having two issues: 1. Local changes are overwritten when the notebook gets updates from the

[GitHub] zeppelin issue #2177: [ZEPPELIN-2301] DON'T overwrite editor text when note ...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2177 I am not sure I understand the exact difference between this and #2176. Can you try to upload bigger versions of the gif or elaborate a bit on the difference? Are the two PR (#2176 and #2177)

[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2176 Hi @1ambda. What are the implications of this change? In the scenario you describe, it makes sense. So if person A is editing a cell and person B is running it at the same

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-03-22 Thread conker84
Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 Hi guys any news/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 feature enabled and

[GitHub] zeppelin pull request #2177: [ZEPPELIN-2301] DON'T overwrite editor text whe...

2017-03-22 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2177 --- 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 #2177: [ZEPPELIN-2301] DON'T overwrite editor text whe...

2017-03-22 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2177 [ZEPPELIN-2301] DON'T overwrite editor text when note is renamed, created, ... ### What is this PR for? DON'T overwrite local text change in editor when note is renamed, created, and

[GitHub] zeppelin pull request #2177: [ZEPPELIN-2301] DON'T overwrite editor text whe...

2017-03-22 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2177 [ZEPPELIN-2301] DON'T overwrite editor text when note is renamed, created, ... ### What is this PR for? DON'T overwrite local text change in editor when note is renamed, created, and so

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 Here you go @1ambda ![zeppelin-1492-1](https://cloud.githubusercontent.com/assets/3427394/24218616/2cf9cd0c-0f44-11e7-93bb-5ae375c0296b.gif) --- If your project is set up for it, you

[jira] [Created] (ZEPPELIN-2301) dirtyText is overwritten by note-level actions (rename, ..)

2017-03-22 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2301: --- Summary: dirtyText is overwritten by note-level actions (rename, ..) Key: ZEPPELIN-2301 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2301 Project: Zeppelin

Re: Roadmap for 0.8.0

2017-03-22 Thread Felix Cheung
+1 with latest/stable. From: moon soo Lee Sent: Tuesday, March 21, 2017 8:41:58 AM To: us...@zeppelin.apache.org Cc: dev@zeppelin.apache.org Subject: Re: Roadmap for 0.8.0 And if i suggest simplest way for us to set quality expectation to

[jira] [Created] (ZEPPELIN-2299) got 'NOTE' websocket message twice when renamed a note

2017-03-22 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2299: --- Summary: got 'NOTE' websocket message twice when renamed a note Key: ZEPPELIN-2299 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2299 Project: Zeppelin

[GitHub] zeppelin issue #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2174 @AhyoungRyu sure! --- 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 issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2119 I updated and now PR CI is green :) --- 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 #2170: [HOTFIX][ZEPPELIN-2149] correct variable name u...

2017-03-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2170 --- 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 #2098: [ZEPPELIN-2217][WIP] AdvancedTransformation for Visual...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 @Leemoonsoo, @AhyoungRyu implementation is finally finished (IMO 😭 ) - Please help review this. - Testing these 2 examples will be helpful (not published yet)

[GitHub] zeppelin pull request #1339: [ZEPPELIN-1332] Remove spark-dependencies & sug...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu closed the pull request at: https://github.com/apache/zeppelin/pull/1339 --- 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 #1339: [ZEPPELIN-1332] Remove spark-dependencies & suggest ne...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1339 I'm closing this PR since there'll be better solution for this (e.g. [ZEPPELIN-1993](https://issues.apache.org/jira/browse/ZEPPELIN-1993)) :) --- If your project is set up for it, you can

[jira] [Created] (ZEPPELIN-2298) Include Pyspark and SparkR in spark-dependencies by default

2017-03-22 Thread Ahyoung Ryu (JIRA)
Ahyoung Ryu created ZEPPELIN-2298: - Summary: Include Pyspark and SparkR in spark-dependencies by default Key: ZEPPELIN-2298 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2298 Project: Zeppelin

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 @FRosner Could you attach a GIF for ``` Enter code in a cell Remove the focus to rename the notebook The code gets reset ``` --- If your project is set up for it, you can

[GitHub] zeppelin issue #2175: [ZEPPELIN-2294]. Interpreter fail exception is not pro...

2017-03-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2175 @jongyoul @Leemoonsoo Please help review. --- 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 #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread sergeymazin
Github user sergeymazin commented on the issue: https://github.com/apache/zeppelin/pull/2120 Issue we are facing is when somebody else is adding/renaming/deleting any other notebook, unsaved text in current notebook is lost. Renaming any other notebook resets current

[GitHub] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2144 Cool! Tested again and it works well as expected. 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

Re: [VOTE] Release Apache Zeppelin 0.7.1 (RC1)

2017-03-22 Thread Jeff Zhang
I will change my vote to -1, as I feel this is a very critical bug which cause bad user experience. User don't know what's wrong if interpreter fails to start unless checking log. But in enterprise usage, most likely user don't have permission to access log. I created ticket

[GitHub] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2144 @AhyoungRyu I resolved your all comments. 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 your project does not have this

[GitHub] zeppelin pull request #2175: [ZEPPELIN-2294]. Interpreter fail exception is ...

2017-03-22 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2175 [ZEPPELIN-2294]. Interpreter fail exception is not propagated to frontend ### What is this PR for? This PR would address the issue of Interpreter fail exception is not propagated to

[GitHub] zeppelin issue #780: [ZEPPELIN-658] Scala: not accepting companion objects i...

2017-03-22 Thread rodbs
Github user rodbs commented on the issue: https://github.com/apache/zeppelin/pull/780 As @saj1th says, I'm still getting this issue with Zeppelin 0.7 while trying to use companion object; even placing the object definition in the same line as the class definition: `

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 @FRosner Sorry for late reply. 😭... ### I can't reproduce the original issue reported in 1492 in current master Usually, paragraph will be updated If interpreter updates

Re: [VOTE] Release Apache Zeppelin 0.7.1 (RC1)

2017-03-22 Thread Ahyoung Ryu
Instead of just replying "+1", will be better to write down what I checked. - there was no problem with running Spark tutorial note \w basic Spark interpreter setting - tested restarting interpreter issue which is the most critical one in 0.7.0 with markdown interpreter (checked every cases:

[GitHub] zeppelin issue #2170: [HOTFIX][ZEPPELIN-2149] correct variable name used in ...

2017-03-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2170 Will merge this soon. --- 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