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

2017-03-29 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 @AhyoungRyu Thanks for detailed review. 1. It would be nice to have some popup or feedback when conf is updated. 2. Let me hide them when panel is minimized (e.g the chart

[GitHub] zeppelin issue #2197: ZEPPELIN-2324. Add property zeppelin.spark.unSupported...

2017-03-29 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2197 > perhaps it should be changed in sparkInterpreter.getSparkVersion().isSupportedVersion() instead I thought about it, but feel it is not proper to create a new member field in class

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

2017-03-29 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2177 @AhyoungRyu Could you 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 feature

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

2017-03-29 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2176 @AhyoungRyu Could you 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 feature

[GitHub] zeppelin issue #2151: [ZEPPELIN-465] Be able to run paragraph and the follow...

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2151 @Remilito Tested and it works as expected. Regarding docs, how about adding new section for this feature instead of `subsequent parameter (optional)` as a subsection under "Run a paragraph

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

2017-03-29 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2133 @Leemoonsoo Thanks for review and suggestions! I think manual options depending on users seems great so far, because I didn't find automatic calculation about width of `xLabel`. Let me dig

[GitHub] zeppelin issue #2204: ZEPPELIN-2189. The order of dynamic forms should be th...

2017-03-29 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2204 Thanks @AhyoungRyu for review, just fix a code style 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 project does not

[GitHub] zeppelin issue #2204: ZEPPELIN-2189. The order of dynamic forms should be th...

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2204 Yeah I saw this as well. Let me take a look :) --- 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 #2204: ZEPPELIN-2189. The order of dynamic forms shoul...

2017-03-29 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2204 ZEPPELIN-2189. The order of dynamic forms should be the order that you create them ### What is this PR for? The order of dynamic forms should be the order that you create them. So I

[GitHub] zeppelin issue #2135: Groovy Interpreter for Apazhe Zeppelin [ZEPPELIN-2176]

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2135 Thanks for addressing my comments! I left some minor comments again about "author name" in the source code and removing unnecessary `log.info`. Except these two things, LGTM 👍 --- If

Zeppelin Spark Streaming Twitter Stuck

2017-03-29 Thread Chaoran Yu
Hello, Has anybody got Spark Streaming Twitter example to work in Zeppelin? When I started the streaming context with ssc.start(), the Zeppelin paragraph seemed to have started but it got stuck there. The top right corner of the paragraph says “RUNNING 0%” I think this is a problem with

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

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2174 @felixcheung 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 feature enabled and

[GitHub] zeppelin issue #2189: [branch-0.7] Minor styling for Helium menu

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2189 @felixcheung Thanks! Merge into branch-0.7 if there are no further 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

[GitHub] zeppelin issue #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...

2017-03-29 Thread markito
Github user markito commented on the issue: https://github.com/apache/zeppelin/pull/2199 I'm sorry, skipped that part from the contribution guide. Will do and amend. Thanks! --- 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 #2135: Groovy Interpreter for Apazhe Zeppelin [ZEPPELIN-2176]

2017-03-29 Thread dlukyanov
Github user dlukyanov commented on the issue: https://github.com/apache/zeppelin/pull/2135 @AhyoungRyu, changes committed,waiting for travis --- 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 #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2174 @1ambda I updated the pagination as you said and this is the current status. Will be appreciated if you can take a look again.

[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-29 Thread sotnich
Github user sotnich commented on the issue: https://github.com/apache/zeppelin/pull/2134 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 pull request #2203: [ZEPPELIN-2297] improvements to jdbc autocomple...

2017-03-29 Thread tinkoff-dwh
GitHub user tinkoff-dwh opened a pull request: https://github.com/apache/zeppelin/pull/2203 [ZEPPELIN-2297] improvements to jdbc autocompleter ### What is this PR for? PR contains some improvements for completion (JDBC Interpreter): - types of completion - display of

Re: Could not find or load main class run when using yarn run dev

2017-03-29 Thread Jeff Zhang
That's it, thanks. Remy Gayet 于2017年3月29日周三 下午5:14写道: > Is there any chance you are using hadoop-yarn instead of yarn pkg manager ( > https://yarnpkg.com/lang/en/)? > > On Wed, Mar 29, 2017 at 9:48 AM, Jeff Zhang wrote: > > > Hi Folks, > > > > I follow

Re: Could not find or load main class run when using yarn run dev

2017-03-29 Thread Remy Gayet
Is there any chance you are using hadoop-yarn instead of yarn pkg manager ( https://yarnpkg.com/lang/en/)? On Wed, Mar 29, 2017 at 9:48 AM, Jeff Zhang wrote: > Hi Folks, > > I follow the instruction here to debug zeppelin-web, but got the following > error, > Do I miss

[GitHub] zeppelin issue #2189: [branch-0.7] Minor styling for Helium menu

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2189 @felixcheung Is this good to go 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

[jira] [Created] (ZEPPELIN-2329) Restart interpreter from error for non admin users

2017-03-29 Thread Guillaume Van Delsen (JIRA)
Guillaume Van Delsen created ZEPPELIN-2329: -- Summary: Restart interpreter from error for non admin users Key: ZEPPELIN-2329 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2329 Project:

[GitHub] zeppelin pull request #2202: [Minor][ZEPPELIN-2328] Separate Helium related ...

2017-03-29 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/2202 [Minor][ZEPPELIN-2328] Separate Helium related docs from 'Contribution' section ### What is this PR for? Actually writing "Visualization", "Spell" or "Application" type of Helium package

[GitHub] zeppelin issue #2198: [ZEPPELIN-2318] Fix proxy configuration for http clien...

2017-03-29 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2198 @felixcheung right `warning` level makes more sense in those cases, addressed in 8ea7d5d9652fa00d00cb6d749f0e1e718beb8c95 --- If your project is set up for it, you can reply to this email

Could not find or load main class run when using yarn run dev

2017-03-29 Thread Jeff Zhang
Hi Folks, I follow the instruction here to debug zeppelin-web, but got the following error, Do I miss something ? Thanks jzhangMBPr:zeppelin-web jzhang$ yarn run dev Error: Could not find or load main class run

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

2017-03-29 Thread Hyung Sung Shim
+1 Tested tutorials of zeppelin-0.7.1-bin-all on CentOS7. Thank you for your effort mina! 2017-03-29 16:04 GMT+09:00 DuyHai Doan : > +1 > Le 29 mars 2017 07:06, "Jeff Zhang" a écrit : > > > Thanks Jongyoul for explanation, let's improve it later. > > > >

[GitHub] zeppelin issue #2201: [ZEPPELIN-1965] Livy SQL Interpreter: Should use df.sh...

2017-03-29 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2201 One minor thing left, could you add test case for 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

[GitHub] zeppelin issue #2201: [ZEPPELIN-1965] Livy SQL Interpreter: Should use df.sh...

2017-03-29 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2201 Thanks @benoyantony for contribution, would you mind to add this configuration into livy.md as well ? --- If your project is set up for it, you can reply to this email and have your reply appear

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

2017-03-29 Thread DuyHai Doan
+1 Le 29 mars 2017 07:06, "Jeff Zhang" a écrit : > Thanks Jongyoul for explanation, let's improve it later. > > +1. > > * Build zeppelin from source successfully > * Run pig tutorial using tez mode successfully > * Run spark tutorial using livy interpreter successfully. > *

[jira] [Created] (ZEPPELIN-2328) Need to separate Helium related docs pages from "Contribute" section

2017-03-29 Thread Ahyoung Ryu (JIRA)
Ahyoung Ryu created ZEPPELIN-2328: - Summary: Need to separate Helium related docs pages from "Contribute" section Key: ZEPPELIN-2328 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2328 Project:

[GitHub] zeppelin issue #162: ZEPPELIN-179: Cassandra Interpreter

2017-03-29 Thread liuyangandy
Github user liuyangandy commented on the issue: https://github.com/apache/zeppelin/pull/162 @doanduyhai My Zeppelin and Cassandra is running in DCOS, my ```cassandra.hosts``` is my one of Cassandra cluster node IP, others properties are default. But when I try to test connectivity in

[GitHub] zeppelin pull request #2201: [ZEPPELIN-1965] Livy SQL Interpreter: Should us...

2017-03-29 Thread benoyantony
GitHub user benoyantony opened a pull request: https://github.com/apache/zeppelin/pull/2201 [ZEPPELIN-1965] Livy SQL Interpreter: Should use df.show(1000, false)… … to display results ### What is this PR for? Livy SQL interpreter truncate result strings of size

[GitHub] zeppelin issue #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...

2017-03-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2199 Hi @markito Thanks for your contribution and taking care of Geode interpreter. - Zeppelin build system relies on travis. So please setup your travis account and travis project for

[GitHub] zeppelin issue #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...

2017-03-29 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2199 looks good, could you please setup CI as referenced here https://zeppelin.apache.org/contribution/contributions.html#continuous-integration --- If your project is set up for it, you can