[GitHub] zeppelin issue #1668: [ZEPPELIN-1695] Centralize libs, plugin versions in al...

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1668 Sounds reasonable. CI is green, merging to master then. --- 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 #1700: [DOCS] Update Shiro configuration file with new packag...

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1700 Looks good to me, merging to master as a hotfix based on user feedback in [this thread](http://markmail.org/message/4z6hxxf3eem2wgp2), if there is no further discussion. --- If your project is set

[GitHub] zeppelin issue #1696: CI test for 1689

2016-11-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1696 @bzz That's because invalid `_tools/checkstyle.xml`. I'v just updated sorry for distraction :) --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1696: CI test for 1689

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1696 Spark 2.0 profile fails with very stragne ``` [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (checkstyle-fail-build) on project zeppelin:

[GitHub] zeppelin issue #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

2016-11-28 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1441 change name for item on select box. Undefined to Interpreter is not set --- 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 #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

2016-11-28 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1441 --- 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 #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

2016-11-28 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1441 [ ZEPPELIN-967 ] Implement search bar in job manager ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out

[GitHub] zeppelin issue #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

2016-11-28 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1441 @Leemoonsoo Thank you for your review a lot. First, the select box's interpreter shows only the first interpreter, because it is information about the default interpreter's

[GitHub] zeppelin pull request #1700: [DOCS] Update Shiro configuration file with new...

2016-11-28 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/1700 [DOCS] Update Shiro configuration file with new package names ### What is this PR for? Since most of authentication related realm classes has moved from `org.apache.zeppelin.server`

[GitHub] zeppelin pull request #1699: [ZEPPELIN-1724] fix conda version

2016-11-28 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1699 [ZEPPELIN-1724] fix conda version ### What is this PR for? Because `conda run` command removed since version `4.0.9`, PythonCondaInterpreter not working after the `conda-4.0.9`. This

[GitHub] zeppelin issue #1681: [ZEPPELIN-1691] Implement NotebookRepoSetting to Zeppe...

2016-11-28 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1681 @khalidhuseynov thanks for the review, i addressed it. Merging if no more discussion --- If your project is set up for it, you can reply to this email and have your reply appear

Re: 0.7 Shiro LDAP authentication changes? Unable to instantiate class [org.apache.zeppelin.server.LdapGroupRealm]

2016-11-28 Thread Khalid Huseynov
I think during refactoring LdapGroupRealm has moved into different package, so could you try in your shiro config with: ldapRealm = org.apache.zeppelin.realm.LdapGroupRealm On Tue, Nov 29, 2016 at 2:33 AM, Ruslan Dautkhanov wrote: > + dev list > > Could somebody please

[GitHub] zeppelin pull request #1698: Downloading dependency/changing version for an ...

2016-11-28 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1698 Downloading dependency/changing version for an interpreter results in error sometimes ### What is this PR for? At times on downloading dependency/changing version for an interpreter

[GitHub] zeppelin pull request #1697: [ZEPPELIN-1352] Refactor Notebook repo versioni...

2016-11-28 Thread anthonycorbacho
GitHub user anthonycorbacho opened a pull request: https://github.com/apache/zeppelin/pull/1697 [ZEPPELIN-1352] Refactor Notebook repo versioning to return Revision Empty instead of NULL ### What is this PR for? In Zeppelin notebookRepo, versioning (checkpoint, get,

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1658 Thanks @agoodm @khalidhuseynov for the review! Merge to master if there're no more comments. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1679: ZEPPELIN-1707. Pass userName when creating interpreter...

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1679 Looks good to me. CI is green now, so will merge it to master if there is no further discussion. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1339: [ZEPPELIN-1332] Remove spark-dependencies & suggest ne...

2016-11-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1339 Short summary and small thought about #1399 1. Using symlink like `local-spark/master` would be safe i think. It enables user replace his local spark without renaming directories.

[jira] [Created] (ZEPPELIN-1726) Downloading dependency/changing version for an interpreter results in error sometimes

2016-11-28 Thread Kshitij Badani (JIRA)
Kshitij Badani created ZEPPELIN-1726: Summary: Downloading dependency/changing version for an interpreter results in error sometimes Key: ZEPPELIN-1726 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1726

[jira] [Created] (ZEPPELIN-1725) Downloading dependency/changing version for an interpreter results in error sometimes

2016-11-28 Thread Kshitij Badani (JIRA)
Kshitij Badani created ZEPPELIN-1725: Summary: Downloading dependency/changing version for an interpreter results in error sometimes Key: ZEPPELIN-1725 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1725

[jira] [Created] (ZEPPELIN-1724) conda run command removed in 4.1.0

2016-11-28 Thread Shim HyungSung (JIRA)
Shim HyungSung created ZEPPELIN-1724: Summary: conda run command removed in 4.1.0 Key: ZEPPELIN-1724 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1724 Project: Zeppelin Issue

[GitHub] zeppelin pull request #1612: ZEPPELIN-1607. Refactor Livy Interpreter to ada...

2016-11-28 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1612 ZEPPELIN-1607. Refactor Livy Interpreter to adapt scope mode ### What is this PR for? ZEPPELIN-1606 will make scoped mode as the default interpreter mode of livy. This PR is to refactor

[GitHub] zeppelin pull request #1612: ZEPPELIN-1607. Refactor Livy Interpreter to ada...

2016-11-28 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/1612 --- 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 #1690: [ZEPPELIN-1709] docs: Describe how to use checkstyle p...

2016-11-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1690 @1ambda Sounds great thanks! 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

[GitHub] zeppelin pull request #1696: CI test for 1689

2016-11-28 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/1696 CI test for 1689 This is a comparison group for #1689 You can merge this pull request into a Git repository by running: $ git pull https://github.com/1ambda/zeppelin ci-test-for-1689

[GitHub] zeppelin issue #1690: [ZEPPELIN-1709] docs: Describe how to use checkstyle p...

2016-11-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1690 I separated plugin link with setting guide @AhyoungRyu :) --- 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 #1672: ZEPPELIN-1701: User impersonation not work with...

2016-11-28 Thread prabhjyotsingh
GitHub user prabhjyotsingh reopened a pull request: https://github.com/apache/zeppelin/pull/1672 ZEPPELIN-1701: User impersonation not work with SPARK_HOME set. ### What is this PR for? User impersonation of front end user does not work with SPARK_HOME set. ### What

[GitHub] zeppelin pull request #1672: ZEPPELIN-1701: User impersonation not work with...

2016-11-28 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/1672 --- 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 #1672: ZEPPELIN-1701: User impersonation not work with SPARK_...

2016-11-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1672 @zjffdu did you try it again, as this is working for me, or if you can let me know your environment I can try on the same. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #1693: [ZEPPELIN-1695] chore: Set xml plugin phase to verify ...

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1693 Looks great to me, thank you @1ambda Merging to master if there is no further discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1668: [ZEPPELIN-1695] Centralize libs, plugin versions in al...

2016-11-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1668 rebased to keep sync with master (+ to see CI is green as well) --- 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 issue #1686: [ZEPPELIN-1686] Added clear output to keyboard shortcu...

2016-11-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1686 @soralee Tested again with `ctrl + option(alt) + l` and it works well. LGTM 👍 --- 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 #1689: [ZEPPELIN-1709] Update checkstyle rule file, plugin

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1689 Looks great to me, thank you @1ambda ! May be, as soon as CI is green let's try to squash some commits, before merging it in - 37 may be a bit too much to be useful in merge commit message

[GitHub] zeppelin pull request #1684: Fix filter icon not displaying problem

2016-11-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1684 --- 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 #1685: [ZEPPELIN-1714] Build error of scio Intepreter on Cent...

2016-11-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1685 Merge into master 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 project does not have

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-28 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1658 tried locally and works 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 does not have

[GitHub] zeppelin issue #1617: ZEPPELIN-1645: JDBC Interpreter does not commit update...

2016-11-28 Thread matthewLiem
Github user matthewLiem commented on the issue: https://github.com/apache/zeppelin/pull/1617 @randerzander @Leemoonsoo Thanks for this. Do you know of a workaround we could use in the meantime? --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1630: [ZEPPELIN-1629] Enable renaming folder from the main p...

2016-11-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1630 @tae-jun Great work! Take your time and feel free to ping me again when you're ready. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1606: [ZEPPELIN-777] Math formula support

2016-11-28 Thread tae-jun
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/1606 @giaosudau Thanks! 👍 I appreciate 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

[GitHub] zeppelin issue #1630: [ZEPPELIN-1629] Enable renaming folder from the main p...

2016-11-28 Thread tae-jun
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/1630 CI failure is because of me. I will fix 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

[GitHub] zeppelin issue #1606: [ZEPPELIN-777] Math formula support

2016-11-28 Thread giaosudau
Github user giaosudau commented on the issue: https://github.com/apache/zeppelin/pull/1606 @tae-jun https://issues.apache.org/jira/browse/ZEPPELIN-1723 --- 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

[jira] [Created] (ZEPPELIN-1723) Math formula support library path error

2016-11-28 Thread Chanh Le (JIRA)
Chanh Le created ZEPPELIN-1723: -- Summary: Math formula support library path error Key: ZEPPELIN-1723 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1723 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #1660: [WIP][ZEPPELIN-1594] Personalized mode

2016-11-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1660 @Leemoonsoo Yes, I haven't check the front-side. I'll fix 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

[GitHub] zeppelin issue #1660: [WIP][ZEPPELIN-1594] Personalized mode

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1660 @jongyoul I think key requirement of personalized mode is two or more different user want to keep their own dynamic form input and the output of paragraph. How can this be done without let

[GitHub] zeppelin issue #1660: [WIP][ZEPPELIN-1594] Personalized mode

2016-11-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1660 @Leemoonsoo I've not tested the graph. If you mean a dynamic change of graph, I'll check it. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1660: [WIP][ZEPPELIN-1594] Personalized mode

2016-11-28 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1660 @Leemoonsoo As we discussed before, the first version of personalized mode is to avoid changing note simultaneously while two user see a same note. I think this version fits it. one user want to

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-28 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1658 Very nice @Leemoonsoo 👍 I have tested this with the matplotlib tutorial notebook just to see if [ZEPPELIN-1638](https://issues.apache.org/jira/browse/ZEPPELIN-1638) is also resolved

[jira] [Created] (ZEPPELIN-1722) Modernize visualization/transformation using ES6 class

2016-11-28 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1722: -- Summary: Modernize visualization/transformation using ES6 class Key: ZEPPELIN-1722 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1722 Project: Zeppelin

[jira] [Created] (ZEPPELIN-1721) Create / manage settings inside of each visualization.

2016-11-28 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1721: -- Summary: Create / manage settings inside of each visualization. Key: ZEPPELIN-1721 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1721 Project: Zeppelin

[GitHub] zeppelin pull request #1695: Zeppelin 1283

2016-11-28 Thread PetroKostyuk
GitHub user PetroKostyuk opened a pull request: https://github.com/apache/zeppelin/pull/1695 Zeppelin 1283 ### What is this PR for? It's adding option to specify column in scatterPlot, that will be displayed in label ### What type of PR is it? Feature

[GitHub] zeppelin issue #1694: [ZEPPELIN-1718] Prevent anonymous user to set note per...

2016-11-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1694 Haven't tried it yet, but IMO we should have a check at backned for the same (both API and websocket). --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1694: [ZEPPELIN-1718] Prevent anonymous user to set note per...

2016-11-28 Thread prabhu-pictink
Github user prabhu-pictink commented on the issue: https://github.com/apache/zeppelin/pull/1694 Haven't tried it yet, but I think we should have a check at backned for the same (both API and websocket). --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

2016-11-28 Thread epahomov
Github user epahomov commented on the issue: https://github.com/apache/zeppelin/pull/1688 Yeaaa, I love 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 enabled and wishes

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

2016-11-28 Thread krskoondra
Github user krskoondra commented on the issue: https://github.com/apache/zeppelin/pull/1688 OK, @Leemoonsoo, the button is now in the dropdown menu of the paragraph and new keyboard shortcut ctrl+shift+c is added. --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1630: [ZEPPELIN-1629] Enable renaming folder from the main p...

2016-11-28 Thread tae-jun
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/1630 @AhyoungRyu Sorry for my late response! But it's due to tests :) I reproduced the error you mentioned and you were right! Furthermore, chasing a reason of the error, I notice that

[GitHub] zeppelin issue #1637: [ZEPPELIN-1665] Z.run with external note executable an...

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1637 @cloverhearts Thanks for the great contribution. LGTM and merge to master if there're no more discussions. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1637: [ZEPPELIN-1665] Z.run with external note executable an...

2016-11-28 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1637 @Leemoonsoo Sorry, I think I misunderstood your answer. All edits are complete. Thank you for your reply. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1637: [ZEPPELIN-1665] Z.run with external note executable an...

2016-11-28 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1637 Sorry, I think I misunderstood your answer. All edits are complete. Thank you for your reply. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1685: [ZEPPELIN-1714] Build error of scio Intepreter on Cent...

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1685 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 #1685: [ZEPPELIN-1714] Build error of scio Intepreter on Cent...

2016-11-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1685 @astroshim Tested on CentOS 7 again and faced the issue you described. Also I checked the other files that contains `import scala.tools.nsc.*` (e.g.

[GitHub] zeppelin issue #1637: [ZEPPELIN-1665] Z.run with external note executable an...

2016-11-28 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1637 ci retry --- 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

[GitHub] zeppelin pull request #1694: [ZEPPELIN-1718] Prevent anonymous user to set n...

2016-11-28 Thread AhyoungRyu
GitHub user AhyoungRyu reopened a pull request: https://github.com/apache/zeppelin/pull/1694 [ZEPPELIN-1718] Prevent anonymous user to set note permission / interpreter owner ### What is this PR for? Currently anonymous user can set note permission / interpreter's owner like

[GitHub] zeppelin issue #1647: [ZEPPELIN-1013] Don't run paragraph on selectbox chang...

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1647 Although it changes user experience, it is more consistent behavior considering other type of dynamic form behavior (input, checkbox). Regarding the test, while other types of dynamic

[jira] [Created] (ZEPPELIN-1720) Make tests to ensure behavior of dynamic form

2016-11-28 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1720: -- Summary: Make tests to ensure behavior of dynamic form Key: ZEPPELIN-1720 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1720 Project: Zeppelin

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

2016-11-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1688 Thanks for updatding description! Agree with @Leemoonsoo. And FYI, you can refer #1686 for assigning keyboard shortcut :D --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #1684: Fix filter icon not displaying problem

2016-11-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1684 Merge into master 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 project does not have

[GitHub] zeppelin pull request #1694: [ZEPPELIN-1718] Prevent anonymous user to set n...

2016-11-28 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/1694 [ZEPPELIN-1718] Prevent anonymous user to set note permission / interpreter owner ### What is this PR for? Currently anonymous user can set note permission / interpreter's owner like

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1688 Top right corner of each paragraph was pretty crowded in early days of Zeppelin. Run, Move up, Move down, Delete, Show title, Show codes, Hides output, and so on. Because it was

[GitHub] zeppelin issue #1637: [ZEPPELIN-1665] Z.run with external note executable an...

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1637 I still can see some unnecessary changes on the import code block in many files. Could you manage them unchanged? I have listed files in previous comment. --- If your project is set up

[GitHub] zeppelin issue #1686: [ZEPPELIN-1686] Added clear output to keyboard shortcu...

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1686 Ctrl+Alt+L sounds 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 enabled

[GitHub] zeppelin issue #1681: [ZEPPELIN-1691] Implement NotebookRepoSetting to Zeppe...

2016-11-28 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1681 @anthonycorbacho tested and works as expected. may check comment for log, LGTM! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Created] (ZEPPELIN-1719) Error parsing %table when first cell in row is html

2016-11-28 Thread Abhijit Bhole (JIRA)
Abhijit Bhole created ZEPPELIN-1719: --- Summary: Error parsing %table when first cell in row is html Key: ZEPPELIN-1719 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1719 Project: Zeppelin

[GitHub] zeppelin issue #1632: [ZEPPELIN-1639] Add tests with external python depende...

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1632 I think it is mostly the test output that pollutes the CI logs... So, if I read TravisCI logs right - with 2 new profiles `Elapsed time 31 min`, and on `master` i.e

[jira] [Created] (ZEPPELIN-1718) Should prevent to set permission by anonymous user

2016-11-28 Thread Ahyoung Ryu (JIRA)
Ahyoung Ryu created ZEPPELIN-1718: - Summary: Should prevent to set permission by anonymous user Key: ZEPPELIN-1718 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1718 Project: Zeppelin

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

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1689 Thank you @1ambda ! I think it's a great idea to have checkstyle off the most of profiles, but add a new one with just checkstyle. After CI finishes, let's post the CI time statistics in

[GitHub] zeppelin issue #1690: [ZEPPELIN-1709] docs: Describe how to use checkstyle p...

2016-11-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1690 Looks great to me, thank you for taking care of both, Eclipse and IDEA --- 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 #918: ZEPPELIN-885 Add option to send email in scheduler exec...

2016-11-28 Thread vgmartinez
Github user vgmartinez commented on the issue: https://github.com/apache/zeppelin/pull/918 Hi @Leemoonsoo, I have refactored the code to be able to add more channels of notifications, e.g slack, SNS (in the next PR)... Please take a look... thks --- If your project

[jira] [Created] (ZEPPELIN-1717) Weird output for SparkR

2016-11-28 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1717: Summary: Weird output for SparkR Key: ZEPPELIN-1717 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1717 Project: Zeppelin Issue Type: Bug Affects

[GitHub] zeppelin issue #1691: ZEPPELIN-1716. Error of interpreter not found is not p...

2016-11-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1691 @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 feature enabled

[GitHub] zeppelin issue #1693: [ZEPPELIN-1695] chore: Set xml plugin phase to verify ...

2016-11-28 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1693 Thanks @1ambda for the contribution :smile: I tested on Ubuntu and it works well as expected ``` --- T E S T S

[GitHub] zeppelin issue #1686: [ZEPPELIN-1686] Added clear output to keyboard shortcu...

2016-11-28 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1686 Thanks for review @AhyoungRyu @Leemoonsoo, and could you check this PR again, please? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin pull request #1693: [ZEPPELIN-1695] chore: Set xml plugin phase to ...

2016-11-28 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/1693 [ZEPPELIN-1695] chore: Set xml plugin phase to verify (minor) ### What is this PR for? Changed the phase of xml validation plugin from validate to verify so that developers can use `mvn

[GitHub] zeppelin pull request #1691: ZEPPELIN-1716. Error of interpreter not found i...

2016-11-28 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1691 ZEPPELIN-1716. Error of interpreter not found is not propagated to fr… ### What is this PR for? It is introduced in ZEPPELIN-1399, we call `InterpreterFactory.getInterpreter` before

[GitHub] zeppelin pull request #1691: ZEPPELIN-1716. Error of interpreter not found i...

2016-11-28 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/1691 --- 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 #1692: [ZEPPELIN-1710 - WIP] Ace Editor upgrade

2016-11-28 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1692 [ZEPPELIN-1710 - WIP] Ace Editor upgrade ### What is this PR for? The current version of the ace editor is too old. You should also fix a bug that improperly typed non-English