[GitHub] zeppelin pull request #1817: remove usage of non-standard string method

2016-12-29 Thread felizbear
GitHub user felizbear opened a pull request: https://github.com/apache/zeppelin/pull/1817 remove usage of non-standard string method ### What is this PR for? This PR removes a non-standard string prototype method `contains` that can cause potential bugs in the future maintenance

[GitHub] zeppelin issue #1804: [ZEPPELIN-1578] notes list in customizing zeppelin hom...

2016-12-29 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1804 @AhyoungRyu Thanks for quick information! I just resolved it :smile: --- 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 #1804: [ZEPPELIN-1578] notes list in customizing zeppelin hom...

2016-12-29 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1804 @Leemoonsoo @AhyoungRyu I always appreciate your review for my PR. Could you check this PR if you have time? --- If your project is set up for it, you can reply to this email and have your re

[GitHub] zeppelin issue #1811: [MINOR] Fix paragraph id misalignment in dropdown menu

2016-12-29 Thread cuspymd
Github user cuspymd commented on the issue: https://github.com/apache/zeppelin/pull/1811 Let's try to change the width of browser. It is reprouced when the width of browser is narrow. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1811: [MINOR] Fix paragraph id misalignment in dropdown menu

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1811 @cuspymd Thanks. Let me try :) --- 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

[jira] [Created] (ZEPPELIN-1872) pyspark run error with the paragraph

2016-12-29 Thread Oseong Kwon (JIRA)
Oseong Kwon created ZEPPELIN-1872: - Summary: pyspark run error with the paragraph Key: ZEPPELIN-1872 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1872 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #1808: [ZEPPELIN-1843] Error on invoking the REST API to run ...

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1808 @kavinkumarks Thanks for your contribution! Seems this bug is only shown at the first time after starting Zeppelin. Tested and works well. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1811: [MINOR] Fix paragraph id misalignment in dropdown menu

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1811 @cuspymd I tried to make the browser width narrower and tried to change the paragraph width itself , but still couldn't reproduce. ![changeparagraphwidth](https://cloud.githubusercontent.c

[jira] [Created] (ZEPPELIN-1873) Use exclusively eslint for javascript code linting (remove jscs)

2016-12-29 Thread Ilya Malyanov (JIRA)
Ilya Malyanov created ZEPPELIN-1873: --- Summary: Use exclusively eslint for javascript code linting (remove jscs) Key: ZEPPELIN-1873 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1873 Project: Z

[GitHub] zeppelin pull request #1818: [ZEPPELIN-1873] Use exclusively eslint for java...

2016-12-29 Thread felizbear
GitHub user felizbear opened a pull request: https://github.com/apache/zeppelin/pull/1818 [ZEPPELIN-1873] Use exclusively eslint for javascript code linting ### What is this PR for? [jscs](http://jscs.info) has merged with [eslint](http://eslint.org), therefore both should be un

[jira] [Created] (ZEPPELIN-1874) zeppelin sparkr could not show all the result

2016-12-29 Thread xinzhang (JIRA)
xinzhang created ZEPPELIN-1874: -- Summary: zeppelin sparkr could not show all the result Key: ZEPPELIN-1874 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1874 Project: Zeppelin Issue Type:

[GitHub] zeppelin pull request #1819: [ZEPPELIN-914] Apply new mechanism to IgniteInt...

2016-12-29 Thread DrIgor
GitHub user DrIgor opened a pull request: https://github.com/apache/zeppelin/pull/1819 [ZEPPELIN-914] Apply new mechanism to IgniteInterpreter ### What is this PR for? This handles replacing the registration of interpreter with static block by the interpreter-setting.json file

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

2016-12-29 Thread DrIgor
Github user DrIgor commented on the issue: https://github.com/apache/zeppelin/pull/1561 New pull request #1819 I rebased it on the latest master and fixed failing test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] zeppelin issue #1819: [ZEPPELIN-914] Apply new mechanism to IgniteInterprete...

2016-12-29 Thread DrIgor
Github user DrIgor commented on the issue: https://github.com/apache/zeppelin/pull/1819 Original pull request #1561 IgniteSqlInterpreterTest is known flaky test: [ZEPPELIN-1738](https://issues.apache.org/jira/browse/ZEPPELIN-1738) --- If your project is set up for it, you c

[GitHub] zeppelin issue #1813: [ZEPPELIN-1868]fix to not shows up the login button on...

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1813 Yeah it makes sense. Tested and it's working 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 not

[GitHub] zeppelin pull request #1820: [ZEPPELIN-1841] Some of shortcut key is not wor...

2016-12-29 Thread soralee
GitHub user soralee opened a pull request: https://github.com/apache/zeppelin/pull/1820 [ZEPPELIN-1841] Some of shortcut key is not working ### What is this PR for? Some of note shortcut key is not working on Ubuntu. ### What type of PR is it? [ Bug Fix ] ##

[GitHub] zeppelin issue #1819: [ZEPPELIN-914] Apply new mechanism to IgniteInterprete...

2016-12-29 Thread DrIgor
Github user DrIgor commented on the issue: https://github.com/apache/zeppelin/pull/1819 Travis fails with: ``` testBasicIntp(org.apache.zeppelin.spark.PySparkInterpreterTest) Time elapsed: 58.947 sec <<< FAILURE! java.lang.AssertionError: expected: but was: ``` Kn

[GitHub] zeppelin issue #1815: [ZEPPELIN-1722] Modernize visualization/transformation...

2016-12-29 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1815 \cc @1ambda @felizbear --- 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 issue #1819: [ZEPPELIN-914] Apply new mechanism to IgniteInterprete...

2016-12-29 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1819 Thanks!! LGTM. Will merge 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

[GitHub] zeppelin pull request #1736: [ZEPPELIN-1757] Menu of paragraph includes keyb...

2016-12-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1736 --- 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 #1814: [ZEPPELIN-1869] changed the API response to gen...

2016-12-29 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1814 --- 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 #1814: [ZEPPELIN-1869] changed the API response to gen...

2016-12-29 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1814 [ZEPPELIN-1869] changed the API response to generate to 200. ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check o

[jira] [Created] (ZEPPELIN-1875) permission dialog code is not stable

2016-12-29 Thread CloverHearts (JIRA)
CloverHearts created ZEPPELIN-1875: -- Summary: permission dialog code is not stable Key: ZEPPELIN-1875 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1875 Project: Zeppelin Issue Type: W

[GitHub] zeppelin pull request #1821: [ZEPPELIN-1875] permission dialog code is not s...

2016-12-29 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1821 [ZEPPELIN-1875] permission dialog code is not stable ### What is this PR for? The code for the notification pop-up is not stable. Sometime when response to slow for backend. Then

[GitHub] zeppelin issue #1787: ZEPPELIN-1815. Support Spark 2.1

2016-12-29 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1787 2.1.0 artifacts are on central repo. @zjffdu do you want to update this PR to add build against 2.1? --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] zeppelin issue #1807: ZEPPELIN-1861 Support custom interpreter.sh script to ...

2016-12-29 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1807 interesting. should we have documentation for this? --- 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 hav

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074][ZEPPELIN-1811] Enhance SSL Support

2016-12-29 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1757 looks like we only have a minor comment - could we update and then we could merge? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074][ZEPPELIN-1811] Enhance SSL Support

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1757 @felixcheung Yes. So I can handle it later if you don't mind. I think it'll be faster. Shall we merge then? :) --- If your project is set up for it, you can reply to this email and have your r

[GitHub] zeppelin issue #1761: [ZEPPELIN-1711] Create Docker Images for Released Zepp...

2016-12-29 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1761 My apologies if discussed before, but is there a reason for separating into Scala, Python, R individual Docker images? Don't people want to run Python and R together, for example? Als

[GitHub] zeppelin issue #1821: [ZEPPELIN-1875] permission dialog code is not stable

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1821 @cloverhearts Thanks for quick fixing and improvement. I think it's my bad.. Let me test this! But Selenium test failure looks relevant with this change like below. Could you take a l

[jira] [Created] (ZEPPELIN-1876) Improve JDBC autocompletion - schema, tables, columns and aliases

2016-12-29 Thread Sergey Sotnichenko (JIRA)
Sergey Sotnichenko created ZEPPELIN-1876: Summary: Improve JDBC autocompletion - schema, tables, columns and aliases Key: ZEPPELIN-1876 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1876

[GitHub] zeppelin issue #1821: [ZEPPELIN-1875] permission dialog code is not stable

2016-12-29 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1821 @AhyoungRyu Okay, I will check it. Thank you for your feedback. --- 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 pro

[GitHub] zeppelin issue #1815: [ZEPPELIN-1722] Modernize visualization/transformation...

2016-12-29 Thread felizbear
Github user felizbear commented on the issue: https://github.com/apache/zeppelin/pull/1815 I generally wish that api would be designed more like d3 or jquery, e.g. something along the lines of `visualization(el).render()`; without the need of calling it with `new`; it is more aligned

[GitHub] zeppelin issue #1811: [MINOR] Fix paragraph id misalignment in dropdown menu

2016-12-29 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1811 @cuspymd Could you resolve conflict and test again? I think this problem is gone because this feature is changed. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] zeppelin pull request #1820: [ZEPPELIN-1841] Some of shortcut key is not wor...

2016-12-29 Thread soralee
GitHub user soralee reopened a pull request: https://github.com/apache/zeppelin/pull/1820 [ZEPPELIN-1841] Some of shortcut key is not working ### What is this PR for? Some of note shortcut key is not working on Ubuntu. ### What type of PR is it? [ Bug Fix ]

[GitHub] zeppelin pull request #1820: [ZEPPELIN-1841] Some of shortcut key is not wor...

2016-12-29 Thread soralee
Github user soralee closed the pull request at: https://github.com/apache/zeppelin/pull/1820 --- 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 #1821: [ZEPPELIN-1875] permission dialog code is not s...

2016-12-29 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1821 --- 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 #1821: [ZEPPELIN-1875] permission dialog code is not s...

2016-12-29 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1821 [ZEPPELIN-1875] permission dialog code is not stable ### What is this PR for? The code for the notification pop-up is not stable. Sometime when response to slow for backend. The

[GitHub] zeppelin pull request #1822: [ZEPPELIN-1789] Make exported data file name mo...

2016-12-29 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/1822 [ZEPPELIN-1789] Make exported data file name more explicit ### What is this PR for? Currently we can download table data set as `.csv` or `.tsv` format in Zeppelin note. But since the expo

[GitHub] zeppelin pull request #1823: [ZEPPELIN-1850] Remove strip-loader (webpack)

2016-12-29 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/1823 [ZEPPELIN-1850] Remove strip-loader (webpack) ### What is this PR for? Removed [strip-loader](https://issues.apache.org/jira/browse/ZEPPELIN-1850) which was added to remove `console.log`

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

2016-12-29 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1775 @Leemoonsoo @cloverhearts @1ambda thanks for the review! I addressed all your comments and updated the screenshot above. Please check it out and let me know what you think. Also [

[GitHub] zeppelin issue #1816: [MINOR] Fix broken CSS for plain text paragraph result

2016-12-29 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1816 @AhyoungRyu What a nice catch! Tested and It works very well :+1: [Before] ![z1816_b](https://cloud.githubusercontent.com/assets/8110458/21560618/49767bfe-cea5-11e6-869a-650177440

[GitHub] zeppelin pull request #1824: [ZEPPELIN-1850] Remove grunt-contrib-clean (gru...

2016-12-29 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/1824 [ZEPPELIN-1850] Remove grunt-contrib-clean (grunt) ### What is this PR for? Replaced `grunt-contrib-clean` to [rimraf](https://github.com/isaacs/rimraf) - to remove `Gruntfile.j

[GitHub] zeppelin issue #1823: [ZEPPELIN-1850] Remove strip-loader (webpack)

2016-12-29 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1823 Thanks for fix! I faced this error after merging master into https://github.com/apache/zeppelin/pull/1775 and applied temporary fix by deleting `\'`. It seems to be problem with `strip-load

[jira] [Created] (ZEPPELIN-1877) Documentation for get/set note permission info using rest api

2016-12-29 Thread Ahyoung Ryu (JIRA)
Ahyoung Ryu created ZEPPELIN-1877: - Summary: Documentation for get/set note permission info using rest api Key: ZEPPELIN-1877 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1877 Project: Zeppelin

[GitHub] zeppelin pull request #1825: [DOCS][ZEPPELIN-1586] Add 'Security' section un...

2016-12-29 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/1825 [DOCS][ZEPPELIN-1586] Add 'Security' section under notebook restapi docs ### What is this PR for? After #1567 merged we can get/set note permission information for a certain note. But this

[GitHub] zeppelin issue #1825: [DOCS][ZEPPELIN-1586] Add 'Security' section under not...

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1825 @anthonycorbacho As a creator of this operations, could you please help review this PR? :D --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] zeppelin issue #1816: [MINOR] Fix broken CSS for plain text paragraph result

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1816 @soralee Thanks for testing! --- 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 #1817: remove usage of non-standard string method

2016-12-29 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/1817 Tested it and works as expected. But CI is fail. Could you check CI again? --- 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 issue #1775: [ZEPPELIN-1745] View revisions in non-editable mode

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1775 @khalidhuseynov How about disable "Interpreter binding" & "Note permission" buttons as well? Currently I can change the bound interpreters and note permission for the readonly-mode note. ---

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074][ZEPPELIN-1811] Enhance SSL Support

2016-12-29 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1757 @felixcheung and @AhyoungRyu Sorry for the delay, as I was a little offline in the last few days. I have implemented the suggestion and this should be ready to merge. --- If your project is set

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074][ZEPPELIN-1811] Enhance SSL Support

2016-12-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1757 @lresende Thanks! Will check again. --- 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