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

2017-03-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2098 @1ambda I feel so sorry for my late response. Tested this patch \w your Helium package: zeppelin-ultimate-line-chart. Here are some thought and question(?) in my head. 1. "Save" &

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

2017-03-28 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2158 Ready to 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 enabled and wishes

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

2017-03-28 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2134 ready to 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 enabled and wishes

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

2017-03-28 Thread Jeff Zhang
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. * Verify interpreter fail error can be propagated to frontend. * Verify the restart

[GitHub] zeppelin issue #2200: [HOTFIX] Remove unintended characters from r tutorial

2017-03-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2200 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 #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 Let me add docs and unit tests. I guess it will take few days. Then i will comment. Thanks @Leemoonsoo --- If your project is set up for it, you can reply to this email and have your reply

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

2017-03-28 Thread Jongyoul Lee
Jeff, Moon and I discuss the behaviors of clicking restart buttons from notebook page and interpreter page. In your case, it's intended fully. For more detail, In case of "per user", we can do the same thing in note and interpreter pages, but in case of "per note", we cannot get the exact note

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

2017-03-28 Thread Jeff Zhang
Might find another bug about restarting button. The restarting button on note page behaves differently from that of interpreter page. This might confuse users. e.g. For livy interpreter, if I click the restart button in note page, then only my livy session will be shutdown. But If I click the

[jira] [Created] (ZEPPELIN-2327) Add option to dynamic form to control whether running paragraph automatically

2017-03-28 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-2327: Summary: Add option to dynamic form to control whether running paragraph automatically Key: ZEPPELIN-2327 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2327

[GitHub] zeppelin pull request #2200: [HOTFIX] Remove unintended characters from r tu...

2017-03-28 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/zeppelin/pull/2200 [HOTFIX] Remove unintended characters from r tutorial ### What is this PR for? If user runs **Create a R Dataframe** paragraph in **Zeppelin Tutorial/R (SparkR)** note, he/she will get

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

2017-03-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2098 @1ambda Great work!!! Looks great to me! I think it's worth to move usages and examples in this PR description under `/docs`. Where do you think this information can be

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

2017-03-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2133 I tested a bit more and found some cases. 1. Label rotated when unnecessary

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

2017-03-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2135 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 this

[jira] [Created] (ZEPPELIN-2326) Updating Geode dependencies and imports due to package rename

2017-03-28 Thread William Markito Oliveira (JIRA)
William Markito Oliveira created ZEPPELIN-2326: -- Summary: Updating Geode dependencies and imports due to package rename Key: ZEPPELIN-2326 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2326

[GitHub] zeppelin pull request #2198: [ZEPPELIN-2318] Fix proxy configuration for htt...

2017-03-28 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/2198 [ZEPPELIN-2318] Fix proxy configuration for http client of zeppelinhub storage layer ### What is this PR for? it fixes proxy configuration for http client of zeppelinhub storage

[jira] [Created] (ZEPPELIN-2325) using %dep for spark interpreter throw npe when user impersonate is enabled

2017-03-28 Thread Guillaume Van Delsen (JIRA)
Interpreter896995192 ==> zeppelin-zeppelin-cfbe-dev-cld-01.log <== WARN [2017-03-28 15:59:52,578] ({pool-2-thread-18} NotebookServer.java[afterStatusChange]:2162) - Job 20170328-151439_549179296 is finished, status: ERROR, exception: null, result: %text java

[GitHub] zeppelin issue #2192: [ZEPPELIN-2320] fixing test + modifying map browsing

2017-03-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2192 @astroshim Can you merge it into master and branch-0.7? --- 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 #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2174 @AhyoungRyu Yes, it would be great to hide unnecessary things --- 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 pull request #2191: [HOTFIX] Keep interpreter group when save and r...

2017-03-28 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/2191 --- 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 #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2174 @1ambda Thanks for the detailed explanation. ![record](https://cloud.githubusercontent.com/assets/10060731/24408816/ae0257ec-1409-11e7-8534-8ef1b73a14d0.gif) --- If your project is set

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

2017-03-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2174 @AhyoungRyu more precisely, 1. If we are in page 1, we don't need `<<`

[GitHub] zeppelin issue #2113: [ZEPPELIN-2190] Support custom web development port

2017-03-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2113 LGTM and 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 does not

[GitHub] zeppelin issue #2178: [ZEPPELIN-2302] Add info level logs when installing no...

2017-03-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2178 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 this

[GitHub] zeppelin pull request #2163: [ZEPPELIN-2287] Add more test to ensure 'RunOnS...

2017-03-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2163 --- 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 #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2174 @1ambda No worries :) Then how about 1 page ? I mean if there is only 1 page, we can hide `<<` & `>>`. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #2181: [ZEPPELIN-2304] Fix vis Icon sizes in helium page

2017-03-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2181 @1ambda I couldn't auto merge this PR into branch-0.7 since there are some conflicts. If you don't mind, can you create separate PR for branch-0.7? --- If your project is set up for it, you

[GitHub] zeppelin pull request #2181: [ZEPPELIN-2304] Fix vis Icon sizes in helium pa...

2017-03-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2181 --- 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 #2181: [ZEPPELIN-2304] Fix vis Icon sizes in helium page

2017-03-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2181 Thanks. Merging into master and branch-0.7 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

[GitHub] zeppelin issue #2196: fix bugs for timer saver

2017-03-28 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2196 @wary No worry! It's okay! And thanks and Congrats for your contribution. :smile: Let me test it out and give you feedback. --- If your project is set up for it, you can reply to this

[VOTE] Release Apache Zeppelin 0.7.1 (RC3)

2017-03-28 Thread Mina Lee
I propose the following RC to be released for the Apache Zeppelin 0.7.1 release. The commit id is 9eb77adfb3d9969f84d1657b78a95b58e44b5be8 which is corresponds to the tag v0.7.1-rc3: https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;h=refs/tags/v0.7.1-rc3 The release archives (tgz),

[CANCEL][VOTE] Release Apache Zeppelin 0.7.1 (RC2)

2017-03-28 Thread Mina Lee
During the rc2 vote, we've found some critical issues: - Exception not propagated to front-end - Built-in scatter chart throws exception - Editing interpreter setting leaves zombie interpreter process All issue have been resolved and I am going to open rc3 vote right after. Cheers,

[GitHub] zeppelin issue #2195: [ZEPPELIN-2319] new methods for ZeppelinContext

2017-03-28 Thread meniluca
Github user meniluca commented on the issue: https://github.com/apache/zeppelin/pull/2195 @zjffdu I agree in principle, but if you look at the code they are few rather trivial new methods, all of them wrapping up the already existing ``run()``. I have these function running in my

[GitHub] zeppelin pull request #2194: [ZEPPELIN-2229] Providing workaround for nvd3 v...

2017-03-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2194 --- 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 #2194: [ZEPPELIN-2229] Providing workaround for nvd3 voronoi

2017-03-28 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/2194 Thank you for the contribution @Remilito. Merging it into master and branch-0.7. --- 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 #2194: [ZEPPELIN-2229] Providing workaround for nvd3 voronoi

2017-03-28 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2194 I checked the similar issue ["TypeError: Cannot read property '1' of undefined" in the bar chart](https://issues.apache.org/jira/browse/ZEPPELIN-2273) we have. This error is occured when

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

2017-03-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2197 @Leemoonsoo @jongyoul @felixcheung 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

[GitHub] zeppelin issue #2194: [ZEPPELIN-2229] Providing workaround for nvd3 voronoi

2017-03-28 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2194 Tested it out and it works very well! ![z_for_bumpup_check](https://cloud.githubusercontent.com/assets/8110458/24394167/deb1a44e-13d5-11e7-85b7-b1551e9750ab.gif) --- If your project is set

[GitHub] zeppelin issue #1887: [ZEPPELIN-1940] most of eslint rules are NOT applied a...

2017-03-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1887 I will open another PR for this. Too many conflicts 😞 --- 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 pull request #1887: [ZEPPELIN-1940] most of eslint rules are NOT ap...

2017-03-28 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1887 --- 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