from:"cloverhearts"

Re: [VOTE] Release Apache Zeppelin 0.9.0-preview1 (RC1)

2020-03-26 Thread NineQS CloverHearts

+1

보낸 사람: Khalid Huseynov 
회신 대상: "us...@zeppelin.apache.org" 
날짜: 2020년 3월 26일 목요일 오후 4:48
받는 사람: "us...@zeppelin.apache.org" 
참조: dev 
주제: Re: [VOTE] Release Apache Zeppelin 0.9.0-preview1 (RC1)

+ 1

On Thu, Mar 26, 2020 at 4:19 PM Xun Liu 
mailto:liu...@apache.org>> wrote:
+1

On Thu, Mar 26, 2020 at 5:33 AM moon soo Lee 
mailto:m...@apache.org>> wrote:
+1

Tested
 - The binary package can run tutorial notebook out of the box
 - Can build docker image
- scripts/docker/zeppelin/bin/Dockerfile give me some error on R
dependency installation. But I think it's not a blocker. We can address it
on next RC.
 - Zeppelin on Kubernetes
- Everything but Spark-2.4.5 does not run out of the box. I think it is
not a blocker. We can address it on next RC.




On Wed, Mar 25, 2020 at 11:21 AM Prabhjyot Singh 
mailto:prabhjyotsi...@apache.org>>
wrote:

> +1
>
> On Wed, 25 Mar 2020 at 21:12, Eric Pugh 
> mailto:ep...@opensourceconnections.com>>
> wrote:
>
> > Out of curiosity, is there a Docker version of 0.9.0 preview1 available?
> > I tend to run Zeppelin as a Docker image, and I know I had some
> challenges
> > with the 0.8.x line of Docker images from
> > https://hub.docker.com/r/apache/zeppelin.
> >
> > I did modify the ./scripts/docker/zeppelin/Dockerfile to load the
> > v0.0.0-preview1-rc1.bin.tgz distribution.
> >
> > Then I followed the steps in docs/setup/deployment/docker.md to try out
> > Zeppelin and it all appears to work as a Docker image!
> >
> > One thing to note was that it didn’t auto upgrade my old .json files into
> > the new .zpln files.  I had to manually upload them.  Is there any auto
> > upgrade feature for existing notebook files?
> >
> > So non binding +1 ;-)
> >
> > Eric
> >
> >
> >
> > On Mar 25, 2020, at 1:54 AM, Jongyoul Lee 
> > mailto:jongy...@gmail.com>> wrote:
> >
> > +1
> >
> > 2020년 3월 24일 (화) 오전 12:15, Jeff Zhang 
> > mailto:zjf...@gmail.com>>님이 작성:
> >
> >> Hi folks,
> >>
> >> I propose the following RC to be released for the Apache Zeppelin
> >> 0.9.0-preview1 release.
> >>
> >>
> >> The commit id is a74365c0813b451db1bc78def7d1ad1279429224 :
> >>
> >>
> https://gitbox.apache.org/repos/asf?p=zeppelin.git;a=commit;h=a74365c0813b451db1bc78def7d1ad1279429224
> >>
> >> This corresponds to the tag: v0.9.0-preview1-rc1 :
> >>
> >>
> https://gitbox.apache.org/repos/asf?p=zeppelin.git;a=shortlog;h=refs/tags/v0.9.0-preview1-rc1
> >>
> >> The release archives (tgz), signature, and checksums are here
> >>
> >>
> https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.9.0-preview1-rc1/
> >>
> >> The release candidate consists of the following source distribution
> >> archive
> >> zeppelin-v0.9.0-preview1.tgz
> >>
> >> In addition, the following supplementary binary distributions are
> provided
> >> for user convenience at the same location
> >> zeppelin-0.9.0-preview1-bin-all.tgz
> >>
> >>
> >> The maven artifacts are here
> >>
> >>
> https://repository.apache.org/content/repositories/orgapachezeppelin-1276/org/apache/zeppelin/
> >>
> >> You can find the KEYS file here:
> >> https://dist.apache.org/repos/dist/release/zeppelin/KEYS
> >>
> >> Release notes available at
> >>
> >>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342692==12316221
> >>
> >> Vote will be open for next 72 hours (close at 7am 26/March PDT).
> >>
> >> [ ] +1 approve
> >> [ ] 0 no opinion
> >> [ ] -1 disapprove (and reason why)
> >>
> >>
> >> --
> >> Best Regards
> >>
> >> Jeff Zhang
> >>
> >
> >
> > --
> > 이종열, Jongyoul Lee, 李宗烈
> > http://madeng.net
> >
> >
> > ___
> > *Eric Pugh **| *Founder & CEO | OpenSource Connections, LLC |
> 434.466.1467
> > | http://www.opensourceconnections.com | My Free/Busy
> > 
> > Co-Author: Apache Solr Enterprise Search Server, 3rd Ed
> > <
> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw
> >
> > This e-mail and all contents, including attachments, is considered to be
> > Company Confidential unless explicitly stated otherwise, regardless
> > of whether attachments are marked as such.
> >
> >
>
> --
>
> Warm Regards,
>
> Prabhjyot Singh
>

Re: [VOTE] Release Apache Zeppelin 0.8.2 (RC1)

2019-09-26 Thread CloverHearts

+1

2019년 9월 26일 (목) 오후 2:13, Jeff Zhang 님이 작성:

>
> Hi folks,
>
> I propose the following RC to be released for the Apache Zeppelin
> 0.8.2 release.
>
>
> The commit id is b520400c58082ee40b70abb432170ea1c809d3d6 : 
> https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=b520400c58082ee40b70abb432170ea1c809d3d6
>
> This corresponds to the tag: v0.8.2-rc1：
>
> https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=shortlog;h=refs/tags/v0.8.2-rc1
>
> The release archives (tgz), signature, and checksums are here 
> https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.8.2-rc1/
> The release candidate consists of the following source distribution archive
> zeppelin-0.8.2.tgz
>
> In addition, the following supplementary binary distributions are provided
> for user convenience at the same location
> zeppelin-0.8.2-bin-all.tgz
>
>
> The maven artifacts are here 
> https://repository.apache.org/content/repositories/orgapachezeppelin-1273/org/apache/zeppelin/
> You can find the KEYS file here: 
> https://dist.apache.org/repos/dist/release/zeppelin/KEYS
> Release notes available at 
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12344888=Html=12316221=Create_token=A5KQ-2QAV-T4JA-FDED_dbd90e29a5db9fe77fb666b31fada2c824e6905a_lin
>
> Vote will be open for next 72 hours (close at 10pm 28/Sep PDT).
>
> [ ] +1 approve
> [ ] 0 no opinion
> [ ] -1 disapprove (and reason why)
>
>
> --
> Best Regards
>
> Jeff Zhang
>

Re: [VOTE] Release Apache Zeppelin 0.8.0 (RC6)

2018-06-26 Thread NineQS CloverHearts

+1

From: Belousov Maksim Eduardovich 
Sent: Tuesday, June 26, 2018 10:27:49 PM
To: dev@zeppelin.apache.org
Subject: RE: [VOTE] Release Apache Zeppelin 0.8.0 (RC6)

+1


Regards,

Maksim Belousov

-Original Message-
From: Prabhjyot Singh [mailto:prabhjyotsi...@apache.org]
Sent: Monday, June 25, 2018 11:33 AM
To: dev 
Subject: Re: [VOTE] Release Apache Zeppelin 0.8.0 (RC6)

+1

On Mon, 25 Jun 2018 at 06:05, Jeff Zhang  wrote:

> Here's my +1.
>
> I build it from source successfully and ran spark tutorial in local,
> yarn-client, yarn-cluster mode successfully.
>
>
>
> Jeff Zhang 于2018年6月25日周一 上午8:34写道：
>
> > Hi folks,
> >
> > I propose the following RC to be released for the Apache Zeppelin
> > 0.8.0 release.
> >
> > The commit id is 603cc492f5e322c73915501e8cf65970d0c61870
> >
> >  *
> https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=603c
> c492f5e322c73915501e8cf65970d0c61870
> <
> https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=603c
> c492f5e322c73915501e8cf65970d0c61870
> >*
> >
> >
> > This corresponds to the tag: v0.8.0-rc6
> >
> > The release archives (tgz), signature, and checksums are here *
> https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.8.0-rc6/ <
> https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.8.0-rc6/>*
> >
> >
> > The release candidate consists of the following source distribution
> archive zeppelin-0.8.0.tgz
> >
> > In addition, the following supplementary binary distributions are
> provided
> > for user convenience at the same location zeppelin-0.8.0-bin-all.tgz
> >
> > zeppelin-0.8.0-bin-netinst.tgz
> >
> > The maven artifacts are here *
> https://repository.apache.org/content/repositories/orgapachezeppelin-1
> 252
> > <
> https://repository.apache.org/content/repositories/orgapachezeppelin-1
> 252
> >*
> >
> > You can find the KEYS file here:
> https://dist.apache.org/repos/dist/release/zeppelin/KEYS
> >
> > Release notes available at
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12316
> 221=12339144
> >
> > Vote will be open for next 72 hours (close at 6PM 27/June PDT).
> >
> > [ ] +1 approve
> > [ ] 0 no opinion
> > [ ] -1 disapprove (and reason why)
> >
> >
>
> --
>
> Warm Regards,
>
> Prabhjyot Singh
>
>

Re: [VOTE] Release Apache Zeppelin 0.8.0 (RC5)

2018-06-19 Thread CloverHearts

Awesome +1






On Wed, Jun 20, 2018 at 11:41 AM +0900, "Jeff Zhang"  wrote:










Hi folks,

I propose the following RC to be released for the Apache Zeppelin
0.8.0 release.

The commit id is adc2e63c1ff1f7690ba7b3c10984c8024e379a31

 
*https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=adc2e63c1ff1f7690ba7b3c10984c8024e379a31
*



This corresponds to the tag: v0.8.0-rc5

The release archives (tgz), signature, and checksums are here
*https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.8.0-rc5/
*


The release candidate consists of the following source distribution
archive zeppelin-0.8.0.tgz

In addition, the following supplementary binary distributions are provided
for user convenience at the same location zeppelin-0.8.0-bin-all.tgz

zeppelin-0.8.0-bin-netinst.tgz

The maven artifacts are here
*https://repository.apache.org/content/repositories/orgapachezeppelin-1242/
*

You can find the KEYS file here:
https://dist.apache.org/repos/dist/release/zeppelin/KEYS

Release notes available at
https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12316221=12339144

Vote will be open for next 72 hours (close at 8PM 22/June PDT).

[ ] +1 approve
[ ] 0 no opinion
[ ] -1 disapprove (and reason why)

[GitHub] zeppelin issue #2707: [ZEPPELIN-3100] Upgrade node and npm version

2017-12-18 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2707
  
I was tested. Looks good :)
Lgtm


---

[GitHub] zeppelin pull request #2583: Improved stability for conda interpreter.

2017-09-12 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2583


---

[GitHub] zeppelin pull request #2583: Improved stability for conda interpreter.

2017-09-12 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2583

Improved stability for conda interpreter.

### What is this PR for?
Currently, when configuring environment of conda interpreter, when 
configuring environment without installing python separately,
The python interpreter may fall into an infinite loop.
This PR solves that.

ORIGINAL : #2574 

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2921

### How should this be tested?


### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no 


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
feat/keep-python-interpreter-cmd

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2583.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2583


commit 9d7739c66dcda0d0db1a264079ac17d82aba9edc
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-09-12T08:53:04Z

keep python interpreter cmd




---

[GitHub] zeppelin issue #2576: [hotfix] JDBC connection does not release when got exc...

2017-09-11 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2576
  
I will merge to master


---

[GitHub] zeppelin issue #2576: [hotfix] JDBC connection does not release when got exc...

2017-09-11 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2576
  
@felixcheung 
Thank you :)


---

[GitHub] zeppelin issue #2576: [hotfix] JDBC connection does not release when got exc...

2017-09-11 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2576
  
This is a hot-fix change.
Therefore, they merge quickly.


---

[GitHub] zeppelin issue #2578: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-11 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2578
  
@zjffdu Thank you a lot :)


---

[GitHub] zeppelin issue #2578: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-11 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2578
  
If there are no comments anymore, I will merge to master on this pr.


---

[GitHub] zeppelin issue #2578: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2578
  
fixed test case and conda list feature


---

[GitHub] zeppelin issue #2578: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2578
  
@1ambda okay :)


---

[GitHub] zeppelin issue #2578: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2578
  
@zjffdu Sure, Thank you :)


---

[GitHub] zeppelin issue #2578: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2578
  
The problem of CI is irrelevant to this pr.

```
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 12.462 sec 
<<< FAILURE! - in 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterServerTest

testStartStop(org.apache.zeppelin.interpreter.remote.RemoteInterpreterServerTest)
  Time elapsed: 10.043 sec  <<< FAILURE!
java.lang.AssertionError: expected: but was:
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterServerTest.testStartStop(RemoteInterpreterServerTest.java:62)
```


---

[GitHub] zeppelin pull request #2575: [ZEPPELIN-2921] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2575


---

[GitHub] zeppelin pull request #2574: [ZEPPELIN-2921 : 0.7x] does not work conda envi...

2017-09-10 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2574


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
I have separated this PR.
I'm sorry for the confusion.
https://github.com/apache/zeppelin/pull/2578


---

[GitHub] zeppelin issue #2578: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2578
  
@1ambda @zjffdu @felixcheung 

Sorry.
I divided the existing PR by the minimum function.
Can you review this PR again?
Thank you for your help :)


---

[GitHub] zeppelin pull request #2578: [ZEPPELIN-2921] does not work conda environment...

2017-09-10 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2578

[ZEPPELIN-2921] does not work conda environment in python interpreter 

### What is this PR for?
It seems that the environment of the python interpreter has changed to the 
py4j environment, causing problems with the library path and the default 
environment.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2921

### How should this be tested?
Please run the following command line for each paragraph.
If the module such as scipy is normally imported, it is a success.
`%python.conda create --name Hello6 python=2.7`
`%python.conda activate Hello6`
`%python.conda install seaborn pandas numpy scipy matplotlib`
```
%python
import scipy as sp
import seaborn as sns
```
`%python.conda deactivate`

### Screenshots (if appropriate)
 Before

![image](https://user-images.githubusercontent.com/10525473/30199920-c75022ca-94af-11e7-8811-0c22310f1bac.png)

 After

![image](https://user-images.githubusercontent.com/10525473/30198880-23aaceb2-94ab-11e7-8bc6-bfad76c675f7.png)


### Questions:
* Does the licenses files need update? no 
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin ZEPPELIN-2921

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2578.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2578


commit 8dfb33b1159695e109a696c38a3fd6a112a79e13
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-09-10T16:36:19Z

replace conda install environment name




---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
Thank you @1ambda @zjffdu @felixcheung 
I think my code with various purposes is distracting you.
And, unfortunately, my English ability is difficult to describe in a 
sophisticated way.
Therefore, I will share PR.
In this PR, we will only deal with the wrong library installation of conda.

Many thanks for your comments.


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
@zjffdu @1ambda @felixcheung 
Thank you for your kind reply despite my lack of English.
I hope this PR is merged.
So, i will try to reflect your opinions as much as possible.
Thank you. :)


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
In my opinion, allowing for fluctuations in the python environment is 
itself a problem.
But zeppelin are supporting it.
If it does not, then the semantics of conda activate and conda deactivate 
disappear.

In fact, I also think that changing the python environment can be a problem.
However, the current Zeppelin supports it, and a bug exists.
This PR is a PR that solves the bug.


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
I have a solution to remove conda related members from PythonInterpreter. 
also PythonCondaInterpreter will not have any members.
However, it differs in focus from this PR.
(The problem is that there are changes related to the start of the Python 
Interpreter.)

This PR is intended to be used when the user changes the environment by 
installing new python while using conda.
This also solves the Zeppelin Python Interpreter's infinite loop problem.


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
@1ambda 
Currently there is a possibility that it will be structurally.
However, if you store it in PythonCondaInterpreter, the problem is greatly 
aggravated.

1. PythonInterpreter needs to monitor PythonCondaInterpreter at all times.
Â Â Â Â  (Python Interpreter should check Conda's environment activation 
when restarting.)

2. Other configurations other than PythonCondaInterpreter If an interpreter 
is added (as you might have guessed), the Python Interpreter is in a situation 
where you need to constantly query for all other Interpreter environments. This 
will affect the life cycle of the Python Interpreter.


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-10 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
@zjffdu 
This change will work exactly as your request.
There is only one python library path to be added.
1. If the user does not install python via conda.
(If the user installs only the library)
Zeppelin use to `zeppelin.python`.
(Used in the original environment.)

2. If you install python via conda.
(separate python from zeppelin.python)
if when the conda environment is active, add to library path and replace to 
installed python via conda from zeppelin.python.
Zeppelin to work in the new python environment.

3. The user disables the environment. (conda deactiavte)
It is restored to the original zeppelin.python environment.



---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-09 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
I think we are worried about `conda interpreter` and coupling for` conda 
env name`.
However, the conda env name is more important to the python interpreter 
than the conda interpreter.
Also, the python interpreter should not query the conda interpreter for its 
own state.

I think this misunderstanding is due to the member name `conda env name`.
If I have any additional problems, I will change this to `virtualEnvName`.


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-09 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
@zjffdu 
Thank you for your opinion,
I present scenarios for environment changes using conda.
Python version and module names are arbitrary, so do not worry.
What is important is change in every environment.

1. Basically Zeppelin's python
python: 2.71
installed library: (none)

use to zeppelin
```
%python
import sys
print(sys.version)
print(sys.path)
```
result
```
python version 2.7.1
%ZEPPELIN_HOME%/interpreter/python/lib:%PYTHON_HOME
```
```
%python
import myModule
```
result
```
no module myModule
```


2. install conda and new python 
```
%python.conda create --name python3 python=3.4
```
```
%python.conda activate python3
```
```
%python
import sys
print(sys.version)
print(sys.path)
```
result
```
python version 3.4
%ZEPPELIN_HOME%/interpreter/python/lib:%PYTHON_HOME:%CONDA_ENV%/python3/lib
```
```
%python
import myModule
```
result
```
no module myModule
```
try again
```
%python.conda install myModule
```

```
import myModule
```
result
```
imported.
```

3. restore python environment
```
%python.conda deactivate
```
check for recovered to the original Zeppelin environment.
```
%python
import sys
print(sys.version)
print(sys.path)
```
result
```
python version 2.7.1
%ZEPPELIN_HOME%/interpreter/python/lib:%PYTHON_HOME
```
```
import myModule
```
result
```
no module myModule
```

All three must be supported by conda.
1. Support to installing and using the python library
2. Support to change the python version
3. Support to configuring non-python environments

old zeppelin does works for this part(0.71).
but, the Python Interpreter has changed significantly since 0.71.
I guess missing the environment part for this.
I modified it.


python env set code

(https://github.com/apache/zeppelin/blob/master/python/src/main/java/org/apache/zeppelin/python/PythonInterpreter.java#L151)



---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-09 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
@zjffdu @felixcheung 
Thank you for your opinions :)
I was left my comment.
Could you check this one?





---

[GitHub] zeppelin issue #2575: [ZEPPELIN-2921] does not work conda environment in pyt...

2017-09-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2575
  
https://github.com/apache/zeppelin/pull/2574

same pr and different base branch


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
https://travis-ci.org/malayhm/zeppelin/jobs/272788659
https://travis-ci.org/1ambda/zeppelin/jobs/273195461



---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
```
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 12.456 sec 
<<< FAILURE! - in 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterServerTest

testStartStop(org.apache.zeppelin.interpreter.remote.RemoteInterpreterServerTest)
  Time elapsed: 10.037 sec  <<< FAILURE!
java.lang.AssertionError: expected: but was:
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterServerTest.testStartStop(RemoteInterpreterServerTest.java:62)
```

This error occurs equally in other PRs and branches.
It has been confirmed that it is not related to my changes.


---

[GitHub] zeppelin issue #2576: [hotfix] JDBC connection does not release when got exc...

2017-09-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2576
  
Tested :)
Thank you for good fix.


---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
@1ambda I'll check it, thank you :)


---

[GitHub] zeppelin pull request #2575: [ZEPPELIN-2921] does not work conda environment...

2017-09-08 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2575

[ZEPPELIN-2921] does not work conda environment in python interpreter

### What is this PR for?
It seems that the environment of the python interpreter has changed to the 
py4j environment, causing problems with the library path and the default 
environment.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2921

### How should this be tested?
Please run the following command line for each paragraph.
If the module such as scipy is normally imported, it is a success.
`%python.conda create --name Hello6 python=2.7`
`%python.conda activate Hello6`
`%python.conda install seaborn pandas numpy scipy matplotlib`
```
%python
import scipy as sp
import seaborn as sns
```
`%python.conda deactivate`

### Screenshots (if appropriate)
 Before

![image](https://user-images.githubusercontent.com/10525473/30199920-c75022ca-94af-11e7-8811-0c22310f1bac.png)

 After

![image](https://user-images.githubusercontent.com/10525473/30198880-23aaceb2-94ab-11e7-8bc6-bfad76c675f7.png)


### Questions:
* Does the licenses files need update? no 
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
fix/conda-interpreter-invalid-path-0.8

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2575.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2575


commit e2d2f53ec2de8c64d1df2cbf3e31ba7611509f74
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-09-08T06:23:11Z

add conda local env

commit a6b1a00a8cd944346a61484baf6b822b04d2a993
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-09-08T08:47:36Z

keep python binary




---

[GitHub] zeppelin issue #2574: [ZEPPELIN-2921 : 0.7x] does not work conda environment...

2017-09-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2574
  
@1ambda Thank you, and i will more test :)


---

[GitHub] zeppelin pull request #2574: [ZEPPELIN-2921 : 0.7x] does not work conda envi...

2017-09-08 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2574

[ZEPPELIN-2921 : 0.7x] does not work conda environment in python interpreter

### What is this PR for?
It seems that the environment of the python interpreter has changed to the 
py4j environment, causing problems with the library path and the default 
environment.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2921

### How should this be tested?
Please run the following command line for each paragraph.
If the module such as scipy is normally imported, it is a success.
`%python.conda create --name Hello6 python=2.7`
`%python.conda activate Hello6`
`%python.conda install seaborn pandas numpy scipy matplotlib`
```
%python
import scipy as sp
import seaborn as sns
```
`%python.conda deactivate`

### Screenshots (if appropriate)
 Before

![image](https://user-images.githubusercontent.com/10525473/30199920-c75022ca-94af-11e7-8811-0c22310f1bac.png)

 After

![image](https://user-images.githubusercontent.com/10525473/30198880-23aaceb2-94ab-11e7-8bc6-bfad76c675f7.png)


### Questions:
* Does the licenses files need update? no 
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
fix/conda-interpreter-invalid-path-0.7

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2574.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2574


commit b287825358ff05347d9062d352485e5cf02d23d7
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-09-08T06:23:11Z

add conda local env




---

[jira] [Created] (ZEPPELIN-2921) does not work conda environment in python interpreter

2017-09-08 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2921:
--

 Summary: does not work conda environment in python interpreter
 Key: ZEPPELIN-2921
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2921
 Project: Zeppelin
  Issue Type: Bug
Reporter: CloverHearts
Assignee: CloverHearts


does not work conda environment in python interpreter
It seems that the environment of the python interpreter has changed to the py4j 
environment, causing problems with the library path and the default environment.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin issue #2514: replace pegdown to markdown zeppelin interpreter

2017-08-09 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2514
  
https://travis-ci.org/cloverhearts/zeppelin

ci pass :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2514: replace pegdown to markdown zeppelin interpreter

2017-08-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2514
  
Sorry for the delay.
@Leemoonsoo 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2514: replace pegdown to markdown zeppelin interprete...

2017-08-02 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2514

replace pegdown to markdown zeppelin interpreter

### What is this PR for?
I was change markdown render librarry for Jupyter note convertor.
currently, we can got a same result for markdown.

### What type of PR is it?
Improvement

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2824

### How should this be tested?
1. build jupyter module
  `mvn clean package -DskipTests -pl 'zeppelin-jupyter' --am`
2. `cd zeppelin-jupyter/target`
3. `java -classpath zeppelin-jupyter-0.8.0-SNAPSHOT.jar 
org.apache.zeppelin.jupyter.JupyterUtil -i {your ipynb note file 
path!/getting_started.ipynb`
(good sample : [go to 
sample](https://github.com/SciRuby/sciruby-notebooks/blob/master/getting_started.ipynb)
4. get a `note.json` and import to zeppelin on frontend!
5. enjoy
### Screenshots (if appropriate)

 problem

![28689484-9b13f3d2-72ca-11e7-9bda-02d33b30f036](https://user-images.githubusercontent.com/10525473/28861908-0d05c592-779e-11e7-9a4e-94e3fd2bd176.png)

 after

![image](https://user-images.githubusercontent.com/10525473/28807730-029510e6-76b2-11e7-9111-0e18569b1630.png)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin ZEPPELIN-2824-2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2514.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2514


commit 2e8b3326b6fe3fb017f78cdfd6cecf1135484706
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-08-02T07:12:45Z

replace pegdown to markdown zeppelin interpreter




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (ZEPPELIN-2824) We have to do the same with Zeppelin Markdown in Jupyter Convertor.

2017-08-01 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2824:
--

 Summary: We have to do the same with Zeppelin Markdown in Jupyter 
Convertor.
 Key: ZEPPELIN-2824
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2824
 Project: Zeppelin
  Issue Type: Bug
Reporter: CloverHearts


The markdown of the Zeppelin note changed in the Jupyter note convertor does 
not match the results of some Zeppelin.
This is because Zeppelin uses some user change plugin.
This is confusing.
We need to make this same change.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin issue #2509: [ZEPPELIN-2818] Improve to better rendering from jupyt...

2017-07-31 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2509
  
https://travis-ci.org/cloverhearts/zeppelin

ci pass :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2509: [ZEPPELIN-2818] Improve to better rendering from jupyt...

2017-07-31 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2509
  
And Zeppelin changed Markdown4j to the default interpreter Pegdown to rende.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2509: [ZEPPELIN-2818] Improve to better rendering from jupyt...

2017-07-31 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2509
  
I was Improved test case.
and I was fixed command line bug. 
(https://github.com/apache/zeppelin/pull/2509/files#diff-fde78c9194e779309be4036e9a31904cR187)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2510: [ZEPPELIN-2756] Support ansi escape code for colorizin...

2017-07-30 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2510
  
AWESOME!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2509: [ZEPPELIN-2818] Improve to better rendering from jupyt...

2017-07-28 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2509
  
@Leemoonsoo 

## Original Markdown
```
  ---
keyword   text
  ---
red   Sunsets, apples, and
  other red or reddish
  things.

green Leaves, grass, frogs
  and other things it's
  not easy being.
  ---
```

## Jupyter
```

keyword text

  red   Sunsets, apples, and
  other red or reddish
  things.


  green Leaves, grass, frogs
  and other things it's
  not easy being.


```
## Pagedown
```

  keyword text
  âââ ââââââââ
  red Sunsets, apples, and 
  other red or reddish things.

```

## Markdown4j
```

keyword   text
red   Sunsets, apples, and
  other red or reddish
  things.

green Leaves, grass, frogs


and other things it's
not easy being.
  


```

I did some testing.
And some of the Markdown documents I used as samples
Sometimes there are grammars close to exceptions.

When I checked in jupyter,
If you try to apply a syntax close to the exception,
The structure of the changed html document is most similar to Markdown4j.
(actually, some element is little different)
If the structure of the document is similar, it is easy to implement the 
same experience as Jupyter in `LooknFeel`, so I chose Markdown4j which showed 
the most similar results.

one problem is that if you run the markdown paragraph with the default 
interpreter,
some results may look different.

What do you think?




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (ZEPPELIN-2818) Improve to better rendering from jupyter note

2017-07-28 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2818:
--

 Summary: Improve to better rendering from jupyter note
 Key: ZEPPELIN-2818
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2818
 Project: Zeppelin
  Issue Type: Improvement
Reporter: CloverHearts
Assignee: CloverHearts


Zeppelin currently has a library to convert and use the Jupiter Notes.
However, there is a problem with paragraphs in some notes.
We need to improve this for maximum data validation and experience.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin issue #2343: [ZEPPELIN-2538] JDBC completer improvements for work w...

2017-05-31 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2343
  
If there is no more comment on this, I 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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2343: [ZEPPELIN-2538] JDBC completer improvements for work w...

2017-05-24 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2343
  
I will test on this pr.
Thank you for greate feature. :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2353: [branch-0.7] missed try catch.

2017-05-21 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2353


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2353: [branch-0.7] missed try catch.

2017-05-21 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2353
  
merge


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2353: missing try catch

2017-05-19 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2353

missing try catch

### What is this PR for?
Currently we can not build on branch-0.7
The cause is a missed try catch.
So, I fixed this part.

### What type of PR is it?
Hot Fix

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin missing-try-catch

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2353.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2353


commit 8a9ea5ce79cca0a7909fd2b6b6242fd3a0b272d4
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-05-19T10:24:31Z

missing try catch




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2298: [ZEPPELIN-2466] Chart resize problem

2017-05-05 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2298


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2288: [ZEPPELIN-2452] block update paragraph event on revisi...

2017-05-01 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2288
  
if no more opinions, i will merge.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2289: [ZEPPELIN-2452] Can not handle message in revision mod...

2017-05-01 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2289
  
if no more opinions, i will merge.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2298: [ZEPPELIN-2466] Chart resize problem

2017-04-28 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2298
  
Fixed nvd3 basic resize event.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2298: [ZEPPELIN-2466] Chart resize problem

2017-04-28 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2298

[ZEPPELIN-2466] Chart resize problem

### What is this PR for?
If you change the size of the chart after changing the mode of the chart,
The chart size is not reflected.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2466

### How should this be tested?
1. resize for your browser.
2. chart toggle -> for example, bar chart -> table -> bar chart
3. and try resize for browser.

### Screenshots (if appropriate)
 bug

![resize-inc](https://cloud.githubusercontent.com/assets/10525473/25518072/3f8fbb4c-2c2d-11e7-94a4-e051f1a1e382.gif)

 fixed

![resize-c](https://cloud.githubusercontent.com/assets/10525473/25518077/4213bfd0-2c2d-11e7-8f9c-13fd5e8d2806.gif)


### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin fix/resize-problem-chart

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2298.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2298


commit 1dcf660e95f49abac4156a0ee7026870c9f1606c
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-28T07:06:34Z

fix resize problem

commit 13dee7630a28544633cf0312af2b0fec9948b542
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-28T07:11:49Z

remove semi colon




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (ZEPPELIN-2466) Chart resize problem

2017-04-28 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2466:
--

 Summary: Chart resize problem
 Key: ZEPPELIN-2466
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2466
 Project: Zeppelin
  Issue Type: Bug
Reporter: CloverHearts
Assignee: CloverHearts


If you change the size of the chart after changing the mode of the chart,
The chart size is not reflected.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin issue #2288: [ZEPPELIN-2452] block update paragraph event on revisi...

2017-04-27 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2288
  
CI PASS :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1218: [Zeppelin-1213] Customize editor configuration

2017-04-27 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1218
  
@hyonaldo 
Sorry,
We can not support this feature on zeppelin.
I do not care for this feature into master.
It also differs from the current source in many ways.
Re-implementation is required.
I have no plans to reimplement this until now.
Perhaps, if you want this feature.
You can create an issue in our jira. 
(https://issues.apache.org/jira/browse/ZEPPELIN)
Sorry, 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2257: [Branch-0.7] Support for Zeppelin Context redef...

2017-04-27 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2257


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2289: [ZEPPELIN-2452] Can not handle message in revision mod...

2017-04-27 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2289
  
CI PASS :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2289: [ZEPPELIN-2452] Can not handle message in revision mod...

2017-04-27 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2289
  
@khalidhuseynov 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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2289: [ZEPPELIN-2452] Can not handle message in revis...

2017-04-26 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2289


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2288: [ZEPPELIN-2452] block update paragraph event on...

2017-04-26 Thread cloverhearts

GitHub user cloverhearts reopened a pull request:

https://github.com/apache/zeppelin/pull/2288

[ZEPPELIN-2452] block update paragraph event on revision mode

### What is this PR for?
In revision mode using git-repository, by default all paragraphs must 
remain unmodifiable.
However, we are currently performing an incorrect update.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2452

### How should this be tested?
1. create note and modify to paragraph and version control commit on web
2. modify to paragraph and execute
3. open your 2 browser - one browser is HEAD, and other browser move to 
before commit
4. insert paragarph or modify and execute.
check update

### Screenshots (if appropriate)

 problem

![incorrectrevisionupdate](https://cloud.githubusercontent.com/assets/10525473/25425492/1a0ebcce-2aa7-11e7-9a06-cfc84a1c1fe3.gif)

 fixed (this pr)

![correctrevisionupdate](https://cloud.githubusercontent.com/assets/10525473/25425498/1dce1bfc-2aa7-11e7-816c-c25a64963475.gif)



### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
fix/invalidupdateonrevision

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2288.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2288


commit 19e747999146af56b803696561001977d786032d
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-26T08:30:43Z

block update paragraph event on revision mode




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2288: [ZEPPELIN-2452] block update paragraph event on...

2017-04-26 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/2288


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (ZEPPELIN-2455) Does not work personalized mode on zeppelin

2017-04-26 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2455:
--

 Summary: Does not work personalized mode on zeppelin
 Key: ZEPPELIN-2455
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2455
 Project: Zeppelin
  Issue Type: Bug
Reporter: CloverHearts
Priority: Critical
 Attachments: incooo.gif

Does not work personalized mode on zeppelin

how to reproduce
1. create a note
2. With each different browser, log in as a different user.
3. A and B users.
4. Activate personalized as user A.
5. Run paragraph A as user.
6. The code is shared, but the results are not shared.
7. Modify and execute the paragraph with the B user.
8. All changes are delivered to the A user. (Problem)

I was Attach an action screenshot.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin pull request #2289: [ZEPPELIN-2452] Can not handle message in revis...

2017-04-26 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2289

[ZEPPELIN-2452] Can not handle message in revision mode on web.

### What is this PR for?
Requests interpreter editor mode information in revision mode.
However, no information is needed in revision mode.
Produces misleading logs with misleading requests to the server.
```
ERROR [2017-04-26 18:07:48,718] ({qtp1273765644-60} 
NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException
ERROR [2017-04-26 18:07:49,081] ({qtp1273765644-109} 
NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException
ERROR [2017-04-26 18:07:49,083] ({qtp1273765644-14} 
NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException
```

This log is requested as many as the number of paragraphs contained in the 
note.


### What type of PR is it?
Bug fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2452

### How should this be tested?
1. Create a git-repo based note. (Supported by default)
2. Modify the paragraph and commit.
3. Go to the committed revision.
4. Reload the page.
5. Check the server logs.

NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
fix/incorrectcall-geteditorsetting-on-revisionmode

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2289.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2289


commit 8c77f769826d239bcfaf2ddf1ed6cf4771347b63
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-26T09:10:18Z

fix call geteditorsetting




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2288: [ZEPPELIN-2452] block update paragraph event on...

2017-04-26 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2288

[ZEPPELIN-2452] block update paragraph event on revision mode

### What is this PR for?
In revision mode using git-repository, by default all paragraphs must 
remain unmodifiable.
However, we are currently performing an incorrect update.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2452

### How should this be tested?
1. create note and modify to paragraph and version control commit on web
2. modify to paragraph and execute
3. open your 2 browser - one browser is HEAD, and other browser move to 
before commit
4. insert paragarph or modify and execute.
check update

### Screenshots (if appropriate)

 problem

![incorrectrevisionupdate](https://cloud.githubusercontent.com/assets/10525473/25425492/1a0ebcce-2aa7-11e7-9a06-cfc84a1c1fe3.gif)

 fixed (this pr)

![correctrevisionupdate](https://cloud.githubusercontent.com/assets/10525473/25425498/1dce1bfc-2aa7-11e7-816c-c25a64963475.gif)



### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
fix/invalidupdateonrevision

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2288.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2288


commit 19e747999146af56b803696561001977d786032d
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-26T08:30:43Z

block update paragraph event on revision mode




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (ZEPPELIN-2454) broken paragraph or note

2017-04-26 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2454:
--

 Summary: broken paragraph or note
 Key: ZEPPELIN-2454
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2454
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: CloverHearts
Assignee: CloverHearts


When you run a note, the paragraph is sometimes blank.
Sometimes the order changes.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (ZEPPELIN-2453) call incorrect websocket event on revision mode

2017-04-26 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2453:
--

 Summary: call incorrect websocket event on revision mode 
 Key: ZEPPELIN-2453
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2453
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: CloverHearts
Assignee: CloverHearts


When approaching Zeppelin in revision mode, it calls the wrong web socket event 
as many as the number of paragraphs.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (ZEPPELIN-2452) revison mode incorrect update

2017-04-26 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2452:
--

 Summary: revison mode incorrect update
 Key: ZEPPELIN-2452
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2452
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: CloverHearts


Note Update events are performed in revision mode.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin issue #2278: [ZEPPELIN-2431] Corrected deletion of notes by incorre...

2017-04-25 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2278
  
if no more opinions, i will merge.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2278: [ZEPPELIN-2431] Corrected deletion of notes by incorre...

2017-04-24 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2278
  
added screenshot



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2278: [ZEPPELIN-2431] Corrected deletion of notes by ...

2017-04-24 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2278

[ZEPPELIN-2431] Corrected deletion of notes by incorrect interpreter.json

### What is this PR for?

We sometimes can not delete a note, or we will be accompanied by an NPE for 
deleting a note.

This problem occurs when:
When interpreter.json 's note binding is wrong, or there is a problem.
If you are configuring an interpreter that is not through zeppelin's user 
interface.
As a result, it happens when synchronization of notes deletion and setting 
retention is not normal.
Therefore, we should add handling for note deletion and exception handling 
for nonexistent interpreter bindings.
It reduces the synchronization problem of interpreter.json.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2431

### How should this be tested?
1. zeppelin stop
2. edit con/interpreter.json and `interpreterBindings`
  fix any notes or incorrect information.
  for example 
  ```
},  
  "interpreterBindings": {
"2CFS9YSM5": [
  "2CFRR1D3TINVALIDINVALIDINVALID", <-- edit
  "2CFZ1JKUR",
  "2CEAJK1VW",
  "2CGSESWBH",
  "2CERNPGW5",
   }
  ```
3. zeppelin start
4. You can try remove invalid interpreter bind note on web. (on example = 
`2CFS9YSM5`)

result : 
If the modifications to this PR are not reflected,
It will not be deleted or an error will appear on the server.
Also, the interpreterBindings information in interpreter.json does not 
respond to delete events.

### Screenshots (if appropriate)


### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
fix/invalidsyncInterpreterJson

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2278.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2278

----
commit 35da524cc00c40531072130424fb38419e4612f4
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-24T07:16:47Z

notebook interpreter binding synchronization process




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (ZEPPELIN-2431) Raise Exception when remove note in folder

2017-04-20 Thread CloverHearts (JIRA)

CloverHearts created ZEPPELIN-2431:
--

 Summary: Raise Exception when remove note in folder
 Key: ZEPPELIN-2431
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2431
 Project: Zeppelin
  Issue Type: Bug
Reporter: CloverHearts
Assignee: CloverHearts


When remove note in folder, raise exception on zeppelin server
```
ERROR [2017-04-20 18:53:56,917] ({qtp1459672753-16} 
NotebookServer.java[onMessage]:357) - Can't handle message
java.lang.NullPointerException
at 
org.apache.zeppelin.interpreter.InterpreterSettingManager.removeInterpretersForNote(InterpreterSettingManager.java:806)
at 
org.apache.zeppelin.interpreter.InterpreterSettingManager.removeNoteInterpreterSettingBinding(InterpreterSettingManager.java:895)
at org.apache.zeppelin.notebook.Notebook.removeNote(Notebook.java:341)
at 
org.apache.zeppelin.socket.NotebookServer.removeNote(NotebookServer.java:973)
at 
org.apache.zeppelin.socket.NotebookServer.onMessage(NotebookServer.java:232)
at 
org.apache.zeppelin.socket.NotebookSocket.onWebSocketText(NotebookSocket.java:59)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextMessage(JettyListenerEventDriver.java:128)
at 
org.eclipse.jetty.websocket.common.message.SimpleTextMessage.messageComplete(SimpleTextMessage.java:69)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.appendMessage(AbstractEventDriver.java:65)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextFrame(JettyListenerEventDriver.java:122)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.incomingFrame(AbstractEventDriver.java:161)
at 
org.eclipse.jetty.websocket.common.WebSocketSession.incomingFrame(WebSocketSession.java:309)
at 
org.eclipse.jetty.websocket.common.extensions.ExtensionStack.incomingFrame(ExtensionStack.java:214)
at 
org.eclipse.jetty.websocket.common.Parser.notifyFrame(Parser.java:220)
at org.eclipse.jetty.websocket.common.Parser.parse(Parser.java:258)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.readParse(AbstractWebSocketConnection.java:632)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.onFillable(AbstractWebSocketConnection.java:480)
at 
org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
at java.lang.Thread.run(Thread.java:745)
```



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin pull request #2257: [Branch-0.7] Fixes a build problem in Zeppelin ...

2017-04-17 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2257

[Branch-0.7] Fixes a build problem in Zeppelin branch0.7.

Fixes a build problem in Zeppelin branch0.7.
Hotfix.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
fix/branch-0.7-build-hotfix

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2257.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2257


commit a3a615c3bf0be92089d03105b0fad96a05b66940
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-12T08:14:36Z

[Zeppelin-802] Support for Zeppelin Context redefinition on Python and 
Pyspark

If you override the reserved word ZeppelinContext such as `z` in the python 
language, the whole paragraph output problem occurred.
I have taken care to avoid this issue.

`z` == `_zc` == `zeppelin context`

Improvement

https://issues.apache.org/jira/browse/ZEPPELIN-802

The error should not occur in the following situations:
```
%python
z = 1
print("Hello Zeppelin")
```

```
%pyspark
z = 1
print("Hello Zeppelin")
```

![replace zeppelin 
context-err](https://cloud.githubusercontent.com/assets/10525473/24521772/319946be-15c8-11e7-96cf-7fdf41c70a66.png)

![replace zeppelin 
context](https://cloud.githubusercontent.com/assets/10525473/24521775/349fa7cc-15c8-11e7-8fe4-4f3f5597deff.png)

* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
    
Author: CloverHearts <cloverhearts...@gmail.com>
Author: cloverhearts <cloverhearts...@gmail.com>
    
Closes #2207 from cloverhearts/ZEPPELIN-802-pyspark-zeppelin-context and 
squashes the following commits:

cc986010 [CloverHearts] added completion on namespace
14695cb8 [CloverHearts] Recovering a member name that is not associated 
with a namespace
31af92ab [CloverHearts] fix test case _zc to __zeppelin__
6697d677 [CloverHearts] apply to namespace and replace name _zc to 
__zeppelin__
ca795cff [cloverhearts] replace output
1d372df4 [cloverhearts] change name logger
4e8435ac [CloverHearts] added test case on python
b6b804ad [CloverHearts] replace name zeppelin context on python
9fbf70d6 [CloverHearts] fix pyspark test case
987e2118 [CloverHearts] added test code
5da3d6ed [CloverHearts] replace name zeppelin context on pyspark

(cherry picked from commit 8d03920b9bec86dd8e2fc343d32749a08f501362)
Signed-off-by: CloverHearts <cloverhearts...@gmail.com>

commit 2da0b423bb3075d6b960d2f8dbb5466f5f11b733
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-04-17T13:39:08Z

fix branch-0.7 build error




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-13 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
will be merging if no further 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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
@zjffdu 
How to use is the same as the existing one.
If I add a document, should I add something like __zeppelin__ or __spark__?
What do you think? :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
I would like to merge this.
This solves the problem of usability.
I will merge this if I no longer have any opinion.
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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
Okay, Ci pass :)
@felixcheung @zjffdu 
Could you please check on this pr?
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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
and tested environment.


| | pyspark intp | python intp |
|-||--|
|python3 with matplotlib | O| O |
|python2 with matplotlib | O | O |



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
apply to namespace on python

and zeppelin context name change

before
```
z = _zc = zeppelin context
```
to
```
z = __zeppelin__ = zeppelin context
```

same
```
z.input("title", "my title")
__zeppelin__.input("title", "my title")
```


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-11 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
@zjffdu 
Yes, I will also reflect the namespace.
Thank you for your advice :)
.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-03-31 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
@felixcheung Thank you for your good advice.
I will check your opinion.
and fix :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-03-30 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
@astroshim 
So what about `__zeppelin__`?
Actually, I feel similar like a python system environment word
It is less likely to be redefined by other libraries.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2207: [Zeppelin-802] Support for Zeppelin Context red...

2017-03-30 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2207

[Zeppelin-802] Support for Zeppelin Context redefinition on Python and 
Pyspark 

### What is this PR for?
If you override the reserved word ZeppelinContext such as `z` in the python 
language, the whole paragraph output problem occurred.
I have taken care to avoid this issue.

`z` == `_zc` == zeppelin context


### What type of PR is it?
Improvement

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-802

### How should this be tested?
The error should not occur in the following situations:
```
%python
z = 1
print("Hello Zeppelin")
```

```
%pyspark
z = 1
print("Hello Zeppelin")
```


### Screenshots (if appropriate)

 before
![replace zeppelin 
context-err](https://cloud.githubusercontent.com/assets/10525473/24521772/319946be-15c8-11e7-96cf-7fdf41c70a66.png)

 after
![replace zeppelin 
context](https://cloud.githubusercontent.com/assets/10525473/24521775/349fa7cc-15c8-11e7-8fe4-4f3f5597deff.png)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
ZEPPELIN-802-pyspark-zeppelin-context

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2207.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2207


commit 7edb1ad4196f4d9e9791545ad25980727ad418db
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:26:07Z

replace name zeppelin context on pyspark

commit 2d0ea1a030109243a7d530d0e88793859b3f9a14
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:35:28Z

added test code

commit b54209da0bd60ce0ae1e46a41ed1695e5fa442b8
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:44:53Z

fix pyspark test case

commit 0eaf9a896c30037dd4e51dc18df0a2c5b9abf69c
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:46:08Z

replace name zeppelin context on python

commit f3805b3b73c533fc8dabcc39c15dfcecae0259a4
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:46:27Z

added test case on python




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2187: [ZEPPELIN-2314] Fix watcher websocket origin header

2017-03-26 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2187
  
Tested!

Thank you for fix.
That's well working!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2140: [HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] "Restart" butto...

2017-03-15 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2140
  
@jongyoul Tested.

Spark and Markdown and Misc interpreter (without shiro)
```
restart interpreter on web ui > restart feature well work.
zeppelin shutdown (daemon stop) --->  well working, I can't reproduce with 
zombie process. :)
```
Thank you for good fix :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2035: [ZEPPELIN-2049 : z0.7.1] Note name is gone when switch...

2017-02-18 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2035
  
@jongyoul @Leemoonsoo 
Could you review on 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2035: [ZEPPELIN-2049 : z0.7.1] Note name is gone when...

2017-02-18 Thread cloverhearts

GitHub user cloverhearts opened a pull request:

https://github.com/apache/zeppelin/pull/2035

[ZEPPELIN-2049 : z0.7.1] Note name is gone when switch to personal mode

### What is this PR for?
Note name is gone when switch to personal mode
There was a part of the note name that was missing processing.
So, I fixed it.


### What type of PR is it?
Bug Fix - 0.7.1 related

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2049

### How should this be tested?
You can personalized mode.
and then plase refresh note view.
and check to note name.

### Screenshots (if appropriate)
 before(problem)

![screenshot](https://issues.apache.org/jira/secure/attachment/12850799/record.gif)

 after (resolved)

![personal](https://cloud.githubusercontent.com/assets/10525473/23093214/31f0b0a6-f620-11e6-89ce-fabafa7c.gif)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin ZEPPELIN-2049

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2035.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2035


commit eefd7fb0020c1f154b1652eace239f28efbd1e56
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-02-18T12:04:31Z

missing get note name




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2034: [ZEPPELIN-2133] All interpreters sometimes throw rando...

2017-02-18 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2034

> Leemoonsoo 3 hours ago Member
I think void close(String sessionId) supposed not necessarily terminate
process while an interpreter process can have multiple sessions.

I also agree with @Leemoonsoo's opinion.
The "close () method" should only serve to close the session.
"Close () method" To force a process termination call,
It is very effective in this issue, but it has the potential to become a
problem in the future.
(Especially about user management functions or roles for sessions)

**In my personal opinion, we need to identify this as an increase and a
decrease in the erroneous reference.**
In the past we actually had a separate code to forcefully terminate the
"Remote Interpreter".

Ref Code (destroy) :
https://github.com/apache/zeppelin/blob/branch-0.6/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/InterpreterGroup.java#L204
remove PR :
https://github.com/apache/zeppelin/commit/c6c67d7b112eceb36846e03207cb9c37519f1730

Actually this is very similar to the current code.
Actually, this problem is almost solved by recovering to the point of the
code of the corresponding PR or adding the content of the problem.
However, in my opinion, the essence of this problem is not a "zombie
process"
I think the **session management is wrong.**
If you do not know what to do with the destroy () method, you can simply
restore it.
But, in my humble opinion, the deletion of the "destory () method" is not
the wrong way.
Of course I also need more specific research on this part.

what do you think?

Sorry. @prabhjyotsingh
I think you presented a good solution.
But I think I have to approach this problem carefully.
Please tell us your opinion at any time.
Thank you. Have a good day.

---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

2017-02-09 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1886
  
@sotnich 
now Well working!
good works!
Thank you for great feature!


![mysql](https://cloud.githubusercontent.com/assets/10525473/22814715/2e2e814c-ef99-11e6-8b3a-f292554ea075.gif)



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-02-08 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1976
  

Thank you for your reply, @del680202 
I understand your publicity purpose.
And I think it's right.
What I am worried call "terminateRemoteProcess ();" From "public void close 
(String sessionId) in InterpreterGroup.java".
"Close (String sessionId)" should only perform the action corresponding to 
the session.
And if all sessions are closed, the process must be terminated.
However, the current code always performs "terminateRemoteProcess ();" 
Within "close (String sessionId)".
"TerminateRemoteProcess ();" The function forces all sessions to be 
terminated and the process is terminated.
I think this is the wrong behavior of "close (String sessionId)".
We think this can be a problem, because we can write several users with an 
interpreter.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-02-07 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1976
  
@jongyoul sure :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

2017-02-03 Thread cloverhearts

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1886
  

![auto-com](https://cloud.githubusercontent.com/assets/10525473/22586604/05b60ba2-ea41-11e6-9842-daf768b3c9c1.gif)

Hello, I was check again.
but, does not work on mysql.
Is there anything I missed?



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1930: [ZEPPELIN-1886] implementation z.getZeppelinJob...

2017-02-01 Thread cloverhearts

GitHub user cloverhearts reopened a pull request:

https://github.com/apache/zeppelin/pull/1930

[ZEPPELIN-1886] implementation z.getZeppelinJobStatus 

### What is this PR for?
You can get the state of a paragraph through ZeppelinContext.
This allows you to implement code according to the paragraph condition.

related
https://github.com/apache/zeppelin/pull/1799

### What type of PR is it?
Feature

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1886

### How should this be tested?

1. run paragraph.
2. Record the noteid and paragraph id of the paragraph you have run.
3. You can use the following code:
for example
```
%spark
z.getJobStatus("noteID", "paragraphId").name()
z.getJobStatus("2C8D89Y9N", "20170121-042805_156018837").isFinished()
z.getJobStatus("2C8D89Y9N", "20170121-042805_156018837").isAbort()
z.getJobStatus("2C8D89Y9N", "20170121-042805_156018837").isError()
z.getJobStatus("2C8D89Y9N", "20170121-042805_156018837").isPending()
z.getJobStatus("2C8D89Y9N", "20170121-042805_156018837").isRunning()
```

### Screenshots (if appropriate)

![zeppelinstatus](https://cloud.githubusercontent.com/assets/10525473/22175433/770168b2-dfa9-11e6-8471-5dfa92e56eaa.png)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? yes


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloverhearts/zeppelin 
ZEPPELIN-GET-PARAGRAPH-STATUS

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/1930.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1930


commit 43c9ea5051745a013fd78d5a9497e960d42c6570
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2016-12-30T22:29:38Z

extends z.status on RemoteWorks

commit f0e9825e0e5867b750a8b386c0b82233b3915302
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-01T05:53:09Z

implement check status logic in workflow

commit 44fd762e1e9646deb0fdccfec7e3cabaa2042bfb
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-21T15:10:10Z

add status getter

commit 7eac7572dcaf81f9b14a255f531fb7df7d465d7e
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-21T20:08:42Z

change method name

commit 01614e6be21f7d08038db55b38a6da58744df505
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-21T20:55:01Z

added test case

commit cdbdb4ce51453239323e228847f7b6d87cbc2bb6
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-21T21:43:09Z

remove â

commit b9948df748281579c1998434915b0a9e29dd274d
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-23T22:03:14Z

change test case for println

commit 7ecde8aeca227ff43c7ac2c5bb139dfedbeee4b6
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-23T22:57:53Z

remove println in testcase

commit a6b1719594c481c9236eea639b419526ab7918db
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-24T02:40:24Z

remove "

commit 0482333c3394b878dec5cdb9f60322c6d19a2de5
Author: cloverhearts <cloverhearts...@gmail.com>
Date:   2017-01-24T07:13:05Z

remove system.out




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1930: [ZEPPELIN-1886] implementation z.getZeppelinJob...

2017-02-01 Thread cloverhearts

Github user cloverhearts closed the pull request at:

https://github.com/apache/zeppelin/pull/1930


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

1 2 3 4 5 6 >

1 - 100 of 556 matches

Mail list logo