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

2016-11-25 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1617 @randerzander Could you rebase? --- 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 #1685: [ZEPPELIN-1714] Build error of scio Intepreter ...

2016-11-25 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1685 [ZEPPELIN-1714] Build error of scio Intepreter on Centos ### What is this PR for? This PR fixes build error of scio Intepreter on CentOS. ### What type of PR is it? Bug

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

2016-11-26 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1685 @AhyoungRyu Thank you for reviewing. I am using Zeppelin master on CentOS-7. --- 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 #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 #1679: ZEPPELIN-1707. Pass userName when creating interpreter...

2016-11-29 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1679 Sorry for late review. User name is already passed to interpreter. We can get the user id on interpreterContext.getAuthenticationInfo().getUser() I think passing password

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

2016-11-29 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1679 @zjffdu It's my misunderstanding. 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 pull request #1707: Add PythonDockerInterpreter to interpreter grou...

2016-11-29 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1707 Add PythonDockerInterpreter to interpreter group ### What is this PR for? Recently PythonDockerInterpreter added to PythonInterpreter in https://github.com/apache/zeppelin/pull/1654

[GitHub] zeppelin issue #1707: Add PythonDockerInterpreter to interpreter group

2016-11-29 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1707 @Leemoonsoo It was my mistake. I just fixed. Thank you. --- 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 #1706: [HOTFIX] Exclude org/apache/zeppelin/scio/avro/* once ...

2016-11-29 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1706 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 #1708: [ZEPPELIN-1728] Add an example configuring classpath f...

2016-11-30 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1708 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 #1566: [ZEPPELIN-1587] (WIP) Add impersonation routine in Spa...

2016-11-30 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1566 @khalidhuseynov How about using `System.setProperty("HADOOP_USER_NAME", getUserName());` instead of `proxy-user` setting? Could you review https://github.com/astroshim/zeppel

[GitHub] zeppelin pull request #1284: [ZEPPELIN-1291]Tagging meta value on interprete...

2016-12-01 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1284 --- 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 #1715: [ZEPPELIN-1740] Focus next paragraph after paragraph d...

2016-12-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1715 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 #1716: [ ZEPPELIN-531 ] execute and stop note in job menu

2016-12-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1716 It's working as expected. Thank you for great improvement. --- 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 #1717: [ZEPPELIN-1744] replace to movement for paragraph in j...

2016-12-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1717 Working well. 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

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

2016-12-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1721 Working 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 this feature enabled

[GitHub] zeppelin issue #1719: [ZEPPELIN-965] missing lastExecuteTime function

2016-12-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1719 Tested. 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

[GitHub] zeppelin pull request #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

2016-12-07 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1735 [ZEPPELIN-1767] Blank paragraph should not be run. ### What is this PR for? This PR fixes not to run blank paragraph. ### What type of PR is it? Improvement

[GitHub] zeppelin issue #1735: [ZEPPELIN-1767] Blank paragraph should not be run.

2016-12-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1735 I just fixed typo. Thanks @rhymoid and @cloverhearts to review 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

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

2016-12-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1736 @soralee Thank you for contribution of useful function. but I am usually using the `Shift + Enter` to run paragraph, How can be presented this shortcut? and it seems necessary a

[GitHub] zeppelin pull request #1744: Better output of JDBC Interpreter.

2016-12-10 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1744 Better output of JDBC Interpreter. ### What is this PR for? Currently, the output format of the JDBC Interpreter is the table format with all the results except EXPLAIN statement, It is

[GitHub] zeppelin issue #1735: [ZEPPELIN-1767] Blank paragraph should not be run.

2016-12-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1735 Thank you @AhyoungRyu ! I didn't catch the 3rd. I just fix that and let me add testcase for this. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1735: [ZEPPELIN-1767] Blank paragraph should not be run.

2016-12-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1735 Testcase also added. @AhyoungRyu Please 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

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

2016-12-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1694 I agree with @prabhjyotsingh and @cloverhearts blocking to change the permission on a backend side too. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

2016-12-12 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1735 --- 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 pull request #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

2016-12-12 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1735 [ZEPPELIN-1767] Blank paragraph should not be run. ### What is this PR for? This PR fixes not to run blank paragraph. ### What type of PR is it? Improvement

[GitHub] zeppelin issue #1566: [ZEPPELIN-1587] (WIP) Add impersonation routine in Spa...

2016-12-13 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1566 I agree with @khalidhuseynov's opinion. I was keep wondering it's possible impersonation on `shared` or `scoped` mode. My PR is just created for the testing and only teste

[GitHub] zeppelin issue #1744: Better output of JDBC Interpreter.

2016-12-13 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1744 Thank you @Leemoonsoo and @bzz help to review. CI failure is not irrelevant to this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin pull request #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

2016-12-13 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1735 --- 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 pull request #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

2016-12-13 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1735 [ZEPPELIN-1767] Blank paragraph should not be run. ### What is this PR for? This PR fixes not to run blank paragraph. ### What type of PR is it? Improvement

[GitHub] zeppelin issue #1752: [HOT FIX] Fix broken display_formula.png by using rela...

2016-12-13 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1752 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 #1735: [ZEPPELIN-1767] Blank paragraph should not be run.

2016-12-13 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1735 CI is green now. :) --- 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 pull request #1780: [BugFix] Tutorial note json format.

2016-12-16 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1780 [BugFix] Tutorial note json format. ### What is this PR for? Since https://github.com/apache/zeppelin/pull/1658, the output format had changed but note.json of Tutorials didn't ch

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

2016-12-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1777 Thanks for contribution! Looks great 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

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

2016-12-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1780 @AhyoungRyu Thank you for review and let me check that problem. --- 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 #1780: [BugFix] Tutorial note json format.

2016-12-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1780 In my case, It's normal output of results. @AhyoungRyu Did you build this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

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

2016-12-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1780 @bzz I agree with you and I will follow your word! 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

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

2016-12-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1780 CI has passed after rebase. --- 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 #1780: [BugFix] Tutorial note json format.

2016-12-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1780 @AhyoungRyu 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 enabled and wishes

[GitHub] zeppelin pull request #1813: [ZEPPELIN-1868]fix to not shows up the login bu...

2016-12-28 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1813 [ZEPPELIN-1868]fix to not shows up the login button on Anonymous mode. ### What is this PR for? This PR fixs to not shows up the login button on Anonymous mode. ### What type

[GitHub] zeppelin pull request #1832: [ZEPPELIN-1884] Prevent NullPointerException wh...

2017-01-01 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1832 [ZEPPELIN-1884] Prevent NullPointerException when JDBC query. ### What is this PR for? Sometimes users don't want to set the properties which they don't want to care

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

2017-01-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1813 @soralee Thank you for reviewing and commenting!! --- 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 #1832: [ZEPPELIN-1884] Prevent NullPointerException wh...

2017-01-01 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1832 --- 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 pull request #1832: [ZEPPELIN-1884] Prevent NullPointerException wh...

2017-01-01 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1832 [ZEPPELIN-1884] Prevent NullPointerException when JDBC query. ### What is this PR for? Sometimes users don't want to set the properties which they don't want to care

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

2017-01-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1831 In my test, I got a ` INFO [2017-01-02 09:08:12,358] ({Exec Default Executor} RemoteInterpreterManagedProcess.java[onProcessComplete]:164) - Interpreter process exited 0` error when i try to

[GitHub] zeppelin issue #1833: [ZEPPELIN-1880] Fix shell interpreter output streaming...

2017-01-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1833 Hi. In my case it seems doesn`t work properly.. What am i missing? ![test1](https://cloud.githubusercontent.com/assets/3348133/21593751/75fa43be-d0cf-11e6-8f83-aa102458dc29.gif

[GitHub] zeppelin pull request #1832: [ZEPPELIN-1884] Prevent NullPointerException wh...

2017-01-02 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1832 --- 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 pull request #1832: [ZEPPELIN-1884] Prevent NullPointerException wh...

2017-01-02 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1832 [ZEPPELIN-1884] Prevent NullPointerException when JDBC query. ### What is this PR for? Sometimes users don't want to set the properties which they don't want to care

[GitHub] zeppelin issue #1832: [ZEPPELIN-1884] Prevent NullPointerException when JDBC...

2017-01-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1832 @felixcheung Thank you for review! Currently only `COMMON_KEY` uses for max line count in JDBC interpreter but as like you mentioned it seems more make sense to move the code to `open

[GitHub] zeppelin issue #1832: [ZEPPELIN-1884] Prevent NullPointerException when JDBC...

2017-01-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1832 @felixcheung I updated code, please review. Thanks in advance. --- 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 #1848: [ZEPPELIN-1812] Ace editor show/hide problem.

2017-01-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1848 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin pull request #1853: [ZEPPELIN-1911] Adding how to install evaluate ...

2017-01-05 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1853 [ZEPPELIN-1911] Adding how to install evaluate package to build R. ### What is this PR for? This PR inform how to avoid build failing with R. ### What type of PR is it

[GitHub] zeppelin pull request #1858: [ZEPPELIN-1229] Clear browser cache with versio...

2017-01-06 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1858 [ZEPPELIN-1229] Clear browser cache with version hashtag. ### What is this PR for? This PR avoids browser cache. ### What type of PR is it? Bug Fix ### What is the

[GitHub] zeppelin issue #1858: [ZEPPELIN-1229] Clear browser cache with version hasht...

2017-01-06 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1858 @corneadoug I updated screenshot and codes. Could you help to 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

[GitHub] zeppelin pull request #1858: [ZEPPELIN-1229] Clear browser cache with versio...

2017-01-06 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1858 [ZEPPELIN-1229] Clear browser cache with version hashtag. ### What is this PR for? This PR avoids browser cache. ### What type of PR is it? Bug Fix ### What is the

[GitHub] zeppelin pull request #1858: [ZEPPELIN-1229] Clear browser cache with versio...

2017-01-06 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1858 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] zeppelin pull request #1853: [ZEPPELIN-1911] Adding how to install evaluate ...

2017-01-06 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1853 --- 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 pull request #1853: [ZEPPELIN-1911] Adding how to install evaluate ...

2017-01-06 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1853 [ZEPPELIN-1911] Adding how to install evaluate package to build R. ### What is this PR for? This PR inform how to avoid build failing with R. ### What type of PR is it

[GitHub] zeppelin pull request #1853: [ZEPPELIN-1911] Adding how to install evaluate ...

2017-01-07 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1853 --- 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 #1853: [ZEPPELIN-1911] Adding how to install evaluate package...

2017-01-07 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1853 @felixcheung You're right! `evaluate` is dependency package of `knitr`. Thanks for pointing out. It's my misunderstood. Let me close this PR. --- If your project is set up f

[GitHub] zeppelin issue #1862: [HOTFIX] Removed non-string properties values since it...

2017-01-07 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1862 Tested and LGTM! Thanks @jongyoul --- 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 issue #1867: [MINOR][ZEPPELIN-1919] Reduce markdown intp's plain te...

2017-01-07 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1867 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 #1858: [ZEPPELIN-1229] Clear browser cache with version hasht...

2017-01-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1858 @corneadoug Let me find different of more efficient way. --- 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 #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter synta...

2017-01-09 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1877 [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error. ### What is this PR for? This PR fixes syntax error of PythonInterpreter. ### What type of PR is it? Bug Fix

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1877 @agoodm pyspark interpreter is working well. ![image](https://cloud.githubusercontent.com/assets/3348133/21768320/e79bebe0-d62d-11e6-8878-3c837fb7938f.png) --- If your project is set up

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1877 @Leemoonsoo I added testcase, could you help to 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

[GitHub] zeppelin pull request #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter synta...

2017-01-09 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1877 --- 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 pull request #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter synta...

2017-01-09 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1877 [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error. ### What is this PR for? This PR fixes syntax error of PythonInterpreter. ### What type of PR is it? Bug Fix

[GitHub] zeppelin issue #1858: [ZEPPELIN-1229] Clear browser cache with version hasht...

2017-01-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1858 @corneadoug please help to 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

[GitHub] zeppelin issue #1845: [ZEPPELIN-1906] Use multiple InterpreterResult for dis...

2017-01-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1845 Sorry for late response @prabhjyotsingh, @jongyoul . Let me review soon. --- 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 issue #1845: [ZEPPELIN-1906] Use multiple InterpreterResult for dis...

2017-01-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1845 @prabhjyotsingh Thank you for great improvement! I tested this with mysql and all features are working well except one. In following case, database connection would be closed and user

[GitHub] zeppelin issue #1845: [ZEPPELIN-1906] Use multiple InterpreterResult for dis...

2017-01-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1845 @prabhjyotsingh Yea I know you didn't change it. That's because it doesn't have state before. I agree that we can take care/discuss it in a different PR. Thanks.

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

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

[GitHub] zeppelin pull request #1922: [ZEPPELIN-1991] Can't get the PARAGRAPH_APPEND_...

2017-01-20 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1922 [ZEPPELIN-1991] Can't get the PARAGRAPH_APPEND_OUTPUT from the Interpreter. ### What is this PR for? This PR fixes the problem of streaming events(PARAGRAPH_APPEND_OUTPUT). It'

[GitHub] zeppelin issue #1918: Remove multple artifactId - hadoop-common

2017-01-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1918 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 #1907: [MINOR] Add enough padding at the bottom of notebook

2017-01-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1907 Thank you for contribution but the `margin` affect also on the `report` mode. --- 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 #1917: [ZEPPELIN-1988] precode execution in JDBCInterpreter

2017-01-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1917 Thank you for contribution!. please rebase PR and If you let us know about testing way on detail, it's easy to review. --- If your project is set up for it, you can reply to

[GitHub] zeppelin pull request #1924: [Pyspark] checking nhooks count.

2017-01-20 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1924 [Pyspark] checking nhooks count. ### What is this PR for? if the `nhooks` value is `0` in this(https://github.com/apache/zeppelin/blob/master/spark/src/main/resources/python

[GitHub] zeppelin pull request #1922: [ZEPPELIN-1991] Can't get the PARAGRAPH_APPEND_...

2017-01-21 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1922 --- 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 pull request #1922: [ZEPPELIN-1991] Can't get the PARAGRAPH_APPEND_...

2017-01-21 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1922 [ZEPPELIN-1991] Can't get the PARAGRAPH_APPEND_OUTPUT from the Interpreter. ### What is this PR for? This PR fixes the problem of streaming events(PARAGRAPH_APPEND_OUTPUT).

[GitHub] zeppelin pull request #1927: [ZEPPELIN-1994] bugfix of streaming output.

2017-01-21 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1927 [ZEPPELIN-1994] bugfix of streaming output. ### What is this PR for? If you run the following code, then streaming output doesn't work properly from the secon

[GitHub] zeppelin issue #1924: [Pyspark] checking nhooks count.

2017-01-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1924 Thank you @felixcheung and @Leemoonsoo for reviewing this. Let me check the CI. --- 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 #1924: [Pyspark] checking nhooks count.

2017-01-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1924 CI has passed. --- 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

[GitHub] zeppelin issue #1833: [ZEPPELIN-1880] Fix shell interpreter output streaming...

2017-01-23 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1833 LGTM 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 have this feature enable

[GitHub] zeppelin issue #1924: [Pyspark] checking nhooks count.

2017-01-27 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1924 @Leemoonsoo I just create JIRA issue that is https://issues.apache.org/jira/browse/ZEPPELIN-2024. Thanks. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1953: User Impersonation fails on setting hive_conf_list in ...

2017-01-30 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1953 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 #1886: [ZEPPELIN-1876] improved comptetion with schema/table/...

2017-01-31 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1886 @sotnich Thank you for great improvement. I tested `mysql` and `postgresql` and only `postgresql` is working expectedly like @cloverhearts. --- If your project is set up for it, you can

[GitHub] zeppelin issue #1962: [ZEPPELIN-2014] Jetty Directory Listing on app, assets...

2017-01-31 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1962 Tested and working expectably. Thank you for contributing but Could you make the CI as green? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1886: [ZEPPELIN-1876] improved comptetion with schema/table/...

2017-02-03 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1886 @sotnich Great Job! but still doesn't work with `mysql`. --- 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 pull request #1985: [ZEPPELIN-2075] Can't stop infinite `while` sta...

2017-02-07 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1985 [ZEPPELIN-2075] Can't stop infinite `while` statement in pyspark Interpreter. ### What is this PR for? If following code runs with Pyspark Interpreter, there is no way to cancel e

[GitHub] zeppelin pull request #1985: [ZEPPELIN-2075] Can't stop infinite `while` sta...

2017-02-12 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1985 [ZEPPELIN-2075] Can't stop infinite `while` statement in pyspark Interpreter. ### What is this PR for? If following code runs with Pyspark Interpreter, there is no way to cancel e

[GitHub] zeppelin pull request #1985: [ZEPPELIN-2075] Can't stop infinite `while` sta...

2017-02-12 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1985 --- 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 #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

2017-02-14 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1985 Hello. Sorry for late response. @Leemoonsoo Let me fix CI error. @karuppayya Let me fix `scala` too in another PR. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1985: [ZEPPELIN-2075] Can't stop infinite `while` sta...

2017-02-16 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1985 --- 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 pull request #1985: [ZEPPELIN-2075] Can't stop infinite `while` sta...

2017-02-16 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1985 [ZEPPELIN-2075] Can't stop infinite `while` statement in pyspark Interpreter. ### What is this PR for? If following code runs with Pyspark Interpreter, there is no way to cancel e

[GitHub] zeppelin issue #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

2017-02-19 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1985 @zjffdu yes, this PR is not kill the python process. Thank you for reviewing! --- 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 #1985: [ZEPPELIN-2075] Can't stop infinite `while` sta...

2017-02-19 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1985 [ZEPPELIN-2075] Can't stop infinite `while` statement in pyspark Interpreter. ### What is this PR for? If following code runs with Pyspark Interpreter, there is no way to cancel e

[GitHub] zeppelin pull request #1985: [ZEPPELIN-2075] Can't stop infinite `while` sta...

2017-02-19 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1985 --- 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 pull request #2052: [ZEPPELIN-2048] Can't run first paragraph when ...

2017-02-21 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/2052 [ZEPPELIN-2048] Can't run first paragraph when personalize mode on. ### What is this PR for? Problem of ZEPPELIN-2048 was because not set the user id when inserting new para

[GitHub] zeppelin pull request #2052: [ZEPPELIN-2048] Can't run first paragraph when ...

2017-02-21 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/2052 --- 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

<    1   2   3   4   5   6   >