[jira] [Created] (ZEPPELIN-1847) Reduce zeppelin-web build time for html files in dev

2016-12-22 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-1847: --- Summary: Reduce zeppelin-web build time for html files in dev Key: ZEPPELIN-1847 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1847 Project: Zeppelin Iss

[GitHub] zeppelin issue #1789: [MINOR] Optimize watchers of paragraph results

2016-12-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1789 Thanks for the contribution, one minor issue: Before: https://cloud.githubusercontent.com/assets/674497/21419733/7b477450-c84f-11e6-9f06-9623585b6402.png";> After: h

[GitHub] zeppelin pull request #1791: [ZEPPELIN-1847] fix: Copy only html when html f...

2016-12-22 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/1791 [ZEPPELIN-1847] fix: Copy only html when html file changed ### What is this PR for? Reduce build time by avoiding to copy non-related files. Approx 3~4 secs as you can see below.

[GitHub] zeppelin issue #1791: [ZEPPELIN-1847] fix: Copy only html when html file cha...

2016-12-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1791 \cc @cloverhearts @felizbear for review. 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

[GitHub] zeppelin issue #1789: [MINOR] Optimize watchers of paragraph results

2016-12-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1789 Hard to figure out what is different at first. See border of the icons. --- 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 proje

[GitHub] zeppelin issue #1791: [ZEPPELIN-1847] fix: Copy only html when html file cha...

2016-12-22 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1791 @1ambda Thank you for your good fix. Sure, I will :) --- 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 #1791: [ZEPPELIN-1847] fix: Copy only html when html file cha...

2016-12-22 Thread felizbear
Github user felizbear commented on the issue: https://github.com/apache/zeppelin/pull/1791 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 and wishes so, or if the f

[GitHub] zeppelin issue #1791: [ZEPPELIN-1847] fix: Copy only html when html file cha...

2016-12-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1791 one more question. Can we boost dev mode build using webpack and its useful plugins? --- 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 #1791: [ZEPPELIN-1847] fix: Copy only html when html file cha...

2016-12-22 Thread felizbear
Github user felizbear commented on the issue: https://github.com/apache/zeppelin/pull/1791 We totally can but it requires some effort, i.e. migrating code to es6 modules --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[jira] [Created] (ZEPPELIN-1848) ZEPPELIN_NOTEBOOK_S3_KMS_KEY_ID defaults to search keys in us-east-1

2016-12-22 Thread Mikhail (JIRA)
Mikhail created ZEPPELIN-1848: - Summary: ZEPPELIN_NOTEBOOK_S3_KMS_KEY_ID defaults to search keys in us-east-1 Key: ZEPPELIN-1848 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1848 Project: Zeppelin

[GitHub] zeppelin pull request #1792: Change gitignore for font files

2016-12-22 Thread marchpig
GitHub user marchpig opened a pull request: https://github.com/apache/zeppelin/pull/1792 Change gitignore for font files ### What is this PR for? Some web fonts were changed by 5552319378e8361a16c91df2ba3cdf5519b74d58 but `.gitignore` was not changed accordingly. ### Wh

[GitHub] zeppelin pull request #1793: [MINOR] Refactoring Job and Paragraph

2016-12-22 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1793 [MINOR] Refactoring Job and Paragraph ### What is this PR for? Job class has only two sub classes. I don't think we need Job abstraction class anymore, then I'll move all function of Job int

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

2016-12-22 Thread tae-jun
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/1730 @AhyoungRyu Always thanks for the review :) Also thanks for your understanding. @1ambda Thanks for the review! I appreciate it and sorry for my late response. That was a really nice

[GitHub] zeppelin issue #1561: [Zeppelin 914]- Apply new mechanism to IgniteInterpret...

2016-12-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1561 @meenakshisekar Could you rebase 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 have this featur

[GitHub] zeppelin pull request #1759: [ZEPPELIN-922] New interpreter registration mec...

2016-12-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1759 --- 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

[jira] [Created] (ZEPPELIN-1849) Web interface does not render information properly

2016-12-22 Thread Sergey_Sokur (JIRA)
Sergey_Sokur created ZEPPELIN-1849: -- Summary: Web interface does not render information properly Key: ZEPPELIN-1849 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1849 Project: Zeppelin

[GitHub] zeppelin issue #1790: [ZEPPELIN-1845] InterpreterFactory.createOrGetInterpre...

2016-12-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1790 CI Green, ready for 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 enable

[GitHub] zeppelin issue #1789: [MINOR] Optimize watchers of paragraph results

2016-12-22 Thread cuspymd
Github user cuspymd commented on the issue: https://github.com/apache/zeppelin/pull/1789 In the process of modifying the above issue, I divided the buttons of the chart selector into two groups. The role of the two button groups seems to be slightly different. - First grou

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

2016-12-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1730 Hi @tae-jun It's good idea to introduce limitation when notes are in trash. Let me comment (1) what you mentioned is about restricting modification of archived notes (i mean, not

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

2016-12-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1751 --- 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 #1791: [ZEPPELIN-1847] fix: Copy only html when html file cha...

2016-12-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1791 Did simple test for applying webpack. Let me share result in short. - (**IMPORTANT**) It is impossible to handle in a single PR. It requires changing directory structure and this causes soo

[jira] [Created] (ZEPPELIN-1850) Improve frontend build structure, tools

2016-12-22 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-1850: --- Summary: Improve frontend build structure, tools Key: ZEPPELIN-1850 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1850 Project: Zeppelin Issue Type: Impr

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

2016-12-22 Thread tae-jun
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/1730 @1ambda Thanks for your prompt response! I see what you mean now. Thanks for the explanation :) Now, only `remove permanently` and `restore` buttons are shown. https://cloud.githubu

[GitHub] zeppelin issue #1793: [WIP][MINOR] Refactoring Job and Paragraph

2016-12-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1793 ``` Failed tests: NotebookTest.testAbortParagraphStatusOnInterpreterRestart:760 expected: but was: ``` This test passed in my local --- If your project is set up for it,

[GitHub] zeppelin pull request #1730: [ZEPPELIN-1736] Introduce trash & enable removi...

2016-12-22 Thread tae-jun
GitHub user tae-jun reopened a pull request: https://github.com/apache/zeppelin/pull/1730 [ZEPPELIN-1736] Introduce trash & enable removing folder ### What is this PR for? ![image](https://cloud.githubusercontent.com/assets/8201019/20925378/03a722f8-bbfa-11e6-9afc-c62cfe6b0e2a.p

[GitHub] zeppelin pull request #1730: [ZEPPELIN-1736] Introduce trash & enable removi...

2016-12-22 Thread tae-jun
Github user tae-jun closed the pull request at: https://github.com/apache/zeppelin/pull/1730 --- 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 #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-12-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1503 @gss2002 Livy interpreter has been refactored, this issue has been fixed. Mind to close it ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1729: [ZEPPELIN-1758] support livy pyspark Interpreter's mag...

2016-12-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1729 @purechoc refactoring LivyHelper has been done. You can continue this PR. I think the right place for you is https://github.com/apache/zeppelin/blob/master/livy/src/main/java/org/apache/zeppelin/li

[GitHub] zeppelin pull request #1730: [ZEPPELIN-1736] Introduce trash & enable removi...

2016-12-22 Thread tae-jun
Github user tae-jun closed the pull request at: https://github.com/apache/zeppelin/pull/1730 --- 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 #1730: [ZEPPELIN-1736] Introduce trash & enable removi...

2016-12-22 Thread tae-jun
GitHub user tae-jun reopened a pull request: https://github.com/apache/zeppelin/pull/1730 [ZEPPELIN-1736] Introduce trash & enable removing folder ### What is this PR for? ![image](https://cloud.githubusercontent.com/assets/8201019/20925378/03a722f8-bbfa-11e6-9afc-c62cfe6b0e2a.p

[GitHub] zeppelin pull request #1730: [ZEPPELIN-1736] Introduce trash & enable removi...

2016-12-22 Thread tae-jun
Github user tae-jun closed the pull request at: https://github.com/apache/zeppelin/pull/1730 --- 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 #1730: [ZEPPELIN-1736] Introduce trash & enable removi...

2016-12-22 Thread tae-jun
GitHub user tae-jun reopened a pull request: https://github.com/apache/zeppelin/pull/1730 [ZEPPELIN-1736] Introduce trash & enable removing folder ### What is this PR for? ![image](https://cloud.githubusercontent.com/assets/8201019/20925378/03a722f8-bbfa-11e6-9afc-c62cfe6b0e2a.p

[jira] [Created] (ZEPPELIN-1851) LazyOpenInterpreter would open interpreter multiple times when open fails

2016-12-22 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1851: Summary: LazyOpenInterpreter would open interpreter multiple times when open fails Key: ZEPPELIN-1851 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1851 Project

[GitHub] zeppelin pull request #1795: ZEPPELIN-1851. LazyOpenInterpreter would open i...

2016-12-22 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1795 ZEPPELIN-1851. LazyOpenInterpreter would open interpreter multiple times when open fails ### What is this PR for? Change `opened` to be `volatile` as it would be accessed by multiple thr

[GitHub] zeppelin issue #1795: ZEPPELIN-1851. LazyOpenInterpreter would open interpre...

2016-12-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1795 @Leemoonsoo @jongyoul Please help review it. 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 th

[jira] [Created] (ZEPPELIN-1852) Use multiple InterpreterResult for displaying appInfo

2016-12-22 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1852: Summary: Use multiple InterpreterResult for displaying appInfo Key: ZEPPELIN-1852 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1852 Project: Zeppelin

[GitHub] zeppelin pull request #1796: [WIP] ZEPPELIN-1852. Use multiple InterpreterRe...

2016-12-22 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1796 [WIP] ZEPPELIN-1852. Use multiple InterpreterResult for displaying appInfo ### What is this PR for? Refactor the livy interpreter to use multiple `InterpreterResult` for displaying appInfo.

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

2016-12-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1780 --- 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 #1729: [ZEPPELIN-1758] support livy pyspark Interpreter's mag...

2016-12-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1729 @purechoc Thanks for the quick update. Could you add test in `LivyInterpreterIT` ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

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

2016-12-22 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1775 @1ambda actually i've tried to reproduce [ZEPPELIN-1737](https://issues.apache.org/jira/browse/ZEPPELIN-1737) and even with git 2.11.0 wasn't able to do it. also from documentation [here](

[GitHub] zeppelin pull request #1775: [ZEPPELIN-1745] View revisions in non-editable ...

2016-12-22 Thread khalidhuseynov
GitHub user khalidhuseynov reopened a pull request: https://github.com/apache/zeppelin/pull/1775 [ZEPPELIN-1745] View revisions in non-editable mode ### What is this PR for? This is to make view of revisions non-editable ### What type of PR is it? Improvement

[GitHub] zeppelin pull request #1775: [ZEPPELIN-1745] View revisions in non-editable ...

2016-12-22 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at: https://github.com/apache/zeppelin/pull/1775 --- 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 feat

[jira] [Created] (ZEPPELIN-1853) Add README for livy interpreter

2016-12-22 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1853: Summary: Add README for livy interpreter Key: ZEPPELIN-1853 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1853 Project: Zeppelin Issue Type: Improvemen

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

2016-12-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1775 FYI - you can install 2.11.0 using brew. @khalidhuseynov is right, that should be handled in the separate issue. Let's talk about it on ZEPPELIN-1737. --- If your project is set up for i

[GitHub] zeppelin issue #1777: [ZEPPELIN-1831] Fix paragraph control dropdown alignme...

2016-12-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1777 @marchpig can you resolve conflicting file ? --- 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 th

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

2016-12-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1785 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

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

2016-12-22 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1721 @anthonycorbacho change finished. ![asc](https://cloud.githubusercontent.com/assets/10525473/21448241/a177ac24-c923-11e6-91ce-abef42bb0ea8.png) ![desc](https://cloud.githubuserco

[GitHub] zeppelin issue #1792: Change gitignore for font files

2016-12-22 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1792 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 th

[jira] [Created] (ZEPPELIN-1854) "Per User" mode in interpreter setting should be visible even if there is no role assigned to a user

2016-12-22 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-1854: - Summary: "Per User" mode in interpreter setting should be visible even if there is no role assigned to a user Key: ZEPPELIN-1854 URL: https://issues.apache.org/jira/brow

[GitHub] zeppelin pull request #1797: [ZEPPELIN-1854] make Per User mode in interpret...

2016-12-22 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1797 [ZEPPELIN-1854] make Per User mode in interpreter setting visible even if there is no role assigned to a user ### What is this PR for? There can be a case where user may have enabled s

[GitHub] zeppelin issue #1792: Change gitignore for font files

2016-12-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1792 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

[GitHub] zeppelin issue #1798: ZEPPELIN-1853. Add README for livy interpreter

2016-12-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1798 @AhyoungRyu Please help review it. 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

[GitHub] zeppelin pull request #1798: ZEPPELIN-1853. Add README for livy interpreter

2016-12-22 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1798 ZEPPELIN-1853. Add README for livy interpreter ### What is this PR for? Add `README` file for developer, besides update the `livy.md` for the recent changes on livy interpreter.

[GitHub] zeppelin issue #1797: [ZEPPELIN-1854] make Per User mode in interpreter sett...

2016-12-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1797 @jongyoul can you help review this one. --- 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 #1798: ZEPPELIN-1853. Add README for livy interpreter

2016-12-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1798 +1 for having more doc. --- 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 #1791: [ZEPPELIN-1847] fix: Copy only html when html file cha...

2016-12-22 Thread felizbear
Github user felizbear commented on the issue: https://github.com/apache/zeppelin/pull/1791 @1ambda thank you for the research. A couple of questions: 1. Why do you think it requires changing dir structure? 2. What do you mean by: "we need index.html first."? --- If your projec

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-12-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1447 @purechoc Do you want to rebase it and continue work on 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

[GitHub] zeppelin pull request #1799: [ZEPPELIN-1165 : WIP] Code-based job workflow

2016-12-22 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1799 [ZEPPELIN-1165 : WIP] Code-based job workflow ### What is this PR for? Code based workflow (**work in progress**) Re-implementation on this pr https://github.com/apache/

[GitHub] zeppelin issue #1176: [Zeppelin 1165 : improve] Paragraph dynamic relationsh...

2016-12-22 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1176 https://github.com/apache/zeppelin/pull/1799 I was create new pr. and re-implementation codebased. --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin pull request #1176: [Zeppelin 1165 : improve] Paragraph dynamic rel...

2016-12-22 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1176 --- 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