[GitHub] zeppelin pull request #1834: [ZEPPELIN-1864] Improvement to show folder and ...

2017-01-10 Thread soralee
GitHub user soralee reopened a pull request:

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

[ZEPPELIN-1864] Improvement to show folder and note after searching note

### What is this PR for?
This PR is for improvement to show folder and note after using the filter. 
And I found some bugs and fixed like this.

the following list is improvement and bug fixed.
1. After using the filter in Zeppelin Home, every note is shown by the form 
[FolderName/NoteName] like below. It would be nice to show folder icon and note 
icon as previous status.

![z1864_f_b](https://cloud.githubusercontent.com/assets/8110458/21604777/79dbb228-d1e8-11e6-974d-737520729d68.gif)
 
2. When using the filter, some functions and icons next to note and folder 
are disappeared.

![z1864_f_2_b](https://cloud.githubusercontent.com/assets/8110458/21605057/f5a0143e-d1e9-11e6-86f3-ebff5be4c41d.gif)

3. When using the filter, the `Rename Note` function doesn't work. ( When I 
was handling this PR, this bug was discovered.)

![z1864_f_4_b](https://cloud.githubusercontent.com/assets/8110458/21606384/71305796-d1f2-11e6-91a4-2ec14b8b4959.gif)


### What type of PR is it?
[ Bug Fix | Improvement ]

### Todos
* [x] - improve to show folder and note when finishing to use filter.
* [x] - some functions and icons next to note and folder are appeared when 
using filter.
* [x] - fix `Rename Note` function when using the filter.

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

### How should this be tested?
 - **[Test 1]** Before/After using the filter, check the list structure of 
your notes. It must be same appearance.

 - **[Test 2]** When using the filter, check the notes. Some icons and 
functions next to note and folder must be appeared.

 - **[Test 3]** When using the filter, click `Rename Note`. It must be 
appeared the path of Note.

### Screenshots (if appropriate)
**Test 1**.

![z1864_f_1_a](https://cloud.githubusercontent.com/assets/8110458/21606849/a79dbf82-d1f5-11e6-8d48-3a1977099ab1.gif)

**Test 2**.

![z1864_f_2_a](https://cloud.githubusercontent.com/assets/8110458/21606880/d1cf9550-d1f5-11e6-9311-ee54424d7e76.gif)

**Test 3**.

![z1864_f_4_a](https://cloud.githubusercontent.com/assets/8110458/21606272/9e0b096a-d1f1-11e6-8bfb-ab1ea1a30b88.gif)


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


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

$ git pull https://github.com/soralee/zeppelin ZEPPELIN-1864

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

https://github.com/apache/zeppelin/pull/1834.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 #1834


commit 69d6d6d710c72e07d28e0c29978fe24e15002f8d
Author: soralee 
Date:   2017-01-03T09:35:55Z

Improvement to show folder and note after searching note

commit 61680e5f814ca67dc015406852315a1a92d4e7a8
Author: soralee 
Date:   2017-01-03T11:51:31Z

Improvement to show folder and note after searching note

commit 3e4b8ae80f53e9c7b4709bb41a65ac31b0f62fb4
Author: soralee 
Date:   2017-01-09T09:19:11Z

fix for that filter of navbar and home don't work

commit fd2c2438d6fbc3880194cd6c6e3ed1993b71f3a0
Author: soralee 
Date:   2017-01-11T06:04:53Z

resolve conflict

commit 6fb53aa420e39ee9144513d2b68521bbac1f83a5
Author: soralee 
Date:   2017-01-11T06:26:23Z

resolve conflict




---
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 #1834: [ZEPPELIN-1864] Improvement to show folder and ...

2017-01-10 Thread soralee
Github user soralee closed the pull request at:

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


---
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 #1834: [ZEPPELIN-1864] Improvement to show folder and note af...

2017-01-10 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1834
  
@AhyoungRyu Yes, you're right. 
I mean, If I change **navbar** folder option, it works together both the 
filter in the **home** and **navbar**. So, I did not include this part because 
I think this improvement should be opened other PR.

Thanks for review 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 issue #1881: ZEPPELIN-1935 Add jceks stored password support for jd...

2017-01-10 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1881
  
Got 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 #1830: ZEPPELIN-1867. Update document for pig interpreter and...

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1830
  
Thanks! Merge into master if there are no more comments.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have 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 #1884: [ZEPPELIN-1936] Fix user impersonation setting ...

2017-01-10 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request:

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

[ZEPPELIN-1936] Fix user impersonation setting propagation 

### What is this PR for?
This is to fix bug of creating interpreter with impersonation enabled


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

### Todos
* [x] - fix model

### What is the Jira issue?
[ZEPPELIN-1936](https://issues.apache.org/jira/browse/ZEPPELIN-1936)

### How should this be tested?
create new interpreter with "User Impersonation" enabled

### Screenshots (if appropriate)
before:

![before_imp_setting](https://cloud.githubusercontent.com/assets/1642088/21839491/e187fa18-d78c-11e6-864d-755fc40f0096.gif)

after:

![after_imp_setting](https://cloud.githubusercontent.com/assets/1642088/21839721/2756a7f0-d78e-11e6-9efd-4f3fd29df616.gif)


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


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

$ git pull https://github.com/khalidhuseynov/incubator-zeppelin 
fix/new-interp-impersonation-setting

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

https://github.com/apache/zeppelin/pull/1884.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 #1884


commit d5f839396be6bec9adf74bc37db10971a4eac735
Author: Khalid Huseynov 
Date:   2017-01-11T07:22:35Z

fix model on interpreter-create




---
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 #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1758
  
@AlexanderShoshin Checked again and it looks nice. Except the minor 
suggestion, LGTM. Thanks for your effort and the awesome tutorial note! 



---
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.
---


Re: branch-0.7 cut

2017-01-10 Thread Mina Lee
I would like to inform you that branch-0.7 has cut.
You can check it through apache git repository URL [1] but it seems like it
takes some time for Github mirror.

PMCs/Commiters, please note that you need to merge bug fix both to master
and branch-0.7
if it needs to be included in 0.7.x release.

Thanks,
Mina

[1] https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=summary

On Wed, Jan 11, 2017 at 3:09 PM Hyung Sung Shim  wrote:

> Thanks Mina! 
>
> 2017-01-10 18:23 GMT-08:00 Mina Lee :
>
> > Moon, thank you for reminding.
> > I think it would be beneficial to community to provide public docker
> image.
> > I will definitely take it into account as infra part already has been
> taken
> > cared and Dockerfile is almost ready.
> >
> > On Wed, Jan 11, 2017 at 4:10 AM moon soo Lee  wrote:
> >
> > > Thanks Mina taking care of it. Sounds good to me.
> > >
> > > One thing is, shell we try have docker image in this release?
> > > Following issues tracks the the job
> > > https://issues.apache.org/jira/browse/ZEPPELIN-1711
> > > https://issues.apache.org/jira/browse/INFRA-12781
> > >
> > > I don't think it need to be a blocker for 0.7.0 release, but if they
> can
> > be
> > > resolved by the time we create release candidate and have docker image
> in
> > > the release, it would be great.
> > >
> > > Thanks,
> > > moon
> > >
> > > On Mon, Jan 9, 2017 at 10:24 PM Mina Lee  wrote:
> > >
> > > > Hi,
> > > >
> > > > It's been almost 3 month since we did 0.6.2 release and there were
> > number
> > > > of improvements and bug fixes.
> > > > Most of the items that we planned for 0.7.0 release [1] are ready
> > > > so I am planning to cut branch-0.7 on 10th Jan 11pm(PST).
> > > >
> > > > After branch cut, bug fixes, documentation patches and issues linked
> > as a
> > > > blocker in jira [2] will be merged both to branch-0.7, master branch
> > but
> > > > new features will be merged only to master branch.
> > > >
> > > > Thanks,
> > > > Mina
> > > >
> > > > [1]
> > > https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+Roadmap
> > > > [2] https://issues.apache.org/jira/browse/ZEPPELIN-1795
> > > >
> > >
> >
>


[GitHub] zeppelin issue #1830: ZEPPELIN-1867. Update document for pig interpreter and...

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1830
  
Thanks @AhyoungRyu , I rename the note, please check the updated screenshot 
in PR description. 


---
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-1936) User Impersonation setting isn't propagated when creating new interpreter

2017-01-10 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-1936:
-

 Summary: User Impersonation setting isn't propagated when creating 
new interpreter
 Key: ZEPPELIN-1936
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1936
 Project: Zeppelin
  Issue Type: Bug
Reporter: Khalid Huseynov
Assignee: Khalid Huseynov


When creating new interpreter with enabled "User Impersonation", the setting 
isn't applied on new interpreter.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1830: ZEPPELIN-1867. Update document for pig interpreter and...

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1830
  
Sorry for my late response. It looks nice! Except one minor suggestion, 
LGTM 👍 



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


[GitHub] zeppelin issue #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
LGTM. Merge to master and branch-0.7 if there is no more discussion


---
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 #1883: Bump up version to 0.8.0-SNAPSHOT

2017-01-10 Thread minahlee
GitHub user minahlee opened a pull request:

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

Bump up version to 0.8.0-SNAPSHOT

### What is this PR for?
Bump up version to 0.8.0-SNAPSHOT

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

$ git pull https://github.com/minahlee/zeppelin 0.8.0-SNAPSHOT

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

https://github.com/apache/zeppelin/pull/1883.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 #1883


commit 9511e4898168241a0dbe34a497b3878b575ebaa0
Author: Mina Lee 
Date:   2017-01-11T05:41:23Z

Bump up version to 0.8.0-SNAPSHOT




---
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 #1830: ZEPPELIN-1867. Update document for pig interpreter and...

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1830
  
ping @AhyoungRyu , please check the updated 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 issue #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
@minahlee PR is rebased. 


---
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 #1881: ZEPPELIN-1935 Add jceks stored password support for jd...

2017-01-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1881
  
@r-kamath IMO this shouldn't be called "backward compatibility" as these 
are two ways in which password can provided to interpreter.

Tested locally, works as expected, LGTM!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have 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 #1834: [ZEPPELIN-1864] Improvement to show folder and note af...

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1834
  
@soralee Thanks for the update and kind reminder! Let me test 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 issue #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
@felixcheung Let me update release script in another PR. @zjffdu do you 
mind to resolve conflict?


---
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 #1881: ZEPPELIN-1935 Add jceks stored password support for jd...

2017-01-10 Thread r-kamath
Github user r-kamath commented on the issue:

https://github.com/apache/zeppelin/pull/1881
  
@felixcheung JCEKS is used only when the password field is empty. When 
password and JCEKS properties are empty the connection attempt is made without 
password.


---
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 #1871: ZEPPELIN-1862 Move packages related Helium into...

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1846: ZEPPELIN-1770. Restart only the client user's interpre...

2017-01-10 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1846
  
Tests LGTM. Thanks!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 #1881: ZEPPELIN-1935 Add jceks stored password support for jd...

2017-01-10 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1881
  
this is important - is there any backward compatibility concerns with 
changing the password storage?


---
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 #1875: [ZEPPELIN-1926] Remove grunt googlefonts task

2017-01-10 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1875
  
hmm, it doesn't break RAT tests for all these binaries? :)



---
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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
great, we are good to go then, yeah?

@minahlee btw, should we update the hadoop-2.4 in the release script - we 
are testing against 2.6 in CI and almost everyone out there should be running 
at least 2.6??



---
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.
---


Re: branch-0.7 cut

2017-01-10 Thread Hyung Sung Shim
Thanks Mina! 

2017-01-10 18:23 GMT-08:00 Mina Lee :

> Moon, thank you for reminding.
> I think it would be beneficial to community to provide public docker image.
> I will definitely take it into account as infra part already has been taken
> cared and Dockerfile is almost ready.
>
> On Wed, Jan 11, 2017 at 4:10 AM moon soo Lee  wrote:
>
> > Thanks Mina taking care of it. Sounds good to me.
> >
> > One thing is, shell we try have docker image in this release?
> > Following issues tracks the the job
> > https://issues.apache.org/jira/browse/ZEPPELIN-1711
> > https://issues.apache.org/jira/browse/INFRA-12781
> >
> > I don't think it need to be a blocker for 0.7.0 release, but if they can
> be
> > resolved by the time we create release candidate and have docker image in
> > the release, it would be great.
> >
> > Thanks,
> > moon
> >
> > On Mon, Jan 9, 2017 at 10:24 PM Mina Lee  wrote:
> >
> > > Hi,
> > >
> > > It's been almost 3 month since we did 0.6.2 release and there were
> number
> > > of improvements and bug fixes.
> > > Most of the items that we planned for 0.7.0 release [1] are ready
> > > so I am planning to cut branch-0.7 on 10th Jan 11pm(PST).
> > >
> > > After branch cut, bug fixes, documentation patches and issues linked
> as a
> > > blocker in jira [2] will be merged both to branch-0.7, master branch
> but
> > > new features will be merged only to master branch.
> > >
> > > Thanks,
> > > Mina
> > >
> > > [1]
> > https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+Roadmap
> > > [2] https://issues.apache.org/jira/browse/ZEPPELIN-1795
> > >
> >
>


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

2017-01-10 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1831
  
**For reviewers**

Fixed to use `spark.jars` instead of `classpath`. 

- classpath doesn't include submitted jars at this moment (i could get 7 
days ago, but not now)
- it enable to simplify logic since we don't need to get classpath before 
setting new classpath. In other words, we can directly set PYTHONPATH in 
`setupPySparkEnv` function. 
- Also tested in spark 1.6.2, spark 2.0.0 

@zjffdu 

The spark code we talked came from 
https://github.com/apache/spark/pull/6360. It seems like intended so it's ok 
not to raise an issue.




---
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 #1845: [ZEPPELIN-1906] Use multiple InterpreterResult for dis...

2017-01-10 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1845
  
@prabhjyotsingh Yea I know you didn't change it. That's because it doesn't 
have state before.
I agree that we can take care/discuss it in a different PR.
Thanks.




---
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 #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-10 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1879
  
@soralee 
Sure :)

rebase is done.


---
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 #1840: [ZEPPELIN-1730, 1587] add spark impersonation through ...

2017-01-10 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1840
  
I just pushed changes to keep compatibility using 
`ZEPPELIN_IMPERSONATE_SPARK_PROXY_USER` env. variable that will disable usage 
of `--proxy-user` option.  after 
[SPARK-19143](https://issues.apache.org/jira/browse/SPARK-19143) resolved, 
maybe can come back to it 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 issue #1842: [ZEPPELIN-1619] Load js package as a plugin visualizat...

2017-01-10 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1842
  
And when I test npm online package for using bubble chart, It works very 
well!

![z1842_b](https://cloud.githubusercontent.com/assets/8110458/21835960/94d69aee-d804-11e6-8b4f-8b51c0bf72cc.gif)

LGTM! 



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


[GitHub] zeppelin issue #1842: [ZEPPELIN-1619] Load js package as a plugin visualizat...

2017-01-10 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1842
  
It's very cool feature!
Tested and It works very nice like below.

**[Test result]**
When I clicked `Enable` button for using `zeppelin_horizontalbar` 
visualization package. 

![z1842_e](https://cloud.githubusercontent.com/assets/8110458/21835438/50aaa7ba-d800-11e6-9a4e-5dcbc6f4ffd4.gif)

When using `Disable` button 

![z1842_d](https://cloud.githubusercontent.com/assets/8110458/21835586/85b63f5e-d801-11e6-9784-f5dbc1fa1f21.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 #1879: [HOTFIX : ZEPPELIN-1932] paragraph blur error

2017-01-10 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1879
  
@cloverhearts I'd like to test this. could you resolve some conflict file?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have 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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
@minahlee Thanks for help. I have updated these release script. 


---
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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
We always have been deliver release with the most recent minor spark 
release, so I think it's better to update release profile to use 2.1.
@zjffdu It would be appreciated if you can update spark profile in 
following three lines:
https://github.com/apache/zeppelin/blob/master/dev/create_release.sh#L126
https://github.com/apache/zeppelin/blob/master/dev/create_release.sh#L127
https://github.com/apache/zeppelin/blob/master/dev/publish_release.sh#L47



---
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 #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter synta...

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at:

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


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


[GitHub] zeppelin issue #1871: ZEPPELIN-1862 Move packages related Helium into a new ...

2017-01-10 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1871
  
Tested and it works well :+1: 


---
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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
@ejono I have a following commit to fix the test failure. Could you retry 
that ?


---
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 #1878: [ZEPPELIN-1931] add yarn_error.log in gitignore

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1825: [DOCS][ZEPPELIN-1586] Add 'Security' section un...

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1831: [ZEPPELIN-1883] Can't import spark submitted packages ...

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1831
  
Thanks, @1ambda Do you mind to create a spark ticket as well ? The behavior 
inconsistency between different modes seems an issue of spark, we need to 
clarify it with spark community. 


---
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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
Agree with @Leemoonsoo 


---
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.
---


Re: branch-0.7 cut

2017-01-10 Thread Prabhjyot Singh
Thank you Mina for taking care of this.

On 11 Jan 2017 7:23 a.m., "Anthony Corbacho" 
wrote:

> Thank you Mina for taking care of this tasks!
>
> On Wed, 11 Jan 2017 at 5:14 AM Vinay Shukla  wrote:
>
> > Thanks Mina Lee.
> >
> >
> >
> > On Tue, Jan 10, 2017 at 11:10 AM moon soo Lee  wrote:
> >
> >
> >
> > > Thanks Mina taking care of it. Sounds good to me.
> >
> > >
> >
> > > One thing is, shell we try have docker image in this release?
> >
> > > Following issues tracks the the job
> >
> > > https://issues.apache.org/jira/browse/ZEPPELIN-1711
> >
> > > https://issues.apache.org/jira/browse/INFRA-12781
> >
> > >
> >
> > > I don't think it need to be a blocker for 0.7.0 release, but if they
> can
> > be
> >
> > > resolved by the time we create release candidate and have docker image
> in
> >
> > > the release, it would be great.
> >
> > >
> >
> > > Thanks,
> >
> > > moon
> >
> > >
> >
> > > On Mon, Jan 9, 2017 at 10:24 PM Mina Lee  wrote:
> >
> > >
> >
> > > > Hi,
> >
> > > >
> >
> > > > It's been almost 3 month since we did 0.6.2 release and there were
> > number
> >
> > > > of improvements and bug fixes.
> >
> > > > Most of the items that we planned for 0.7.0 release [1] are ready
> >
> > > > so I am planning to cut branch-0.7 on 10th Jan 11pm(PST).
> >
> > > >
> >
> > > > After branch cut, bug fixes, documentation patches and issues linked
> > as a
> >
> > > > blocker in jira [2] will be merged both to branch-0.7, master branch
> > but
> >
> > > > new features will be merged only to master branch.
> >
> > > >
> >
> > > > Thanks,
> >
> > > > Mina
> >
> > > >
> >
> > > > [1]
> >
> > > https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+Roadmap
> >
> > > > [2] https://issues.apache.org/jira/browse/ZEPPELIN-1795
> >
> > > >
> >
> > >
> >
> >
>


[GitHub] zeppelin issue #1878: [ZEPPELIN-1931] add yarn_error.log in gitignore

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1878
  
Merge if there are no more comments on this :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not 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.
---


Re: branch-0.7 cut

2017-01-10 Thread Anthony Corbacho
Thank you Mina for taking care of this tasks!

On Wed, 11 Jan 2017 at 5:14 AM Vinay Shukla  wrote:

> Thanks Mina Lee.
>
>
>
> On Tue, Jan 10, 2017 at 11:10 AM moon soo Lee  wrote:
>
>
>
> > Thanks Mina taking care of it. Sounds good to me.
>
> >
>
> > One thing is, shell we try have docker image in this release?
>
> > Following issues tracks the the job
>
> > https://issues.apache.org/jira/browse/ZEPPELIN-1711
>
> > https://issues.apache.org/jira/browse/INFRA-12781
>
> >
>
> > I don't think it need to be a blocker for 0.7.0 release, but if they can
> be
>
> > resolved by the time we create release candidate and have docker image in
>
> > the release, it would be great.
>
> >
>
> > Thanks,
>
> > moon
>
> >
>
> > On Mon, Jan 9, 2017 at 10:24 PM Mina Lee  wrote:
>
> >
>
> > > Hi,
>
> > >
>
> > > It's been almost 3 month since we did 0.6.2 release and there were
> number
>
> > > of improvements and bug fixes.
>
> > > Most of the items that we planned for 0.7.0 release [1] are ready
>
> > > so I am planning to cut branch-0.7 on 10th Jan 11pm(PST).
>
> > >
>
> > > After branch cut, bug fixes, documentation patches and issues linked
> as a
>
> > > blocker in jira [2] will be merged both to branch-0.7, master branch
> but
>
> > > new features will be merged only to master branch.
>
> > >
>
> > > Thanks,
>
> > > Mina
>
> > >
>
> > > [1]
>
> > https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+Roadmap
>
> > > [2] https://issues.apache.org/jira/browse/ZEPPELIN-1795
>
> > >
>
> >
>
>


[GitHub] zeppelin issue #1868: [ZEPPELIN-1917] Improve python.conda interpreter

2017-01-10 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1868
  
@Leemoonsoo now `python.conda` support other `env *` commands :)

https://cloud.githubusercontent.com/assets/4968473/21827211/0130cd40-d7ce-11e6-8925-4757c6a2d6fc.png;>



---
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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
Tested and it works well.

Shell we add `spark-2.1` profile  
[here](https://github.com/apache/zeppelin/blob/master/docs/install/build.md#build-profiles)
 and 
[here](https://github.com/apache/zeppelin/blob/master/docs/install/build.md#build-command-examples)
 ? \cc @AhyoungRyu 

Also shell we use `spark-2.1` profile in [release artifact 
build](https://github.com/apache/zeppelin/blob/master/dev/create_release.sh#L126)
 ? \cc @minahlee 




---
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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
LGTM. Sorry I didn't spot the version number earlier.


---
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 #1863: [ZEPPELIN-1920] Fail to load app.js when ZEPPEL...

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1860: [ZEPPELIN-1848] add option for S3 KMS key regio...

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1852: [ZEPPELIN-1910] DON'T show the same dialogs mul...

2017-01-10 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1840: [ZEPPELIN-1730, 1587] add spark impersonation through ...

2017-01-10 Thread Tagar
Github user Tagar commented on the issue:

https://github.com/apache/zeppelin/pull/1840
  
As far as credentials refresh are concerned, please see new comments in 
[SPARK-19143](https://issues.apache.org/jira/browse/SPARK-19143). 
Hope this helps.


---
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 #1846: ZEPPELIN-1770. Restart only the client user's interpre...

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1846
  
CI failure is not relevant.

```
Failed tests: 
  NotebookTest.testAbortParagraphStatusOnInterpreterRestart:760 
expected: but was:
AngularElem
- should provide onclick method *** FAILED ***
  The code passed to eventually never returned normally. Attempted 1 times 
over 2.966494567 seconds. Last failure message: 0 was not equal to 1. 
(AbstractAngularElemTest.scala:64)
AngularElem
```


---
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 #1882: [MINOR] Small Selenium test fixes

2017-01-10 Thread bitchelov
GitHub user bitchelov opened a pull request:

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

[MINOR] Small Selenium test fixes

### What is this PR for?
Remove unused imports;
Replace "Thread.sleep" (unstable solution);
Add "deleteTestNotebook()" method  -> testWidth.


### What type of PR is it?
[Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. 
[ZEPPELIN-533]

### How should this be tested?
Travis-CI

### 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/bitchelov/zeppelin automationTestFix

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

https://github.com/apache/zeppelin/pull/1882.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 #1882


commit f898c59b3424703cd3009d26b34510ea54386907
Author: Unknown 
Date:   2017-01-10T14:03:03Z

[MINOR] Small Selenium test fixes

Remove unused imports;
Replace "Thread.sleep" (unstable solution);
Add "deleteTestNotebook()" method  -> testWidth.




---
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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
@Leemoonsoo @jongyoul @felixcheung @ejono Please help review. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled 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 #1880: ZEPPELIN-1918. Fix build with Spark 2.1.0

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1880
  
CI failure is not relevant.

```
AngularElem
- should provide onclick method *** FAILED ***
  The code passed to eventually never returned normally. Attempted 1 times 
over 602.103885 milliseconds. Last failure message: 0 was not equal to 1. 
(AbstractAngularElemTest.scala:64)
```


---
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 #1846: ZEPPELIN-1770. Restart only the client user's i...

2017-01-10 Thread zjffdu
Github user zjffdu closed the pull request at:

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


---
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 #1846: ZEPPELIN-1770. Restart only the client user's i...

2017-01-10 Thread zjffdu
GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1770. Restart only the client user's interpreter when restarting 
interpreter setting


### What is this PR for?
This PR would only restart the trigger user's interpreter rather than all 
the interpreter. So that restarting won't affect other users. 


### What type of PR is it?
[Improvement]

### Todos
* [ ] - Task

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

### How should this be tested?
Tested manually.

### 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/zjffdu/zeppelin ZEPPELIN-1770

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

https://github.com/apache/zeppelin/pull/1846.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 #1846


commit 8cb28a31b96c71d749f40e850e886958954136ff
Author: Jeff Zhang 
Date:   2016-12-14T07:04:20Z

ZEPPELIN-1770. Restart only the client user's interpreter when restarting 
interpreter setting

commit 5ee076dd93d74d50f34aa3ef27b49450b714c5f2
Author: Jeff Zhang 
Date:   2017-01-10T06:07:33Z

fix scoped mode and add unit test




---
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 #1871: ZEPPELIN-1862 Move packages related Helium into a new ...

2017-01-10 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1871
  
Will merge it if there's no more discussion


---
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 #1881: ZEPPELIN-1935 Add jceks stored password support...

2017-01-10 Thread r-kamath
GitHub user r-kamath opened a pull request:

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

ZEPPELIN-1935 Add jceks stored password support for jdbc interpreter

### What is this PR for?
Add support for jceks stored password instead of using password in clear 
text. This security enhancement prevents any user from reading clear passwords 
from interpreter json using shell/py/spark etc. #1315 is the parent PR which 
fixed similar a issue in Active Directory (shiro.ini config).

### What type of PR is it?
Improvement

### Todos

### What is the Jira issue?
ZEPPELIN-1935

### How should this be tested?
Create a keystore file using the hadoop credential commandline, for this 
the hadoop commons should be in the classpath.

`hadoop credential create jdbc.password -provider 
jceks://file/user/zeppelin/conf/zeppelin.jceks`

Use the jceks file and the key to configure jdbc interpreter. Example 
interpreter setting:

```
default.driver  org.postgresql.Driver
default.jceks.credentialKey jdbc.password
default.jceks.file  jceks://file/tmp/zeppelin.jceks
default.url jdbc:postgresql://rkamath-local-1:5432/
default.userrk-user
```

### Screenshots (if appropriate)
https://cloud.githubusercontent.com/assets/2031306/21808016/5e602982-d767-11e6-88f9-3d15b9a7f0b8.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/r-kamath/zeppelin ZEPPELIN-1935

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

https://github.com/apache/zeppelin/pull/1881.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 #1881


commit d41d56c70d588cf8ecb3f3162d78e3fbe011ddad
Author: Renjith Kamath 
Date:   2017-01-10T13:22:16Z

ZEPPELIN-1935 Add jceks stored password support for jdbc 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-1935) Add jceks stored password support for jdbc interpreter

2017-01-10 Thread Renjith Kamath (JIRA)
Renjith Kamath created ZEPPELIN-1935:


 Summary: Add jceks stored password support for jdbc interpreter
 Key: ZEPPELIN-1935
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1935
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Renjith Kamath






--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1878: [ZEPPELIN-1931] add yarn_error.log in gitignore

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1878
  
LGTM 


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


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

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1825
  
Thanks @minahlee!
Will merge if there are no more comments on this. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not 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 #1761: [ZEPPELIN-1711] Create Docker Images for Released Zepp...

2017-01-10 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1761
  
@felixcheung Thanks for review. 

Previously, we discussed about separating images in 
[ZEPPEILN-1711](https://issues.apache.org/jira/browse/ZEPPELIN-1711)

IMO, having 1 image which includes R, Python related packages is better. 
Since 

- it's usually case. 
- easy to maintain

I would like to use ubuntu which is comfortable as a desktop OS instead of 
centos. 

What do you think? @felixcheung @jongyoul @bzz 


---
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 #1845: [ZEPPELIN-1906] Use multiple InterpreterResult for dis...

2017-01-10 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1845
  
Sorry for late response @prabhjyotsingh, @jongyoul .
Let me review soon.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If 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-1934) Building from source documentation

2017-01-10 Thread Tushar (JIRA)
Tushar created ZEPPELIN-1934:


 Summary: Building from source documentation
 Key: ZEPPELIN-1934
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1934
 Project: Zeppelin
  Issue Type: Improvement
  Components: documentation
Affects Versions: 0.7.0
 Environment: 1. Mac OS X El Capitan (10.11.6 )
2. ubuntu 16.04.1 LTS

Reporter: Tushar
Priority: Minor
 Fix For: 0.7.0


Followed http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/install/install.html 
and  
http://zeppelin.apache.org/docs/snapshot/install/build.html#build-requirements

Worked on ubuntu 16 a few weeks back but not today.

On Mac
---
[INFO] --- maven-checkstyle-plugin:2.13:check (checkstyle-fail-build) @ 
zeppelin-distribution ---
[INFO] 
[INFO] 
[INFO] --- maven-resources-plugin:2.7:copy-resources (copy-resources) @ 
zeppelin-distribution ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 17 resources
[INFO] 
[INFO] --- maven-enforcer-plugin:1.3.1:enforce (enforce) @ 
zeppelin-distribution ---
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.5:process (default) @ 
zeppelin-distribution ---
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.5:process (process-remote-resources) 
@ zeppelin-distribution ---
[INFO] 
[INFO] --- maven-dependency-plugin:2.8:copy-dependencies (copy-dependencies) @ 
zeppelin-distribution ---
[INFO] Skipping plugin execution
[INFO] 
[INFO] --- maven-assembly-plugin:3.0.0:single (make-assembly) @ 
zeppelin-distribution ---
[INFO] Reading assembly descriptor: src/assemble/distribution.xml
[WARNING] The assembly descriptor contains a filesystem-root relative 
reference, which is not cross platform compatible /lib/interpreter
[WARNING] Cannot include project artifact: 
org.apache.zeppelin:zeppelin-distribution:pom:0.7.0-SNAPSHOT; it doesn't have 
an associated file or directory.
[INFO] Copying files to 
/Users/tusharkapila/u/w/s/github/apache/zeppelin/zeppelin-distribution/target/zeppelin-0.7.0-SNAPSHOT
[WARNING] The assembly descriptor contains a filesystem-root relative 
reference, which is not cross platform compatible /lib/interpreter
[WARNING] Cannot include project artifact: 
org.apache.zeppelin:zeppelin-distribution:pom:0.7.0-SNAPSHOT; it doesn't have 
an associated file or directory.
[INFO] Building tar: 
/Users/tusharkapila/u/w/s/github/apache/zeppelin/zeppelin-distribution/target/zeppelin-0.7.0-SNAPSHOT.tar.gz
[INFO] 
[INFO] Reactor Summary:
[INFO] 
[INFO] Zeppelin ... SUCCESS [  2.023 s]
[INFO] Zeppelin: Interpreter .. SUCCESS [  2.398 s]
...

INFO] Zeppelin: Server ... SUCCESS [  3.778 s]
[INFO] Zeppelin: Packaging distribution ... FAILURE [ 16.700 s]
[INFO] 
[INFO] BUILD FAILURE
[INFO] 
[INFO] Total time: 03:28 min
[INFO] Finished at: 2017-01-10T13:15:10+05:30
[INFO] Final Memory: 109M/848M
[INFO] 
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-assembly-plugin:3.0.0:single (make-assembly) on 
project zeppelin-distribution: Execution make-assembly of goal 
org.apache.maven.plugins:maven-assembly-plugin:3.0.0:single failed: user id 
'1926187091' is too big ( > 2097151 ). -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please 
read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :zeppelin-distribution
--- 

On Ububtu today - new VM Java versions have changed, would be good to get an 
updated build html for 2017



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


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

2017-01-10 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1804
  
I improved the template of html because I think `customHome` js controller 
and `Home`js controller should be equally worked when `home` js controller is 
changed. 
So, I'll change from the template in `home.html` to template of 
`customHome.html` after merging this.

@AhyoungRyu @Leemoonsoo could you check this PR again, please?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as 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 #1846: ZEPPELIN-1770. Restart only the client user's interpre...

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1846
  
Thanks @Leemoonsoo  Unit test is added. Please help review \cc @felixcheung 
@jongyoul 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
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 #1858: [ZEPPELIN-1229] Clear browser cache with version hasht...

2017-01-10 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1858
  
@corneadoug please help to review.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
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-1933) Add queue configuration in pig interpreter

2017-01-10 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1933:


 Summary: Add queue configuration in pig interpreter
 Key: ZEPPELIN-1933
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1933
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.7.0
Reporter: Jeff Zhang
Assignee: Jeff Zhang






--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1840: [ZEPPELIN-1730, 1587] add spark impersonation through ...

2017-01-10 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1840
  
Also @Leemoonsoo review on this one would be helpful


---
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 #1825: [DOCS][ZEPPELIN-1586] Add 'Security' section under not...

2017-01-10 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1825
  
Nice improvement, LGTM


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


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

2017-01-10 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1825
  
I updated this docs page again since REST API operations need to be listed 
as CRUD order. And removed unimplemented response code info as well. Please see 
the updated screenshot images :)


---
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 #1787: ZEPPELIN-1815. Support Spark 2.1

2017-01-10 Thread ejono
Github user ejono commented on the issue:

https://github.com/apache/zeppelin/pull/1787
  
Sorry, I was not able to work on this over the past few days. Thank you 
very much for fixing it, @zjffdu!


---
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 #1861: ZEPPELIN-1293. Livy Interpreter: Automatically attach ...

2017-01-10 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1861
  
The bug in LivySparkSQLInterpreter is caused by this PR. 
LivySparkSQLInterpreter delegate LivySparkInterpreter to generate 
InterpreterResult and then convert it to table type. So here since we make it 
as multiple InterpreterResult, we should only convert the InterpreterMessage 
with text type that is for sql output.  


---
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.
---