[GitHub] zeppelin issue #1545: ZEPPELIN-1571. Support pig udf interpreter

2016-10-20 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1545 @minahlee The syntax highlight for java doesn't work, could you help to check what's I missing here ? Thanks --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] zeppelin issue #1545: ZEPPELIN-1571. Support pig udf interpreter

2016-10-20 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1545 @felixcheung @AhyoungRyu Please help review --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featu

[GitHub] zeppelin pull request #1546: [ Hotfix ] Incorrect (per user) setup ui for "c...

2016-10-20 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1546 [ Hotfix ] Incorrect (per user) setup ui for "create new interpreter component" ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. F

[GitHub] zeppelin pull request #1545: ZEPPELIN-1571. Support pig udf interpreter

2016-10-20 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1545 ZEPPELIN-1571. Support pig udf interpreter ### What is this PR for? Support a new interpreter `%pig.udf` for Pig Interpreter group. User can write pig udf in this interpreter, and will compile

[jira] [Created] (ZEPPELIN-1571) Support pig udf interpreter

2016-10-20 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1571: Summary: Support pig udf interpreter Key: ZEPPELIN-1571 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1571 Project: Zeppelin Issue Type: New Feature

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

2016-10-20 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1517 @baekhoseok Tested and it looks good! You chose second option :) https://cloud.githubusercontent.com/assets/10060731/19586063/9f5f8646-978c-11e6-814c-623fb8d5a063.png";> This is the curre

[GitHub] zeppelin issue #1544: spelling error

2016-10-20 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1544 Thanks for the contribution! could you modify PR description template with little info and address the comment. --- If your project is set up for it, you can reply to this email and have y

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

2016-10-20 Thread ravwojdyla
Github user ravwojdyla commented on the issue: https://github.com/apache/zeppelin/pull/1471 @felixcheung after going through the `BeamInterpreter`, I don't see immediate technical advantage of sharing the group, current implementation of `BeamInterpreter`, is actually pure Java interp

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

2016-10-20 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1471 I think that's strictly cosmetic - you can see the Spark/R interpreter on how multiple interpreter classes are going to the same group. I think this helps discoverability and longer te

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

2016-10-20 Thread ravwojdyla
Github user ravwojdyla commented on the issue: https://github.com/apache/zeppelin/pull/1471 @felixcheung I see, thanks for help, is 95dd9bb enough? --- 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

[GitHub] zeppelin issue #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-20 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1462 As mentioned in the PR description, for now you need to export SPARK_HOME and LIVY_HOME. I plan to merge with travis in the following PR. --- If your project is set up for it, you can reply to th

[GitHub] zeppelin issue #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-20 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1462 I just tried it on my local; so basically it skips all the test cases. ``` WARN [2016-10-20 18:17:05,017] ({main} LivyIntegrationTest.java[checkPreCondition]:66) - livy integrat

[GitHub] zeppelin pull request #1544: spelling error

2016-10-20 Thread hunylee
GitHub user hunylee opened a pull request: https://github.com/apache/zeppelin/pull/1544 spelling error ### 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.

[GitHub] zeppelin issue #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-20 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1462 Otherwise yes, 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 w

[GitHub] zeppelin issue #1540: [WIP][ZEPPELIN-1566] Make paragraph editable with doub...

2016-10-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1540 How about making editOnDblClick false by default? --- 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 issue #1522: [ ZEPPELIN-1461 ] Doesn't display "description" value ...

2016-10-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1522 I think this PR tries to make less change for the dealing with that problem. Actually the most right way changes a lot including backend. it also can break backward compatibility. --- If your p

[GitHub] zeppelin issue #1467: [ZEPPELIN-1488] JDBC Interpreter throws error while th...

2016-10-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1467 LGTM. Merging 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 featur

[GitHub] zeppelin issue #1542: [MINOR][DOC] Fix typo (re-open PR #1541 with rebase)

2016-10-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1542 Thanks for the fixing typo --- 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 w

[GitHub] zeppelin pull request #1543: Add missing TOC(Table of Contents) in install_w...

2016-10-20 Thread oganizang
GitHub user oganizang opened a pull request: https://github.com/apache/zeppelin/pull/1543 Add missing TOC(Table of Contents) in install_with_flink_and_spark_cluster.md ### What is this PR for? Add missing TOC(Table of Contents) in install_with_flink_and_spark_cluster.md

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

2016-10-20 Thread baekhoseok
Github user baekhoseok commented on the issue: https://github.com/apache/zeppelin/pull/1517 @jongyoul @soralee @anthonycorbacho @AhyoungRyu Thank you for review ^^ @AhyoungRyu Thank you for suggestion. I fixed it. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1542: [MINOR][DOC] Fix typo (re-open PR #1541 with re...

2016-10-20 Thread laesunk
GitHub user laesunk opened a pull request: https://github.com/apache/zeppelin/pull/1542 [MINOR][DOC] Fix typo (re-open PR #1541 with rebase) ### What is this PR for? Fix minor typos in docs ### What type of PR is it? Documentation ### Questions: * Does t

[GitHub] zeppelin pull request #1541: [MINOR][DOC] Fix typo

2016-10-20 Thread laesunk
Github user laesunk closed the pull request at: https://github.com/apache/zeppelin/pull/1541 --- 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 #1541: [MINOR][DOC] Fix typo

2016-10-20 Thread laesunk
GitHub user laesunk opened a pull request: https://github.com/apache/zeppelin/pull/1541 [MINOR][DOC] Fix typo ### What is this PR for? Fix minor typos in docs ### What type of PR is it? Documentation ### Questions: * Does the licenses files need update?

[GitHub] zeppelin issue #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1520 @kavinkumarks Backend codes looks good to me. @corneadoug Could you please look into it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

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

2016-10-20 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom / @zjffdu @purechoc there is definitely an additional condition. Not sure if it's because the ConcurrentHashMaps are not being used correctly. But the exception doesn't get caught complet

[jira] [Created] (ZEPPELIN-1570) note default owner is not set when create a note

2016-10-20 Thread liupengcheng (JIRA)
liupengcheng created ZEPPELIN-1570: -- Summary: note default owner is not set when create a note Key: ZEPPELIN-1570 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1570 Project: Zeppelin I

[GitHub] zeppelin pull request #1454: [ZEPPELIN-1483] Zeppelin home page list noteboo...

2016-10-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1454 --- 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 #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-20 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom just tested against my build by catching the exception and rethrowing.. It definitely solves the issue. } catch (LivyNoSessionException e) { throw e; } ca

[GitHub] zeppelin issue #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-20 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1462 @felixcheung @prabhjyotsingh I have published livy jars to OSSRH, and now I removed the livy jars. Please help review. --- If your project is set up for it, you can reply to this email and have y