[GitHub] zeppelin pull request #1518: [ZEPPELIN-1549] Change NotebookID variable name...

2016-10-14 Thread hyonzin
Github user hyonzin closed the pull request at:

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


---
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 #1518: [ZEPPELIN-1549] Change NotebookID variable name...

2016-10-14 Thread hyonzin
GitHub user hyonzin reopened a pull request:

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

[ZEPPELIN-1549] Change NotebookID variable name to NoteID

### What is this PR for?
This PR fixes wrong written NotebookID to NoteID.


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


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


### 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/hyonzin/zeppelin ZEPPELIN-1549

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

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


commit cc0d3151efab5280a1ef9cdd61e965943e113be2
Author: hyonzin 
Date:   2016-10-14T06:55:57Z

Change NotebookID variable name to NoteID

commit 294bea522f838011888c23a5f6b9246b15917149
Author: hyonzin 
Date:   2016-10-14T10:33:58Z

Fix some wrong written term: Notebook -> 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.
---


[GitHub] zeppelin pull request #1525: [ZEPPELIN-1552] Search button goes to next line...

2016-10-14 Thread hyonzin
GitHub user hyonzin reopened a pull request:

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

[ZEPPELIN-1552] Search button goes to next line when display's width 
shortens.

### What is this PR for?
In document page(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/), the 
search button goes to next line when display's width shortens.


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

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

### Screenshots (if appropriate)
https://cloud.githubusercontent.com/assets/6567102/19398321/0d76a1c4-9287-11e6-86e0-9f120c00b143.png";>
when (768px ≤ width < 992px)

### 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/hyonzin/zeppelin ZEPPELIN-1552

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

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


commit 97d1bdfa032733a6ae981b282fbb4d45da152713
Author: hyonzin 
Date:   2016-10-14T18:18:17Z

Fix search link button to not goes next line when display's width shortens

commit 244992d4cf54dcc55bfc2e1f06d10a9d660fc853
Author: 정현진 
Date:   2016-10-14T18:30:56Z

Straighten wrong term

commit f1d97ef840093cdfb24234070fe786ebb562f84d
Author: AhyoungRyu 
Date:   2016-10-15T03:22:18Z

Fix search icon's placement in small browser




---
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 #1525: [ZEPPELIN-1552] Search button goes to next line...

2016-10-14 Thread hyonzin
Github user hyonzin closed the pull request at:

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


---
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 #1525: [ZEPPELIN-1552] Search button goes to next line when d...

2016-10-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1525
  
@hyonzin Thx for your prompt response! 
Seems the CI is not happy now with below reason

```
No output has been received in the last 10 minutes, this potentially 
indicates a stalled build or something wrong with the build itself.

The build has been terminated
```

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 #1527: [ZEPPELIN-1553] Updated pom dependency to support spar...

2016-10-14 Thread LantaoJin
Github user LantaoJin commented on the issue:

https://github.com/apache/zeppelin/pull/1527
  
I tried this way and also failed during building. So I opened ZEPPELIN-1544


---
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 #1525: [ZEPPELIN-1552] Search button goes to next line when d...

2016-10-14 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1525
  
@AhyoungRyu Thank you for your reply and good suggestion. I checked that in 
your branch and It seems to work very well!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
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 #1525: [ZEPPELIN-1552] Search button goes to next line when d...

2016-10-14 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1525
  
@AhyoungRyu Thank you for your reply and good suggestion. I checked that in 
your branch and It seems to work very well!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
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 #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
Seems there are some test failures in [ SCALA_VER="2.10" 
SPARK_VER="1.5.2"](https://travis-ci.org/apache/zeppelin/jobs/167607199)

```
Results :

Tests in error: 
  InterpreterRestApiTest.init:53->AbstractTestRestApi.startUp:159 » 
NullPointer
  CredentialsRestApiTest.init:46->AbstractTestRestApi.startUp:159 » 
NullPointer
  ZeppelinRestApiTest.init:56->AbstractTestRestApi.startUp:159 » 
NullPointer
  ConfigurationsRestApiTest.init:39->AbstractTestRestApi.startUp:159 » 
NullPointer
  NotebookRestApiTest.init:55->AbstractTestRestApi.startUp:159 » 
NullPointer
  ZeppelinSparkClusterTest.init:49->AbstractTestRestApi.startUp:159 » 
NullPointer
  SecurityRestApiTest.init:44->AbstractTestRestApi.startUp:159 » 
NullPointer

Tests run: 7, Failures: 0, Errors: 7, Skipped: 0

[INFO] 

[INFO] Reactor Summary:
[INFO] 
[INFO] Zeppelin: Interpreter .. SUCCESS [ 
14.139 s]
[INFO] Zeppelin: Zengine .. SUCCESS [  
7.096 s]
[INFO] Zeppelin: Display system apis .. SUCCESS [  
3.527 s]
[INFO] Zeppelin: Spark dependencies ... SUCCESS [ 
42.764 s]
[INFO] Zeppelin: Spark  SUCCESS [01:52 
min]
[INFO] Zeppelin: Server ... FAILURE [ 
44.231 s]
[INFO] Zeppelin: R Interpreter  SKIPPED
[INFO] 

[INFO] BUILD FAILURE
[INFO] 

[INFO] Total time: 03:44 min
[INFO] Finished at: 2016-10-14T15:35:13+00:00
[INFO] Final Memory: 47M/468M
[INFO] 

[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:2.17:test (default-test) on 
project zeppelin-server: There are test failures.
[ERROR] 
[ERROR] Please refer to 
/home/travis/build/apache/zeppelin/zeppelin-server/target/surefire-reports for 
the individual test results.
[ERROR] -> [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/MojoFailureException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the 
command
[ERROR]   mvn  -rf :zeppelin-server

```

I think they are related with this change.. 


---
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 #1525: [ZEPPELIN-1552] Search button goes to next line when d...

2016-10-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1525
  
@hyonzin Thanks for your contribution and good catch :)
Actually I added this search feature in #1266 so it's my bad.. Appreciate 
for taking care of this. 

Just tested and it seems not working as expected in my 13 inch screen. (FYI 
13inch -> about 1248px)


![before](https://cloud.githubusercontent.com/assets/10060731/19407064/80fa4cbe-92d1-11e6-962b-6c615598ddaf.gif)

So I worked around a bit to fix this, you can check it in my branch: 
[please check this 
out](https://github.com/apache/zeppelin/compare/master...AhyoungRyu:ZEPPELIN-1552-ahyoung?expand=1).
 With my change it will be like this. 


![after](https://cloud.githubusercontent.com/assets/10060731/19407092/4838302a-92d2-11e6-81ce-96c9d2cb2b73.gif)
 
If you have time, please check this and let me know how it is! :D


---
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 #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-14 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
It seems looks great to me. 
What do you think @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 #1523: [ZEPPELIN-1550] fixed 'add from URL' button

2016-10-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1523
  
@yunho0130 Thanks for your contribution. Nice catch! 
Tested and works well. LGTM 👍 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)

2016-10-14 Thread Hyung Sung Shim
+1

Tested Tutorial and RTutorial with EMR-5.0.0.

Thank you mina for your effort.


2016-10-15 8:29 GMT+09:00 Felix Cheung :

> +1
>
> Tested source and netinstall
>
> Thanks Mina!
>
>
> _
> From: Ahyoung Ryu mailto:ahyoung...@apache.org>>
> Sent: Friday, October 14, 2016 5:28 AM
> Subject: Re: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)
> To: mailto:dev@zeppelin.apache.org>>
>
>
> +1 for me
>
> Yeah thanks Mina for all your effort!!!
>
> On Fri, Oct 14, 2016 at 9:10 PM, Anthony Corbacho <
> anthonycorba...@apache.org> wrote:
>
> > Hi,
> >
> > I tested this RC2 with JDK8, and no issue so far.
> >
> > +1 for me.
> >
> > Thanks to mina for the fast iteration on the release process, awesome as
> > always :)
> >
> >
> > On Thu, Oct 13, 2016 at 6:41 PM, DuyHai Doan  > wrote:
> >
> > > Hello, the link https://dist.apache.org/repos/
> > > dist/dev/zeppelin/zeppelin-0.
> > > 6.2-rc2/ is a Zeppelin compiled with Scala 2.11. Do you have the same
> for
> > > Scala 2.10 ?
> > >
> > > On Thu, Oct 13, 2016 at 9:59 AM, Jeff Zhang  zjf...@gmail.com>> wrote:
> > >
> > > > +1, Run tutorial note successfully in embedded mode and yarn-client.
> > > >
> > > >
> > > >
> > > > Mina Lee mailto:mina...@apache.org
> >>?2016?10?12??? ??6:29???
> > > >
> > > > > Hi dev,
> > > > >
> > > > > I propose the following RC to be released for the Apache Zeppelin
> > 0.6.2
> > > > > release.
> > > > >
> > > > > The commit id is 091086de9400dd1c02ca02acf4180b1bf1e9ede7 which is
> > > > > corresponds
> > > > > to the tag v0.6.2-rc2:
> > > > > *
> > > > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > > > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > > > <
> > > > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > > > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > > > >*
> > > > >
> > > > > The release archives (tgz), signature, and checksums are here
> > > > > https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.
> 6.2-rc2/
> > > > >
> > > > > The release candidate consists of the following source distribution
> > > > archive
> > > > > zeppelin-0.6.2.tgz
> > > > >
> > > > > In addition, the following supplementary binary distributions are
> > > > provided
> > > > > for user convenience at the same location
> > > > > zeppelin-0.6.2-bin-all.tgz
> > > > > zeppelin-0.6.2-netinst-all.tgz
> > > > >
> > > > > The maven artifacts are here
> > > > > https://repository.apache
> > > > > .org/content/repositories/orgapachezeppelin-1020/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=12336543&styleName=Html&projectId=12316221
> > > > > <
> > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> > > > version=12336543&styleName=Html&projectId=12316221
> > > > > >*
> > > > >
> > > > > Vote will be open for next 72 hours (close at 4am 15/Oct PDT).
> > > > >
> > > > > [ ] +1 approve
> > > > > [ ] 0 no opinion
> > > > > [ ] -1 disapprove (and reason why)
> > > > >
> > > >
> > >
> >
>
>
>


[GitHub] zeppelin issue #1522: [ ZEPPELIN-1461 ] Doesn't display "description" value ...

2016-10-14 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1522
  
👍 


---
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 #1523: [ZEPPELIN-1550] fixed 'add from URL' button

2016-10-14 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1523
  
+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 #1525: [ZEPPELIN-1552] Search button goes to next line when d...

2016-10-14 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1525
  
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 #1501: [ZEPPELIN-1532] Remove map visualization from source t...

2016-10-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1501
  
Merge if there're no more discussions.


---
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: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)

2016-10-14 Thread Felix Cheung
+1

Tested source and netinstall

Thanks Mina!


_
From: Ahyoung Ryu mailto:ahyoung...@apache.org>>
Sent: Friday, October 14, 2016 5:28 AM
Subject: Re: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)
To: mailto:dev@zeppelin.apache.org>>


+1 for me

Yeah thanks Mina for all your effort!!!

On Fri, Oct 14, 2016 at 9:10 PM, Anthony Corbacho <
anthonycorba...@apache.org> wrote:

> Hi,
>
> I tested this RC2 with JDK8, and no issue so far.
>
> +1 for me.
>
> Thanks to mina for the fast iteration on the release process, awesome as
> always :)
>
>
> On Thu, Oct 13, 2016 at 6:41 PM, DuyHai Doan 
> mailto:doanduy...@gmail.com>> wrote:
>
> > Hello, the link https://dist.apache.org/repos/
> > dist/dev/zeppelin/zeppelin-0.
> > 6.2-rc2/ is a Zeppelin compiled with Scala 2.11. Do you have the same for
> > Scala 2.10 ?
> >
> > On Thu, Oct 13, 2016 at 9:59 AM, Jeff Zhang 
> > mailto:zjf...@gmail.com>> wrote:
> >
> > > +1, Run tutorial note successfully in embedded mode and yarn-client.
> > >
> > >
> > >
> > > Mina Lee mailto:mina...@apache.org>>?2016?10?12??? 
> > > ??6:29???
> > >
> > > > Hi dev,
> > > >
> > > > I propose the following RC to be released for the Apache Zeppelin
> 0.6.2
> > > > release.
> > > >
> > > > The commit id is 091086de9400dd1c02ca02acf4180b1bf1e9ede7 which is
> > > > corresponds
> > > > to the tag v0.6.2-rc2:
> > > > *
> > > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > > <
> > > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > > >*
> > > >
> > > > The release archives (tgz), signature, and checksums are here
> > > > https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.6.2-rc2/
> > > >
> > > > The release candidate consists of the following source distribution
> > > archive
> > > > zeppelin-0.6.2.tgz
> > > >
> > > > In addition, the following supplementary binary distributions are
> > > provided
> > > > for user convenience at the same location
> > > > zeppelin-0.6.2-bin-all.tgz
> > > > zeppelin-0.6.2-netinst-all.tgz
> > > >
> > > > The maven artifacts are here
> > > > https://repository.apache
> > > > .org/content/repositories/orgapachezeppelin-1020/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=12336543&styleName=Html&projectId=12316221
> > > > <
> > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> > > version=12336543&styleName=Html&projectId=12316221
> > > > >*
> > > >
> > > > Vote will be open for next 72 hours (close at 4am 15/Oct PDT).
> > > >
> > > > [ ] +1 approve
> > > > [ ] 0 no opinion
> > > > [ ] -1 disapprove (and reason why)
> > > >
> > >
> >
>




[GitHub] zeppelin issue #1497: [ZEPPELIN-1233] Support Redis interpreter

2016-10-14 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue:

https://github.com/apache/zeppelin/pull/1497
  
Every command methods have a chance to leak Jedis instance when command is 
failing and throwing a kind of JedisException.
You need to replace your borrow-call-return pattern with 
`try-with-resource` thanks to Zeppelin requires Java 1.7.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If 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 #1501: [ZEPPELIN-1532] Remove map visualization from s...

2016-10-14 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at:

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


---
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 #1501: [ZEPPELIN-1532] Remove map visualization from s...

2016-10-14 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request:

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

[ZEPPELIN-1532] Remove map visualization from source tree, because of 
incompatible license

### What is this PR for?
According to https://developers.arcgis.com/terms/ and and 
http://www.esri.com/legal/software-license, current map visualization depends 
on an online service that has incompatible restrictions to Apache 2 license. 
Please see https://github.com/apache/zeppelin/pull/1210#issuecomment-252484382

Possible alternative way is explained 
[here](https://github.com/apache/zeppelin/pull/1210#issuecomment-252763598).

I'm very sad to remove this very good contribution. So please anyone review 
the license and comment here if you have other opinions, or other way around.


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

### Todos
* [x] - remove implementation from source tree

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


### Questions:
* Does the licenses files need update? yes
* 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/Leemoonsoo/zeppelin ZEPPELIN-1532

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

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


commit dad81c414b75ebe574c2de669b63b2095856244f
Author: Lee moon soo 
Date:   2016-10-10T22:37:00Z

Remove map visualization from source tree

commit b5ad91a9f07d55600dd55d9331fb6e9f5eb40b2e
Author: Lee moon soo 
Date:   2016-10-10T22:45:24Z

Remove map visualization




---
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 #1527: [ZEPPELIN-1553] Updated pom dependency to support spar...

2016-10-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1527
  
Thanks @sitsang for the contribution.
But, I think https://github.com/apache/zeppelin/pull/1511 can cover 
ZEPPELIN-1553, too.


---
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 #1526: Updated version of py4j to 0.10.3 for spark 2.0...

2016-10-14 Thread sitsang
Github user sitsang closed the pull request at:

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


---
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 #1527: [ZEPPELIN-1553] Updated pom dependency to suppo...

2016-10-14 Thread sitsang
GitHub user sitsang opened a pull request:

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

[ZEPPELIN-1553] Updated pom dependency to support spark 2.0.1

### What is this PR for?
The spark 2.0 profile have py4j version 0.10.1, but the spark 2.0.1 dist 
tarball only contains py4j 0.10.3. Need to update pom.xml for 2.0.1

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

### Todos
* [ ] - Task

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

### How should this be tested?
Outline the steps to test the PR here.

```
mvn clean package -Pspark-2.0.1 -Pscala-2.11 -Ppyspark -Pvendor-repo 
-DskipTests
```

### 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/sitsang/zeppelin master

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

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


commit 287ba1fd5677f9bc834fbfb74699a2e3cb79
Author: sitsang 
Date:   2016-10-14T19:34:46Z

Update pom.xml




---
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-1553) Incorrect version of py4j when compling for spark 2.0.1

2016-10-14 Thread Wai Tsang (JIRA)
Wai Tsang created ZEPPELIN-1553:
---

 Summary: Incorrect version of py4j when compling for spark 2.0.1 
 Key: ZEPPELIN-1553
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1553
 Project: Zeppelin
  Issue Type: Bug
  Components: pySpark
Reporter: Wai Tsang
Priority: Trivial


The spark 2.0 profile have py4j version 0.10.1, but the spark 2.0.1 dist 
tarball only contains py4j 0.10.3.  Need to update pom.xml for 2.0.1



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


[GitHub] zeppelin pull request #1526: Updated version of py4j to 0.10.3 for spark 2.0...

2016-10-14 Thread sitsang
GitHub user sitsang opened a pull request:

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

Updated version of py4j to 0.10.3 for spark 2.0.1

### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - 
https://zeppelin.apache.org/contribution/contributions.html


### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | 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?
Outline the steps to test the PR here.

### Screenshots (if appropriate)

### 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/sitsang/zeppelin patch-1

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

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


commit 81495494dcd7d2bd39efd8cab6f2c6bd37756252
Author: sitsang 
Date:   2016-10-14T19:24:34Z

Updated version of py4j to 0.10.3 for spark 2.0.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 pull request #1525: [ZEPPELIN-1552] Search button goes to next line...

2016-10-14 Thread hyonzin
GitHub user hyonzin opened a pull request:

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

[ZEPPELIN-1552] Search button goes to next line when display's width 
shortens.

### What is this PR for?
In document page(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/), the 
search button goes to next line when display's width shortens.


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

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

### Screenshots (if appropriate)
https://cloud.githubusercontent.com/assets/6567102/19398321/0d76a1c4-9287-11e6-86e0-9f120c00b143.png";>
when (768px ≤ width < 992px)

### 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/hyonzin/zeppelin ZEPPELIN-1552

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

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


commit 97d1bdfa032733a6ae981b282fbb4d45da152713
Author: hyonzin 
Date:   2016-10-14T18:18:17Z

Fix search link button to not goes next line when display's width shortens




---
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 #1513: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1513
  
@khalidhuseynov and @zjffdu can we look at committing this since tests have 
passed. If not let me know what else is needed.

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 #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1516
  
@zjffdu and @prabhjyotsingh is this safe to be committed at this point?


---
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-1552) Search button goes to next line when display's width shortens.

2016-10-14 Thread Hyeonjin Jung (JIRA)
Hyeonjin Jung created ZEPPELIN-1552:
---

 Summary: Search button goes to next line when display's width 
shortens.
 Key: ZEPPELIN-1552
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1552
 Project: Zeppelin
  Issue Type: Improvement
  Components: documentation
Affects Versions: 0.7.0
Reporter: Hyeonjin Jung
Priority: Minor
 Fix For: 0.7.0


In document page(http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/), the search 
button goes to next line when display's width shortens.



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


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

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1447
  
@spektom this fix is good. Did some extensive load testing with it this AM 
and it solves the session expiration issues. Thanks for the contribution


---
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 #1458: [ZEPPELIN-1486] Allow configuring whether shown values...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1458
  
Also input validation is secure java coding best practice regardless.. 
http://www.oracle.com/technetwork/java/seccodeguide-139067.html#5


---
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 #1458: [ZEPPELIN-1486] Allow configuring whether shown values...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1458
  
Well we will just agree to disagree



---
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 #1458: [ZEPPELIN-1486] Allow configuring whether shown values...

2016-10-14 Thread spektom
Github user spektom commented on the issue:

https://github.com/apache/zeppelin/pull/1458
  
Are you trying to say that people working for enterprise companies require
special input validation in addition to providing a documentation saying
that an option accepts booleans? :)
I don't think entry possible input must be validated, after all these are
Software engineers who will be seeing up Zeppelin...
On Fri, Oct 14, 2016 at 18:01 gss2002  wrote:

> *@gss2002* commented on this pull request.
> --
>
> In
> livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java
> :
>
> > @@ -80,7 +80,8 @@ public InterpreterResult interpret(String line, 
InterpreterContext interpreterCo
>line.replaceAll("\"", "\"")
>.replaceAll("\\n", " ")
>+ "\").show(" +
> -  property.get("zeppelin.livy.spark.sql.maxResult") + ")",
> +  property.get("zeppelin.livy.spark.sql.maxResult") + "," +
> +  property.get("zeppelin.livy.spark.sql.truncateResult") + 
")",
>interpreterContext, userSessionMap);
>
> @spektom  I hate to be the bearer of bad news
> but Zeppelin is being adopted by many Enterprises and unforuantely in most
> cases I agree that it's the users problem. But if you are going to be
> committing to software that is going to be used by large enterprises then 
I
> would expect validation to occur. I am actually going to raise these 
issues
> with my Hadoop vendor.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or mute the thread
> 

> .
>



---
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 #1513: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-14 Thread gss2002
Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1513
  
Rerun build as error is not related to this patch.


---
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 #1513: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-14 Thread gss2002
Github user gss2002 closed the pull request at:

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


---
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 #1513: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-14 Thread gss2002
GitHub user gss2002 reopened a pull request:

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

ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm

### What is this PR for?
ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: 
Provides LdapRealm Functionality similar to what Apache Knox provides. This is 
critical as in large enterprise environments Active Directory Global Catalogs 
are used for lookup with samAccountName and using a DN Template is not an 
option as their are multiple OUs. Also searching on "userPrincipalName" is 
risky in an AD environment since the explicit UPN vs Implicit UPN can be 
different this is definitely the case with environments using Office 365. And 
the LDAP userPrincipalName attribute is the explicit UPN which can be defined 
by the directory administrator to any value and it can be duplicated. 
SamAccountName is unique per domain and Microsoft states best practice is to 
not allow duplicate samAccountName's across the forest.



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

### Todos
None

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

### How should this be tested?
Setup shiro.ini to use the following configuration:
ldapADGCRealm = org.apache.zeppelin.realm.LdapRealm
ldapADGCRealm.contextFactory.systemUsername = 
CN=hdpbind,OU=Svc,DC=exadc,DC=w2k,DC=example,DC=com
ldapADGCRealm.contextFactory.systemPassword = ldapPassword
ldapADGCRealm.searchBase = dc=w2k,dc=example,dc=com
ldapADGCRealm.userSearchBase = dc=w2k,dc=example,dc=com
ldapADGCRealm.groupSearchBase = dc=w2k,dc=example,dc=com
ldapADGCRealm.contextFactory.url = 
ldap://exampledc1.exadc.w2k.example.com:3268
ldapADGCRealm.userSearchAttributeName = sAMAccountName
ldapADGCRealm.contextFactory.authenticationMechanism = simple
ldapADGCRealm.userObjectClass = user
ldapADGCRealm.groupObjectClass = group
ldapADGCRealm.memberAttribute = member

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


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

$ git pull https://github.com/gss2002/zeppelin ZEPPELIN-1472

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

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


commit 34938754ac7e220a03cc1817bf93f2cf2d189ee9
Author: gss2002 
Date:   2016-10-11T03:58:51Z

ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm
Class

commit 8991d647b024d04eed7005173b4a8eec07b18c6c
Author: gss2002 
Date:   2016-10-14T00:48:25Z

Merge remote-tracking branch 'upstream/master' into ZEPPELIN-1472




---
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 #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-10-14 Thread gss2002
Github user gss2002 closed the pull request at:

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


---
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 #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-10-14 Thread gss2002
GitHub user gss2002 reopened a pull request:

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

ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught Kerberos 
Enabled

### What is this PR for?
ZEPPELIN-1546: Zeppelin Livy Interpreter 404 Exception not caught with 
Kerberos
Livy Interpreter gets a NestedRuntimeException when running with Kerberized 
components.



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

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

### How should this be tested?
Enable Kerberos

### Screenshots (if appropriate)

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

KerberosTemplate being enabled

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

$ git pull https://github.com/gss2002/zeppelin ZEPPELIN-1546

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

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


commit 54d8fe43adff15097aa8075a7976d6b96cd68b3f
Author: gss2002 
Date:   2016-10-14T03:24:35Z

ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught with
KerberosTemplate being enabled




---
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 #1497: [ZEPPELIN-1233] Support Redis interpreter

2016-10-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1497
  
I too checked the changes and they look good.The comment from 
@anthonycorbacho about usage of "Strategy pattern" for selecting the relevant 
redis command instance and executing them is the better way to implement it.

Also, could we add some test cases for this?

Thanks,
Kavin
MailTo: kavin.ku...@imaginea.com


---
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: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)

2016-10-14 Thread Ahyoung Ryu
+1 for me

Yeah thanks Mina for all your effort!!!

On Fri, Oct 14, 2016 at 9:10 PM, Anthony Corbacho <
anthonycorba...@apache.org> wrote:

> Hi,
>
> I tested this RC2 with JDK8, and no issue so far.
>
> +1 for me.
>
> Thanks to mina for the fast iteration on the release process, awesome as
> always :)
>
>
> On Thu, Oct 13, 2016 at 6:41 PM, DuyHai Doan  wrote:
>
> > Hello, the link https://dist.apache.org/repos/
> > dist/dev/zeppelin/zeppelin-0.
> > 6.2-rc2/ is a Zeppelin compiled with Scala 2.11. Do you have the same for
> > Scala 2.10 ?
> >
> > On Thu, Oct 13, 2016 at 9:59 AM, Jeff Zhang  wrote:
> >
> > > +1, Run tutorial note successfully in embedded mode and yarn-client.
> > >
> > >
> > >
> > > Mina Lee 于2016年10月12日周三 下午6:29写道:
> > >
> > > > Hi dev,
> > > >
> > > > I propose the following RC to be released for the Apache Zeppelin
> 0.6.2
> > > > release.
> > > >
> > > > The commit id is 091086de9400dd1c02ca02acf4180b1bf1e9ede7 which is
> > > > corresponds
> > > > to the tag v0.6.2-rc2:
> > > > *
> > > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > > <
> > > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > > >*
> > > >
> > > > The release archives (tgz), signature, and checksums are here
> > > > https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.6.2-rc2/
> > > >
> > > > The release candidate consists of the following source distribution
> > > archive
> > > > zeppelin-0.6.2.tgz
> > > >
> > > > In addition, the following supplementary binary distributions are
> > > provided
> > > > for user convenience at the same location
> > > > zeppelin-0.6.2-bin-all.tgz
> > > > zeppelin-0.6.2-netinst-all.tgz
> > > >
> > > > The maven artifacts are here
> > > > https://repository.apache
> > > > .org/content/repositories/orgapachezeppelin-1020/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=12336543&styleName=Html&projectId=12316221
> > > > <
> > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> > > version=12336543&styleName=Html&projectId=12316221
> > > > >*
> > > >
> > > > Vote will be open for next 72 hours (close at 4am 15/Oct PDT).
> > > >
> > > > [ ] +1 approve
> > > > [ ] 0 no opinion
> > > > [ ] -1 disapprove (and reason why)
> > > >
> > >
> >
>


Re: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)

2016-10-14 Thread Anthony Corbacho
Hi,

I tested this RC2 with JDK8, and no issue so far.

+1 for me.

Thanks to mina for the fast iteration on the release process, awesome as
always :)


On Thu, Oct 13, 2016 at 6:41 PM, DuyHai Doan  wrote:

> Hello, the link https://dist.apache.org/repos/
> dist/dev/zeppelin/zeppelin-0.
> 6.2-rc2/ is a Zeppelin compiled with Scala 2.11. Do you have the same for
> Scala 2.10 ?
>
> On Thu, Oct 13, 2016 at 9:59 AM, Jeff Zhang  wrote:
>
> > +1, Run tutorial note successfully in embedded mode and yarn-client.
> >
> >
> >
> > Mina Lee 于2016年10月12日周三 下午6:29写道:
> >
> > > Hi dev,
> > >
> > > I propose the following RC to be released for the Apache Zeppelin 0.6.2
> > > release.
> > >
> > > The commit id is 091086de9400dd1c02ca02acf4180b1bf1e9ede7 which is
> > > corresponds
> > > to the tag v0.6.2-rc2:
> > > *
> > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > <
> > > https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=
> > 091086de9400dd1c02ca02acf4180b1bf1e9ede7
> > > >*
> > >
> > > The release archives (tgz), signature, and checksums are here
> > > https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.6.2-rc2/
> > >
> > > The release candidate consists of the following source distribution
> > archive
> > > zeppelin-0.6.2.tgz
> > >
> > > In addition, the following supplementary binary distributions are
> > provided
> > > for user convenience at the same location
> > > zeppelin-0.6.2-bin-all.tgz
> > > zeppelin-0.6.2-netinst-all.tgz
> > >
> > > The maven artifacts are here
> > > https://repository.apache
> > > .org/content/repositories/orgapachezeppelin-1020/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=12336543&styleName=Html&projectId=12316221
> > > <
> > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> > version=12336543&styleName=Html&projectId=12316221
> > > >*
> > >
> > > Vote will be open for next 72 hours (close at 4am 15/Oct PDT).
> > >
> > > [ ] +1 approve
> > > [ ] 0 no opinion
> > > [ ] -1 disapprove (and reason why)
> > >
> >
>


[GitHub] zeppelin issue #1511: [ZEPPELIN-1544]upgrade spark version to 2.0.1 as profi...

2016-10-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1511
  
@LantaoJin Can you edit your description following the guide?


---
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 #1517: [MINOR] ui improvement

2016-10-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1517
  
@baekhoseok Thanks for your contribution and looks nice to me! 

Probably it's a nitpick, how about keeping consistency for that two buttons 
in interpreter setting page?

Here are two suggestions.
# 1. Icons with tooltips and no button border


![option_1](https://cloud.githubusercontent.com/assets/10060731/19382359/ce8b4b32-923a-11e6-80cd-7b238f655fdf.gif)

# 2. Icons with button border and no tooltip


![option_2](https://cloud.githubusercontent.com/assets/10060731/19382360/d0aea422-923a-11e6-972a-61ec2850bf79.gif)

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 #1467: [ZEPPELIN-1488] JDBC Interpreter throws error while th...

2016-10-14 Thread rajarajan-g
Github user rajarajan-g commented on the issue:

https://github.com/apache/zeppelin/pull/1467
  
ping


---
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 #1524: [ZEPPELIN-1542] Cleanup of Note created during ...

2016-10-14 Thread rajarajan-g
GitHub user rajarajan-g opened a pull request:

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

[ZEPPELIN-1542]  Cleanup of Note created during jUnit test in 
zeppelin-server, zeppelin-zengine

### What is this PR for?
While running jUnit tests in zeppelin-server & zeppelin-zengine projects, 
several note are created during the process and those notes are never purged 
after the execution of the tests.

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

### Todos

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

### How should this be tested?
Prerequisites:
1. Get the list of note that already exist in server.

Now build zeppelin project by running maven commands(mvn clean package)(mvn 
clean package -pl 'zeppelin-interpreter,zeppelin-zengine,zeppelin-server').
1. Now no extra note should get created after running the maven command

### 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/rajarajan-g/zeppelin ZEPPELIN-1542

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

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


commit d97cb4a8530b253665240453dccaa112d137b451
Author: rajarajan-g 
Date:   2016-10-14T09:01:12Z

code changes for ZEPPELIN-1542




---
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 #1522: [ ZEPPELIN-1461 ] Doesn't display "description" value ...

2016-10-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1522
  
Thanks for your contribution @kjhong :)
Tested and it works well. Appreciate for bring it back 👍 


---
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-1551) Default interpreter is different at various places

2016-10-14 Thread Kavin Kumar (JIRA)
Kavin Kumar created ZEPPELIN-1551:
-

 Summary: Default interpreter is different at various places
 Key: ZEPPELIN-1551
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1551
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Reporter: Kavin Kumar
Priority: Minor


Create a note and check the system default interpreter in the interpreter 
section, it would be "md". Now navigate to the interpreter page and check the 
default one, it would be "alluxio" (refer to the screenshots).



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


[GitHub] zeppelin issue #1517: [MINOR] ui improvement

2016-10-14 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1517
  
Change looks great to me, that you for your contribution.
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 pull request #1523: fixed 'add from URL' button

2016-10-14 Thread yunho0130
GitHub user yunho0130 opened a pull request:

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

fixed 'add from URL' button

### What is this PR for?

fixed 'add from URL' button and disable backdrop function.

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

### Todos
None

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

### How should this be tested?
click 'Import note' -> click 'Add from URL' -> click 'x' button -> open the 
dialog again -> it shows the URL page not the initial page.

### Screenshots (if appropriate)

 - before

![import_note_window](https://cloud.githubusercontent.com/assets/7574765/19379302/36a0fc12-922c-11e6-9018-c67893305005.gif)

 - after

![after_2](https://cloud.githubusercontent.com/assets/7574765/19379491/2c5a072a-922d-11e6-8804-55c9766b3c5c.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/yunho0130/zeppelin ZEPPELIN-1550

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

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


commit a0459287b41b0d2daf50ff7c485191caf3048397
Author: Yunho Maeng 
Date:   2016-10-14T07:29:52Z

fixed 'add from URL' button and disable backdrop fuction when user use 
modal window




---
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 #1522: [ ZEPPELIN-1461 ] Doesn't display "description"...

2016-10-14 Thread kjhong
GitHub user kjhong opened a pull request:

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

[ ZEPPELIN-1461 ] Doesn't display "description" value in interpreter 
creation page

### What is this PR for?
As you can see in the attached screenshot image, "description" value 
doesn't show up in interpreter creation page. Moreover, the "+" (action button) 
is not working as well.


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

### Todos
- [x] modification - interpreterfactory description 
- [x] modification - interpreter page

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1461?jql=project%20%3D%20ZEPPELIN%20AND%20text%20~%20%22description%22

### How should this be tested?
1. on click to zeppelin interpreter page.


### Screenshots (if appropriate)
 after
https://cloud.githubusercontent.com/assets/10525473/19378483/d8f20ae2-9227-11e6-9e73-7894fd5064de.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/kjhong/zeppelin ZEPPELIN-1461-r

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

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


commit 2e5fa75abadbaf81f1d65cd558eeec01941449e8
Author: kjhong 
Date:   2016-10-14T07:40:49Z

interpreter description error fixed.




---
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 #1519: [ ZEPPELIN-1461 ] Doesn't display "description"...

2016-10-14 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


---
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 #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-14 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@jongyoul thank you for review. Yes, and I'm still working in process. I'll 
fix more files.


---
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 #1521: [ ZEPPELIN-1461 ] Doesn't display "description"...

2016-10-14 Thread kjhong
Github user kjhong closed the pull request at:

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


---
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 #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-14 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1520
  
The changes are 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 pull request #1521: [ ZEPPELIN-1461 ] Doesn't display "description"...

2016-10-14 Thread kjhong
GitHub user kjhong opened a pull request:

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

[ ZEPPELIN-1461 ] Doesn't display "description" value in interpreter 
creation page

### What is this PR for?
As you can see in the attached screenshot image, "description" value 
doesn't show up in interpreter creation page. Moreover, the "+" (action button) 
is not working as well.


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

### Todos
- [x] modification - interpreterfactory description 
- [x] modification - interpreter page

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1461?jql=project%20%3D%20ZEPPELIN%20AND%20text%20~%20%22description%22

### How should this be tested?
1. on click to zeppelin interpreter page.


### Screenshots (if appropriate)
 after
https://cloud.githubusercontent.com/assets/10525473/19378483/d8f20ae2-9227-11e6-9e73-7894fd5064de.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/kjhong/zeppelin ZEPPELIN-1461

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

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


commit 3ee2476ca9d1c50e539264b51967f1078e7f6f63
Author: CloverHearts 
Date:   2016-10-14T07:26:42Z

interpreter description error fixed




---
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 #1519: [ ZEPPELIN-1461 ] Doesn't display "description" value ...

2016-10-14 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1519
  
```
### What is this PR for?
As you can see in the attached screenshot image, "description" value 
doesn't show up in interpreter creation page. Moreover, the "+" (action button) 
is not working as well.


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

### Todos
- [x] modification - interpreterfactory description 
- [x] modification - interpreter page

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1461?jql=project%20%3D%20ZEPPELIN%20AND%20text%20~%20%22description%22

### How should this be tested?
1. on click to zeppelin interpreter page.
2. blabla

### Screenshots (if appropriate)
 after
https://cloud.githubusercontent.com/assets/10525473/19378483/d8f20ae2-9227-11e6-9e73-7894fd5064de.png";>

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


```


---
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 #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1520
  
WIP?


---
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 #1520: [ZEPPELIN-1061] Select default interpreter whil...

2016-10-14 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request:

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

[ZEPPELIN-1061] Select default interpreter while creating note

### What is this PR for?
This handles setting the default interpreter on creating a note through the 
zeppelin UI.

### What type of PR is it?
Feature

### Todos
NA

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

### How should this be tested?
* Go to the 'Create Note' dialog and provide the name with the default 
interpreter selected.After clicking on the Create button, the selected 
interpreter should be shown properly in the interpreter binding section.
* If there is no interpreter selected, then the system default will be used.

### Screenshots (if appropriate)

![zeppelin-1061](https://cloud.githubusercontent.com/assets/20789766/19378611/200d9d50-920b-11e6-930a-6fdcf67c5215.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/kavinkumarks/zeppelin 
zeppelin-1061-create-note-default-interpreter

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

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


commit c0bd6fb983c40771bf5c6e619bd19c54c7142c1c
Author: Kavin 
Date:   2016-10-13T11:02:15Z

Added option to select the default interpreter on creating a note and
added websocket APIs to get interpreter settings for the former.Added
test cases too.




---
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 #1517: [MINOR] ui improvement

2016-10-14 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1517
  
Looks good to me 
It's very clear :)


---
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 #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@hyonzin @cloverhearts Do you change these two files in 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 #1519: [ ZEPPELIN-1461 ] Doesn't display "description"...

2016-10-14 Thread cloverhearts
GitHub user cloverhearts opened a pull request:

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

[ ZEPPELIN-1461 ] Doesn't display "description" value in interpreter 
creation page

### What is this PR for?
As you can see in the attached screenshot image, "description" value 
doesn't show up in interpreter creation page. Moreover, the "+" (action button) 
is not working as well.


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

### Todos
- [x] modification - interpreterfactory description 
- [x] modification - interpreter page

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1461?jql=project%20%3D%20ZEPPELIN%20AND%20text%20~%20%22description%22

### How should this be tested?
1. on click to zeppelin interpreter page.
2. blabla

### Screenshots (if appropriate)
 after
https://cloud.githubusercontent.com/assets/10525473/19378483/d8f20ae2-9227-11e6-9e73-7894fd5064de.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 
optimazation-notebooklist

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

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


commit 107ed8c2bb013039fef0ff0eb72aa2920f618461
Author: CloverHearts 
Date:   2016-10-14T06:59:02Z

zeppelin description bug fixed - d




---
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 #1517: [MINOR] ui improvement

2016-10-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1517
  
LGTM. It's simple and clean. :-)


---
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 #1518: [ZEPPELIN-1549] Change NotebookID variable name...

2016-10-14 Thread hyonzin
GitHub user hyonzin opened a pull request:

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

[ZEPPELIN-1549] Change NotebookID variable name to NoteID

### What is this PR for?
This PR fixes wrong written NotebookID to NoteID.


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


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


### 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/hyonzin/zeppelin ZEPPELIN-1549

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

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


commit cc0d3151efab5280a1ef9cdd61e965943e113be2
Author: hyonzin 
Date:   2016-10-14T06:55:57Z

Change NotebookID variable name to NoteID




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