[GitHub] zeppelin pull request #1550: [ZEPPELIN-912] Apply new mechanism to GeodeOqlI...

2016-10-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1550 --- 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-23 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1520 @jongyoul & @AhyoungRyu thanks for reviewing it. @AhyoungRyu I will look into the issue. @corneadoug any updates on this? Thanks, Kavin MailTo: kavin.ku...@imaginea.com

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

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1539 it would be even better to let user chose interpreter from dropdown list of interpreters in place of `entity` field in credentials menu, what do you think @Leemoonsoo @astroshim ? Also

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

2016-10-23 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1555 Tested and LGTM. Merge if there is 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

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

2016-10-23 Thread vectorijk
GitHub user vectorijk opened a pull request: https://github.com/apache/zeppelin/pull/1556 [MINOR] remove comments in NotebookRestApi.java ### What is this PR for? Since ZEPPELIN-1162 (PR #1341) has been solved, we should remove TODO comments correspondingly. ### What

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

2016-10-23 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1540 It looks good to hide editor on md. And I'm not good at ui, but in case of angular, I think it's better to hide it, too. --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1479: Notebook: HDFS as a backend storage (Read & Write Mode...

2016-10-23 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1479 @hayssams Do you have any clue that you mentioned? I don't know why two different jars while building Zeppelin. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin pull request #1460: [ZEPPELIN-1490] pass subject while persisting n...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at: https://github.com/apache/zeppelin/pull/1460 --- 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 #1460: [ZEPPELIN-1490] pass subject while persisting note on ...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1460 similar approach was used as a part of #1265 to address this issue and is merged already, closing this one --- If your project is set up for it, you can reply to this email and have your

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

2016-10-23 Thread baekhoseok
GitHub user baekhoseok opened a pull request: https://github.com/apache/zeppelin/pull/1555 [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 issue #1537: [Zeppelin-1561] Improve sync for multiuser environment...

2016-10-23 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue: https://github.com/apache/zeppelin/pull/1537 LGTM merging if 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 issue #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1462 @felixcheung I think it is not necessary as I didn't find junit in https://github.com/apache/zeppelin/blob/master/zeppelin-distribution/src/bin_license/LICENSE --- If your project is set up for

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

2016-10-23 Thread mfelgamal
Github user mfelgamal commented on the issue: https://github.com/apache/zeppelin/pull/1538 @astroshim Thank you for your reviews, Can you check the docker image again? --- 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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-23 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1462 One question - I'm not sure if dependencies for test only code should be listed in LICENSE. Otherwise LGTM. --- If your project is set up for it, you can reply to this email and have

[jira] [Created] (ZEPPELIN-1576) Bump Flink to 1.1.3

2016-10-23 Thread Trevor Grant (JIRA)
Trevor Grant created ZEPPELIN-1576: -- Summary: Bump Flink to 1.1.3 Key: ZEPPELIN-1576 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1576 Project: Zeppelin Issue Type: Bug

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

2016-10-23 Thread hyonzin
Github user hyonzin commented on the issue: https://github.com/apache/zeppelin/pull/1518 @minahlee Thank you. I rebase it and solve conflicts now. Please confirm it! [https://travis-ci.org/hyonzin/zeppelin](https://travis-ci.org/hyonzin/zeppelin) --- If your project is set up

[GitHub] zeppelin issue #1479: Notebook: HDFS as a backend storage (Read & Write Mode...

2016-10-23 Thread hayssams
Github user hayssams commented on the issue: https://github.com/apache/zeppelin/pull/1479 @jongyoul @zjffdu This issue is with hadoop 2.7 only. With hadoop 2.6 making hadoop dependency as provided works well. I get the following exception Caused by:

[GitHub] zeppelin issue #1537: [Zeppelin-1561] Improve sync for multiuser environment...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1537 CI is green, ready 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 feature

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

2016-10-23 Thread baekhoseok
Github user baekhoseok closed the pull request at: https://github.com/apache/zeppelin/pull/1517 --- 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 #1537: [Zeppelin-1561] Improve sync for multiuser envi...

2016-10-23 Thread khalidhuseynov
GitHub user khalidhuseynov reopened a pull request: https://github.com/apache/zeppelin/pull/1537 [Zeppelin-1561] Improve sync for multiuser environment ### What is this PR for? apply multi-tenancy for storage sync mechanism ### What type of PR is it? Bug Fix |

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

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov 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

[GitHub] zeppelin issue #1537: [Zeppelin-1561] Improve sync for multiuser environment...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1537 @Leemoonsoo your comment makes sense if there's already existing acl for the note. addressed it with test under e41b9b9324971b472a9210b03c189f01ffc6a0f3 --- If your project is set up for

[GitHub] zeppelin issue #1554: [WIP] [ZEPPELIN-1320] Run zeppelin interpreter process...

2016-10-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1554 know issue: restart on interpreter does not work as expected. --- 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 #1554: [ZEPPELIN-1320] Run zeppelin interpreter proces...

2016-10-23 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1554 [ZEPPELIN-1320] Run zeppelin interpreter process as web front end user Have recreated this from https://github.com/apache/zeppelin/pull/1322 ### What is this PR for?

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

2016-10-23 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1538 @mfelgamal Thank you for contributing this. I have several questions. 1. I think it needs `-it` option for running docker. so The command to run zeppelin docker should be

[GitHub] zeppelin pull request #1322: [ZEPPELIN-1320] Run zeppelin interpreter proces...

2016-10-23 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/1322 --- 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 #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-10-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1322 Closing this, will open a new one with merge of https://github.com/apache/zeppelin/pull/1265. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-10-23 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @jongyoul I totally agree with you adding multi-tenancy but how about separating that to another PR because we also already have restarting interpreter function in the `Interpreter Setting

[GitHub] zeppelin pull request #1553: [ZEPPELIN-1575] [WIP] Notebook Repo settings UI

2016-10-23 Thread anthonycorbacho
GitHub user anthonycorbacho opened a pull request: https://github.com/apache/zeppelin/pull/1553 [ZEPPELIN-1575] [WIP] Notebook Repo settings UI ### What is this PR for? Let user configure/change notebook repo setting via Zeppelin UI We can imagine one case with

[jira] [Created] (ZEPPELIN-1575) Notebook Repo settings UI

2016-10-23 Thread Anthony Corbacho (JIRA)
Anthony Corbacho created ZEPPELIN-1575: -- Summary: Notebook Repo settings UI Key: ZEPPELIN-1575 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1575 Project: Zeppelin Issue Type: