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

2016-10-24 Thread asfgit
Github user asfgit 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 issue #1553: [ZEPPELIN-1575] Notebook Repo settings UI

2016-10-24 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1553 @khalidhuseynov added some tests --- 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 #1471: Closes [ZEPPELIN-1505] Add Scio interpreter

2016-10-24 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1471 ah, sorry, I added a reply but it didn't go through apparently. I think that's good - I'm not sure if scio should be the default of the beam group (aren't there more Java users?)

[GitHub] zeppelin issue #1553: [ZEPPELIN-1575] Notebook Repo settings UI

2016-10-24 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1553 @anthonycorbacho thanks for suggesting to change repo settings via UI. before testing, do you think we need some unit tests for repo settings feature? --- If your project is set up for

[GitHub] zeppelin pull request #1555: [MINOR] ui improvement

2016-10-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1555 --- 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 #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-10-24 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1538 @mfelgamal first of all thanks for initiative on this one, and few points here: * as i can see here Zeppelin images will be with hardcoded `$ZEPPELIN_HOME/conf` folder and in order

[GitHub] zeppelin issue #1472: Add message of localhost:port on script

2016-10-24 Thread djKooks
Github user djKooks commented on the issue: https://github.com/apache/zeppelin/pull/1472 @AhyoungRyu Sorry for late reply. I updated scripts to read port number from environment setups. Please look on and let me know if there is any problem on running this. --- If your project

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

2016-10-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1462 ping @felixcheung @prabhjyotsingh --- 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 #1339: [ZEPPELIN-1332] Remove spark-dependencies & suggest ne...

2016-10-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1339 @Leemoonsoo @jongyoul Sorry for my late update. I've just added new option: `get-spark` to

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

2016-10-24 Thread ravwojdyla
Github user ravwojdyla commented on the issue: https://github.com/apache/zeppelin/pull/1471 ping @felixcheung --- 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

[GitHub] zeppelin issue #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-10-24 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1538 @mfelgamal I checked point 1 and 3. Thank you fix properly. Could you also fix to run the `R Tutorial` and `Python Tutorial` too? I think all tutorial should be ran without error.

[GitHub] zeppelin issue #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1518 CI passed in https://travis-ci.org/hyonzin/zeppelin/builds/170100291. Merge if there is no further discussion. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1539: [ZEPPELIN-1567] Let JDBC interpreter use user credenti...

2016-10-24 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1539 @khalidhuseynov Thank you for sharing your good idea about the dropdown list. It might gives convenience users. and I agree with @Leemoonsoo about define convention for `entity` name.

[jira] [Created] (ZEPPELIN-1580) JDBC - SQL IN CLAUSE with Form Template does not always

2016-10-24 Thread Sarvesh N (JIRA)
Sarvesh N created ZEPPELIN-1580: --- Summary: JDBC - SQL IN CLAUSE with Form Template does not always Key: ZEPPELIN-1580 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1580 Project: Zeppelin

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

2016-10-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1559 \cc @jongyoul @cloverhearts @kjhong for 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

[GitHub] zeppelin pull request #1559: [ZEPPELIN-1461] Doesn't display "description" v...

2016-10-24 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/zeppelin/pull/1559 [ZEPPELIN-1461] Doesn't display "description" value in interpreter creation page ### What is this PR for? #1522 tried to fix same issue by making as least change as possible, but fixing it

[GitHub] zeppelin pull request #1558: ZEPPELIN-1562 updated api doc

2016-10-24 Thread snaveenp
Github user snaveenp closed the pull request at: https://github.com/apache/zeppelin/pull/1558 --- 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 #1558: ZEPPELIN-1562 updated api doc

2016-10-24 Thread snaveenp
GitHub user snaveenp reopened a pull request: https://github.com/apache/zeppelin/pull/1558 ZEPPELIN-1562 updated api doc ### What is this PR for? The URL for running a paragraph synchronously using REST api is mistakenly given as

[GitHub] zeppelin issue #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-24 Thread hyonzin
Github user hyonzin commented on the issue: https://github.com/apache/zeppelin/pull/1518 @minahlee I have fixed it now. Really thank you for your great help :) --- 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 issue #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1518 @hyonzin I missed one file... https://github.com/apache/zeppelin/blob/master/zeppelin-server/src/test/java/org/apache/zeppelin/socket/NotebookServerTest.java#L157 in this line, `notebook` should

[GitHub] zeppelin issue #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-24 Thread hyonzin
Github user hyonzin commented on the issue: https://github.com/apache/zeppelin/pull/1518 @minahlee Thank you. I'm going to rebase it! I was checking the failure in test [here](https://travis-ci.org/hyonzin/zeppelin). --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1558: ZEPPELIN-1562 updated api doc

2016-10-24 Thread snaveenp
GitHub user snaveenp opened a pull request: https://github.com/apache/zeppelin/pull/1558 ZEPPELIN-1562 updated api doc ### What is this PR for? The URL for running a paragraph synchronously using REST api is mistakenly given as

[GitHub] zeppelin pull request #1290: [ZEPPELIN-708]shift positions automatica...

2016-10-24 Thread mwkang
Github user mwkang closed the pull request at: https://github.com/apache/zeppelin/pull/1290 --- 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 #1350: [ZEPPELIN-1351]Paragraph status should be ERROR...

2016-10-24 Thread mwkang
Github user mwkang closed the pull request at: https://github.com/apache/zeppelin/pull/1350 --- 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 #1290: [ZEPPELIN-708]shift positions automatically to ...

2016-10-24 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1290 I am not sure what is my next step. And this PR was created very long time ago. So enviroment changed before. I think other contributor can resolve better than me. So I closed this PR. Thanks

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-10-24 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1350 I am not sure what is my next step. And this PR was created very long time ago. So enviroment changed before. I think other contributor can resolve better than me. So I closed this PR. Thanks

[GitHub] zeppelin issue #1556: [MINOR] remove comments in NotebookRestApi.java

2016-10-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1556 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 feature

[GitHub] zeppelin pull request #1537: [Zeppelin-1561] Improve sync for multiuser envi...

2016-10-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1537 --- 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 #1556: [MINOR] remove comments in NotebookRestApi.java

2016-10-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1556 LGTM. CI failures is irrelevant. --- 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

[jira] [Created] (ZEPPELIN-1578) Customize Apache Zeppelin homepage > Show notebook list > Notebook corrupted.

2016-10-24 Thread Andreas Weise (JIRA)
Andreas Weise created ZEPPELIN-1578: --- Summary: Customize Apache Zeppelin homepage > Show notebook list > Notebook corrupted. Key: ZEPPELIN-1578 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1578

[GitHub] zeppelin issue #1557: ZEPPELIN-1577. LivyInterpreter should not use FIFOSche...

2016-10-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1557 Will add test after #1462 --- 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 pull request #1557: ZEPPELIN-1577. LivyInterpreter should not use F...

2016-10-24 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1557 ZEPPELIN-1577. LivyInterpreter should not use FIFOScheduler ### What is this PR for? This PR create a new scheduler FIFOPerUserScheduler. So that each user's request is isolated. and it is

[jira] [Created] (ZEPPELIN-1577) LivyInterpreter should not use FIFOScheduler

2016-10-24 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-1577: Summary: LivyInterpreter should not use FIFOScheduler Key: ZEPPELIN-1577 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1577 Project: Zeppelin Issue

[GitHub] zeppelin issue #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1518 @hyonzin I found more places to be changed. There were too much of them to comment here so I made PR to your repository. Can you please check and merge it? --- If your project is set up for it,