[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

[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

[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

[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

[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 stal

[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 you

[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

[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

[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: Inter

[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

[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 doe

[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 GitHu

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, 20

[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 fe

[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 featu

[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 fea

[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

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 y

[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

[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

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

[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 em

[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

[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

[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

[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 contributin

[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 g

[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

[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 proj

[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: Zep

[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 r

[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 thi

[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 enab

[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

[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 thi

[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

[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 Function

[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

[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

[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 be

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

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/

[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

[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 ar

[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

[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 pr

[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 ap

[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 I

[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 yo

[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 No

[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 u

[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 featur

[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

[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 e

[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 wi

[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 u

[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 no

[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 fe

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

[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

[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 do

[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

[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

[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]