[GitHub] zeppelin issue #1861: ZEPPELIN-1293. Livy Interpreter: Automatically attach ...

2017-01-12 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1861 merging if no more comment --- 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 an

[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue: https://github.com/apache/zeppelin/pull/1891 @Leemoonsoo @jongyoul Any comments for this PR ? --- 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 ha

[GitHub] zeppelin pull request #1898: [TBD] set cron job executor to current user by ...

2017-01-12 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/1898 [TBD] set cron job executor to current user by default ### What is this PR for? currently when executing cron job, backend doesn't know who initiated cron job (with subsequent save) u

[GitHub] zeppelin issue #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-12 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1879 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 #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-12 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1879 --- 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 featur

[GitHub] zeppelin pull request #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-12 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1879 [HOTFIX : ZEPPELIN-1932] paragraph blur error ### What is this PR for? When one or more hidden editors are present, clicking on the editor will cause a blur error. This means that

[GitHub] zeppelin pull request #1896: implementation test case for connect to exists ...

2017-01-12 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1896 implementation test case for connect to exists interpreter process ### What is this PR for? We need test case for remote interpreter process ( connect to exists process) ### W

[GitHub] zeppelin pull request #1896: implementation test case for connect to exists ...

2017-01-12 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1896 --- 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 featur

[GitHub] zeppelin issue #1892: [MINOR] broken logo url in web dev mode

2017-01-12 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1892 Thanks for review and considering the side effect @minahlee :+1: You're right! I missed this option. As your mention, I checked `ZEPPELIN_SERVER_CONTEXT_PATH` option. **Test

[GitHub] zeppelin issue #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-12 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1879 @cloverhearts update 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 e

[GitHub] zeppelin pull request #1897: Can't remove interpreter repository.

2017-01-12 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1897 Can't remove interpreter repository. ### What is this PR for? It can't be delete the `interpreter repository`(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/manual/dependencymanagement.html

[GitHub] zeppelin issue #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-12 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1879 fixed --- 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 t

[GitHub] zeppelin issue #1890: [BUILD] Update release script

2017-01-12 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1890 @jongyoul you are right, `-Pr` does nothing in `create_release.sh`, so I removed it in a97b5fe. I kept `-Pr` in `publish_release.sh` though for the one who uses zeppelin-zrinterpreter_2.1x.jar in

[GitHub] zeppelin issue #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-12 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1879 @minahlee Okay, I will take 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 this feature en

[GitHub] zeppelin pull request #1896: implementation test case for connect to exists ...

2017-01-12 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1896 implementation test case for connect to exists interpreter process ### What is this PR for? We need test case for remote interpreter process ( connect to exists process) ### Wha

[GitHub] zeppelin pull request #1884: [ZEPPELIN-1936] Fix user impersonation setting ...

2017-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1884 --- 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 #1858: [ZEPPELIN-1229] Clear browser cache with versio...

2017-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1858 --- 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 #1873: [ZEPPELIN-1922] Exclude jackson-core and jackson-datab...

2017-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1873 @epahomov Thanks for the contribution. I'm trying to understand what cause this problem, because `zeppelin-zengine` shouldn't be a dependency of any interpreter and `/lib` directory su

Re: branch-0.7 cut

2017-01-12 Thread Ashwin Shankar
Cool, thanks! On Thu, Jan 12, 2017 at 1:13 AM, Mina Lee wrote: > Hi Ashwin, > > we still have some issues that need to be handled before release, > hopefully most of them are in review process which will take 2-3 more days. > I am going to start release vote next week, so I expect we will be abl

[GitHub] zeppelin issue #1831: [ZEPPELIN-1883] Can't import spark submitted packages ...

2017-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1831 Thanks @1ambda for the fix. Thanks @zjffdu for reviewing and verifying. It looks good to me, while it provides more consistent behavior between `PysparkInterpreter` and `SPARK_HOME/bin/

[GitHub] zeppelin pull request #1895: [HOTFIX] Improve stability of sync when get fai...

2017-01-12 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/1895 [HOTFIX] Improve stability of sync when get fails ### What is this PR for? This is to improve the stability of sync mechanism when `get` from some backend storage fails (e.g. corrupt f

[GitHub] zeppelin issue #1842: [ZEPPELIN-1619] Load js package as a plugin visualizat...

2017-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1842 @AhyoungRyu Thanks for the review. Great to see High chart working nicely. A lot more improvements on Helium menu will be required, like search page, grouping packages by type, package

[GitHub] zeppelin issue #1868: [ZEPPELIN-1917] Improve python.conda interpreter

2017-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1868 LGTM and merge to 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 thi

[GitHub] zeppelin issue #1894: [ZEPPELIN-1560] avoid generating minus sign in package...

2017-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1894 @robbinspg Great fix! CI failure looks not related. LGTM and merge to master, branch-0.7, branch-0.6 if no more comments! --- If your project is set up for it, you can reply to thi

[GitHub] zeppelin issue #1842: [ZEPPELIN-1619] Load js package as a plugin visualizat...

2017-01-12 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1842 @Leemoonsoo Thanks! I just suggested to have some default message like other pages (e.g. localhost:8080/#/interpreter -> there is "Manage interpreters settings. You can create / edit / remove s

[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue: https://github.com/apache/zeppelin/pull/1891 Flaky test may be another concurrency issue in scheduler execute jobs. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] zeppelin pull request #1894: [ZEPPELIN-1560] avoid generating minus sign in ...

2017-01-12 Thread robbinspg
GitHub user robbinspg opened a pull request: https://github.com/apache/zeppelin/pull/1894 [ZEPPELIN-1560] avoid generating minus sign in package name ### What is this PR for? using Object.hashCode() as part of the REPL wrapper class name can cause a compilation error as hashCode

[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue: https://github.com/apache/zeppelin/pull/1891 I'm not sure, this PR is intent to fix the dead lock 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

[GitHub] zeppelin issue #1890: [BUILD] Update release script

2017-01-12 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1890 @minahlee I think `-Pr` doesn't mean anything because users cannot run R interpreter since `-Psparkr` is enabled. How about removing `-Pr` in a script? --- If your project is set up for it, you

[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1891 Does this PR resolve flaky test that current master encounters? --- 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 doe

FOSDEM 2017 Open Source Conference - Brussels

2017-01-12 Thread Sharan F
Hello Everyone This email is to tell you about ASF participation at FOSDEM. The event will be held in Brussels on 4^th & 5^th February 2017 and we are hoping that many people from our ASF projects will be there. https://fosdem.org/2017/ Attending FOSDEM is completely free and the ASF will ag

[GitHub] zeppelin issue #1892: [MINOR] broken logo url in web dev mode

2017-01-12 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1892 This fix will bring same issue in production mode if user set `ZEPPELIN_SERVER_CONTEXT_PATH`. Do you have any other idea to work around? --- If your project is set up for it, you can reply to th

[GitHub] zeppelin pull request #1889: [MINOR] Rename Pig tutorial note to consider pr...

2017-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1889 --- 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 #1889: [MINOR] Rename Pig tutorial note to consider priority

2017-01-12 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1889 Yup merge into both 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 well. If your

Re: branch-0.7 cut

2017-01-12 Thread Mina Lee
Hi Ashwin, we still have some issues that need to be handled before release, hopefully most of them are in review process which will take 2-3 more days. I am going to start release vote next week, so I expect we will be able to deliver Apache Zeppelin 0.7.0 end of next week or a week later. On Th

[GitHub] zeppelin pull request #1892: [MINOR] broken logo url in web dev mode

2017-01-12 Thread soralee
GitHub user soralee opened a pull request: https://github.com/apache/zeppelin/pull/1892 [MINOR] broken logo url in web dev mode ### What is this PR for? zeppelin logo image (zepLogo.png) is broken in `web dev` mode like below screenshot. ![screenshot from 2017-01-12 14-50-3

[GitHub] zeppelin pull request #1893: [ZEPPELIN-451] Save codes and messages as multi...

2017-01-12 Thread sixmen
GitHub user sixmen opened a pull request: https://github.com/apache/zeppelin/pull/1893 [ZEPPELIN-451] Save codes and messages as multi-line ### What is this PR for? Save Paragraph texts and result msgs as array of strings instead of one long string. One long strings are not

[GitHub] zeppelin pull request #1494: [ZEPPELIN-451] Save codes and messages as multi...

2017-01-12 Thread sixmen
Github user sixmen closed the pull request at: https://github.com/apache/zeppelin/pull/1494 --- 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 #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue: https://github.com/apache/zeppelin/pull/1891 It seems CI has a network issue ``` [ERROR] Error requesting archive. [ERROR] Status: 503 [ERROR] Request options: { [ERROR] "uri": "https://github.com/Medium/phantomj

[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue: https://github.com/apache/zeppelin/pull/1891 we already a test case for this https://github.com/apache/zeppelin/blob/9e75f5b50ac1573d423fa937b801df77d6a36f3c/zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NotebookTest.jav