[GitHub] zeppelin pull request #1765: [WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTe...

2016-12-19 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/1765 --- 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 #1765: [WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTe...

2016-12-19 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1765 [WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .pySpark* fail the build ### What is this PR for? Add more logging as I could not reproduce it locally. ### What type of PR is

[GitHub] zeppelin pull request #1782: [ZEPPELIN-1837] Fix possible reason of DepInter...

2016-12-19 Thread DrIgor
GitHub user DrIgor reopened a pull request: https://github.com/apache/zeppelin/pull/1782 [ZEPPELIN-1837] Fix possible reason of DepInterpreterTest failure ### What is this PR for? Fix possible reason of DepInterpreterTest failure ### What type of PR is it? [Bug Fix]

[GitHub] zeppelin pull request #1782: [ZEPPELIN-1837] Fix possible reason of DepInter...

2016-12-19 Thread DrIgor
Github user DrIgor closed the pull request at: https://github.com/apache/zeppelin/pull/1782 --- 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 #1775: [ZEPPELIN-1745] View revisions in non-editable mode

2016-12-19 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1775 Glad to help reviewing this PR :) Let me check and then give you feedbacks :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin pull request #1784: [ZEPPELIN-1840] Allow fully qualified username ...

2016-12-19 Thread vrathor-hwx
GitHub user vrathor-hwx reopened a pull request: https://github.com/apache/zeppelin/pull/1784 [ZEPPELIN-1840] Allow fully qualified username when principalSuffix is used ### What is this PR for? When principalSuffix is defined in shiro.ini, only the short username are allowed an

[GitHub] zeppelin pull request #1784: [ZEPPELIN-1840] Allow fully qualified username ...

2016-12-19 Thread vrathor-hwx
Github user vrathor-hwx closed the pull request at: https://github.com/apache/zeppelin/pull/1784 --- 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 #1753: [ZEPPELIN-1594] Support personalized mode

2016-12-19 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1753 @Leemoonsoo @minahlee Review it, please. --- 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

Reg:: Usage of javascript in Helium application

2016-12-19 Thread Rajarajan Ganesan
Hi all, I am developing a helium application for our team's own purpose. I would like put some values in context and access it across javascript that is used as frontend . Please let me know if anyone can help me on this front. Regards, *Rajarajan*

[GitHub] zeppelin issue #1775: [ZEPPELIN-1745] View revisions in non-editable mode

2016-12-19 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1775 @bzz it's ready now, also /cc @cloverhearts @1ambda to 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 pr

[GitHub] zeppelin issue #1751: ZEPPELIN-1786. Refactor LivyHelper

2016-12-19 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1751 Just a minor recommendation, rest 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 fe

[GitHub] zeppelin issue #1721: [ZEPPELIN-969] order by note name in job menu

2016-12-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1721 @cloverhearts Looks good 👍 --- 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

[GitHub] zeppelin issue #1721: [ZEPPELIN-969] order by note name in job menu

2016-12-19 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1721 @anthonycorbacho @AhyoungRyu could you please review 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

[GitHub] zeppelin issue #1721: [ZEPPELIN-969] order by note name in job menu

2016-12-19 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1721 PING 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 enabled an

[jira] [Created] (ZEPPELIN-1841) Some of note shortcut key is not working on Ubuntu

2016-12-19 Thread Sora Lee (JIRA)
Sora Lee created ZEPPELIN-1841: -- Summary: Some of note shortcut key is not working on Ubuntu Key: ZEPPELIN-1841 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1841 Project: Zeppelin Issue T

[GitHub] zeppelin issue #1736: [ZEPPELIN-1757] Menu of paragraph includes keyboard sh...

2016-12-19 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1736 I updated description of PR as like below. - add shortcut key to tooltip (`cancel to run paragraph`, `toggle editor`, `toggle output`) - add shortcut key to `Link this paragraph` in paragra

[GitHub] zeppelin issue #1785: [ZEPPELIN-1833] Fix bug in ClientFactory.destroyObject...

2016-12-19 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1785 Thanks @prasadwagle. 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 wi

[GitHub] zeppelin pull request #1785: [ZEPPELIN-1833] Fix bug in ClientFactory.destro...

2016-12-19 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/zeppelin/pull/1785 [ZEPPELIN-1833] Fix bug in ClientFactory.destroyObject that does not close socket connections ### What is this PR for? Fix bug in ClientFactory.destroyObject that does not close socket c

[GitHub] zeppelin pull request #1765: [WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTe...

2016-12-19 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1765 [WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .pySpark* fail the build ### What is this PR for? Add more logging as I could not reproduce it locally. ### What type of PR is

[GitHub] zeppelin pull request #1765: [WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTe...

2016-12-19 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/1765 --- 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 #1751: ZEPPELIN-1786. Refactor LivyHelper

2016-12-19 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1751 @felixcheung The new bug is fixed. CI fail is not relevant. ``` 04:42:36,519 ERROR org.apache.zeppelin.search.LuceneSearch:120 - Failed to open index dir RAMDirectory@615184ee lockFact

[GitHub] zeppelin issue #1730: [ZEPPELIN-1736] Introduce trash & enable removing fold...

2016-12-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1730 @tae-jun Thanks for your precise explanation. Yeah it makes sense. And tested again and it looks good for what we talked about :) It'll be nice to be merged after addressing @1ambda's

[GitHub] zeppelin issue #1738: Strings should not be concatenated using '+' in a loop

2016-12-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1738 @bitchelov Did you have a chance to look into [this comment](https://issues.apache.org/jira/browse/ZEPPELIN-1839?focusedCommentId=15761118&page=com.atlassian.jira.plugin.system.issuetabpanels:co

[GitHub] zeppelin pull request #1755: [ZEPPELIN-1801] To force end table - append %te...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1755 --- 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 #1779: [ZEPPELIN-1835] Upgrade ElasticSearch to versio...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1779 --- 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 #1783: Zeppelin 727

2016-12-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1783 @PetroKostyuk Thanks for your contribution! It's really useful feature indeed. For the quick review, it would be better to edit PR title to sth like "[ZEPPELIN-727] Enable to jump to speci

[GitHub] zeppelin pull request #1750: [ZEPPELIN-1796] Add 'copy to clipboard' next to...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1750 --- 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 #1775: [ZEPPELIN-1745] View revisions in non-editable mode

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1775 @khalidhuseynov Is it WIP? Then please feel free to add WIP in PR title and ping back, when it's ready for reviews. --- If your project is set up for it, you can reply to this email and hav

[GitHub] zeppelin issue #1767: ZEPPELIN-1817 - Added documentation to Zeppelin's shir...

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1767 Great improvement, thank you @vinayshukla ! But indeed, it would be really nice to have documentation available on the website as well, as @Leemoonsoo suggests. --- If your project is set up for

[GitHub] zeppelin issue #1751: ZEPPELIN-1786. Refactor LivyHelper

2016-12-19 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1751 Please hold on the merge. I find another bug in livy interpreter. --- 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

[GitHub] zeppelin issue #1776: [ZEPPELIN-1824] Add MetaData exploration to JDBC Inter...

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1776 Thank you @pmccaffrey6 , it looks great to me, modulo few things noted above. Could you just double check that all feedback from reviews was addressed? --- If your project is set up for it,

[GitHub] zeppelin issue #1779: [ZEPPELIN-1835] Upgrade ElasticSearch to version 2.4.3

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1779 Looks good to me. Just FYI - there already is #1598 from @1ambda but AFAIK it is stuck and need a hand. That would be awesome, if we could come up with a plan to merge that one as well..

[GitHub] zeppelin issue #1780: [BugFix] Tutorial note json format.

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1780 Looks great to me. CI fails here though. I know it's annoying but @astroshim, as PR author, could you please post failure build logs here, so we can see if it is something that already have a

[GitHub] zeppelin issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1758 Is there any other feedback, or was everything addressed and we shall merge it now? Saw @zjffdu 's > How about refer this note in flink.md ? @AlexanderShoshin do you think it's w

[GitHub] zeppelin issue #1689: [ZEPPELIN-1709] Update checkstyle rule file, plugin

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1689 Only the last CI profile fails on very strange curcumstances: ``` 16/12/19 04:54:08 INFO ApplicationMaster: Deleting staging directory .sparkStaging/application_1482123134765_0003 16/12/19

Re: Re: Build error on windows

2016-12-19 Thread Alexander Bezzubov
Thanks for verification and a quick fix, it has been merged to master. -- Alex On Mon, Dec 19, 2016 at 3:39 PM, Jeff Zhang wrote: > Thanks, This PR resolve my issue. > > > > zjp_j...@163.com 于2016年12月19日周一 上午10:21写道: > > > I think the following issue can help u. > > - JIRA : https://issues.apac

[GitHub] zeppelin pull request #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1731 --- 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 #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl Grunt ...

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1731 Thank you very much @cuspymd for fix and @zjffdu for double-checking! --- 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 do

[GitHub] zeppelin issue #1600: Using HDFS to backup and restore notebook

2016-12-19 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1600 I see! This looks like something failing on the TravisCI side though. Could you please rebase this branch on latest master and force-push it here again? This will trigger CI \w latest code\fixes f

[GitHub] zeppelin pull request #1784: [ZEPPELIN-1840] Allow fully qualified username ...

2016-12-19 Thread vrathor-hwx
GitHub user vrathor-hwx opened a pull request: https://github.com/apache/zeppelin/pull/1784 [ZEPPELIN-1840] Allow fully qualified username when principalSuffix is used ### What is this PR for? When principalSuffix is defined in shiro.ini, only the short username are allowed and

[jira] [Created] (ZEPPELIN-1840) Fully qualified username can not be used for login when principalSuffix is defined

2016-12-19 Thread Vipin Rathor (JIRA)
Vipin Rathor created ZEPPELIN-1840: -- Summary: Fully qualified username can not be used for login when principalSuffix is defined Key: ZEPPELIN-1840 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1840

[GitHub] zeppelin issue #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl Grunt ...

2016-12-19 Thread cuspymd
Github user cuspymd commented on the issue: https://github.com/apache/zeppelin/pull/1731 @bzz I've assigned [ZEPPELIN-1239](https://issues.apache.org/jira/browse/ZEPPELIN-1239) to me. Thanks. --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] zeppelin pull request #1781: Followup for ZEPPELIN-1785

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1781 --- 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 #1781: Followup for ZEPPELIN-1785

2016-12-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1781 Thanks @zjffdu for this fix. LGTM and i'm merging this guy to master as a Hotfix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] zeppelin pull request #1783: Zeppelin 727

2016-12-19 Thread PetroKostyuk
GitHub user PetroKostyuk opened a pull request: https://github.com/apache/zeppelin/pull/1783 Zeppelin 727 ### What is this PR for? This PR adds button that allows user to copy link to paragraph in context. Then if user follows this link, it will load note and scroll to selected

Re: Group authentication LDAP

2016-12-19 Thread Vinay Shukla
Can you clarify what you mean by group based authentication? Zeppelin supports LDAP authentication and group based access control. Do you mean the latter? On Thu, Dec 15, 2016 at 12:47 AM, Nihar Khuntia wrote: > I want to do group based authentication. But not able to do it. > > Can you please

[GitHub] zeppelin issue #1746: Collection.isEmpty() should be used to test for emptin...

2016-12-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1746 LGTM and merge if there're 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 have

[GitHub] zeppelin issue #1738: Strings should not be concatenated using '+' in a loop

2016-12-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1738 LGTM. @bitchelov Could you let me know your JIRA id ? so i can make you as a assignee of issue ZEPPELIN-1839 --- If your project is set up for it, you can reply to this email and have yo

[GitHub] zeppelin pull request #1762: [ZEPPELIN-1810] Removed incorrect usage of getS...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1762 --- 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 #1782: [ZEPPELIN-1837] Fix possible reason of DepInterpreterT...

2016-12-19 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1782 Thanks for fixing a flaky test :) It would be great if you accumulate CI multiple results like #1667 for review. Just closing and re-opening will also works instead of pushing the ammende

[GitHub] zeppelin pull request #1778: ZEPPELIN-1834 Deadlock in Zeppelin when running...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1778 --- 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 #1769: [ZEPPELIN-1818] Absolute local repo path in dep...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1769 --- 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 #1764: ZEPPELIN-1816. Implement getProgress in SparkRI...

2016-12-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1764 --- 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 #1694: [ZEPPELIN-1718] Prevent anonymous user to set note per...

2016-12-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1694 @prabhjyotsingh @cloverhearts @astroshim Sorry for my late response. I added checking anonymous user \w notebook rest api. So if anonymous users try to click lock icon / `GET` note permissi

[GitHub] zeppelin issue #1764: ZEPPELIN-1816. Implement getProgress in SparkRInterpre...

2016-12-19 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1764 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 fe

[GitHub] zeppelin issue #1600: Using HDFS to backup and restore notebook

2016-12-19 Thread aspen01
Github user aspen01 commented on the issue: https://github.com/apache/zeppelin/pull/1600 @bzz Thank you for advice. But the build is failed even though I fixed license and checkstyle. --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[jira] [Created] (ZEPPELIN-1839) Code refactoring

2016-12-19 Thread Sergey Sokur (JIRA)
Sergey Sokur created ZEPPELIN-1839: -- Summary: Code refactoring Key: ZEPPELIN-1839 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1839 Project: Zeppelin Issue Type: Improvement

[GitHub] zeppelin pull request #1782: [ZEPPELIN-1837] Fix possible reason of DepInter...

2016-12-19 Thread DrIgor
GitHub user DrIgor opened a pull request: https://github.com/apache/zeppelin/pull/1782 [ZEPPELIN-1837] Fix possible reason of DepInterpreterTest failure ### What is this PR for? Fix possible reason of DepInterpreterTest failure ### What type of PR is it? [Bug Fix]

[jira] [Created] (ZEPPELIN-1838) DepInterpreterTest.testDefault

2016-12-19 Thread Igor Drozdov (JIRA)
Igor Drozdov created ZEPPELIN-1838: -- Summary: DepInterpreterTest.testDefault Key: ZEPPELIN-1838 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1838 Project: Zeppelin Issue Type: Sub-tas

[jira] [Created] (ZEPPELIN-1837) DepInterpreterTest.testDefault

2016-12-19 Thread Igor Drozdov (JIRA)
Igor Drozdov created ZEPPELIN-1837: -- Summary: DepInterpreterTest.testDefault Key: ZEPPELIN-1837 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1837 Project: Zeppelin Issue Type: Sub-tas