[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

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 > Sent: Friday, October 14, 2016 5:28 AM Subject: Re: [VOTE] Release Apache Zeppelin 0.6.2 (RC2) To:

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

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

[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

[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

[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

[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

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

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 >

[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

[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

[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

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

[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

[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

[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

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

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

[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

[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

[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

Reg :: Issue while raising pull request

2016-10-14 Thread Rajarajan Ganesan
Hi, When i was working on an issue, i tried to create pull request. i ended up having a issue like below The steps i followed while doing that 1. git add 'files' 2. git commit 3. git push origin master After that i was following the rebase process, once i resolved the conflicts, i pushed

[GitHub] zeppelin issue #1471: Closes [ZEPPELIN-1505] Add Scio interpreter

2016-10-14 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1471 I don't have more comments. \cc @Leemoonsoo @felixcheung if they have comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[jira] [Created] (ZEPPELIN-1549) NotebookID in the source code should be NoteID

2016-10-14 Thread Hyeonjin Jung (JIRA)
Hyeonjin Jung created ZEPPELIN-1549: --- Summary: NotebookID in the source code should be NoteID Key: ZEPPELIN-1549 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1549 Project: Zeppelin

[jira] [Created] (ZEPPELIN-1550) Import note closing refresh error

2016-10-14 Thread Yunho Maeng (JIRA)
Yunho Maeng created ZEPPELIN-1550: - Summary: Import note closing refresh error Key: ZEPPELIN-1550 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1550 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin pull request #1510: [ZEPPELIN-675] "No data available" in invalid g...

2016-10-14 Thread snaveenp
Github user snaveenp closed the pull request at: https://github.com/apache/zeppelin/pull/1510 --- 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 #1517: [MINOR] ui improvement

2016-10-14 Thread baekhoseok
GitHub user baekhoseok opened a pull request: https://github.com/apache/zeppelin/pull/1517 [MINOR] ui improvement ### What is this PR for? * configuration sorting * remove tooltip in job menu * change repository infomation icon ### What type of PR is it?

[GitHub] zeppelin pull request #1510: [ZEPPELIN-675] "No data available" in invalid g...

2016-10-14 Thread snaveenp
GitHub user snaveenp reopened a pull request: https://github.com/apache/zeppelin/pull/1510 [ZEPPELIN-675] "No data available" in invalid graph types for sql queries ### What is this PR for? Overriding graph message 'No data available' to 'Invalid Data, check graph settings'

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

[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

[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 LGTM. Merging if there's no more discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

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

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

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

[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

[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

[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

[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

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