[GitHub] zeppelin pull request #1407: [ZEPPELIN-1412] add support multiline for pytho...

2016-09-06 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1407 [ZEPPELIN-1412] add support multiline for pythonErrorIn method on python interpreter ### What is this PR for? currently, has not support multiline exception text on python interpreter.

[GitHub] zeppelin issue #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1396 Thanks! I'll look into it. btw, don't you have any problem when you were testing mysql? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Created] (ZEPPELIN-1415) Zeppelin-Web Good Practices #2 - Event Dispatching

2016-09-06 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-1415: Summary: Zeppelin-Web Good Practices #2 - Event Dispatching Key: ZEPPELIN-1415 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1415 Project: Zeppelin

[GitHub] zeppelin issue #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-06 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 Thanks @jongyoul and there was no problem running mysql. --- 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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 @khalidhuseynov thanks for the review! I have replied to the comments and the CI build is in progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #1406: [ZEPPELIN-1414] Remove CONTRIBUTING.md files

2016-09-06 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/zeppelin/pull/1406 [ZEPPELIN-1414] Remove CONTRIBUTING.md files ### What is this PR for? Following the PR #1356, we now have the contribution documentation to on Apache Zeppelin Website. We can now

[GitHub] zeppelin issue #1359: [ZEPPELIN-966] job manager change information collecti...

2016-09-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1359 @cloverhearts Okay i'll check this tomorrow --- 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 #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-06 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1384 @Leemoonsoo @felixcheung Are we good on that PR? CI is failing on tests, not related to this PR as its happening to other PRs. But would be better to merge this one after CI is fixed

[GitHub] zeppelin pull request #1387: [ZEPPELIN-1398] Use relative path for search_da...

2016-09-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1387 --- 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 #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1384 Because now github-markdown-css is downloaded on build time instead of packed in the source tree, related license now need to moved back to under bin_licenses. --- If your project is set up

[GitHub] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-06 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1384 @1ambda 👍 fot tests, looks great to me except for licensing issue raised by Moon and small comment above. If you could let me know you user name in ASF JIRA I'll be happy to help

[GitHub] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] [ZEPPELIN-1338] ...

2016-09-06 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1390 @khalidhuseynov - have fixed the format issues - I didn't change Notebook#getNotebookAuthorization, it was already present. However, a single global NotebookAuthorization as you

[GitHub] zeppelin issue #1407: [ZEPPELIN-1412] add support multiline for pythonErrorI...

2016-09-06 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1407 Thank you for improvements, @cloverhearts Looks good to me. Although as discussed under [ZEPPELIN-1360](https://issues.apache.org/jira/browse/ZEPPELIN-1360) - this approach have

[GitHub] zeppelin issue #1405: Update Zeppelin Website building instructions

2016-09-06 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1405 @corneadoug There is one more `--safe` in here :

[GitHub] zeppelin issue #1359: [ZEPPELIN-966] job manager change information collecti...

2016-09-06 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1359 @Leemoonsoo Thank you very much! --- 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 #1406: [ZEPPELIN-1414] Remove CONTRIBUTING.md files

2016-09-06 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1406 @AhyoungRyu Yes, this one could also be replaced. There is actually a few pages of content inside the docs directory that we may want to move to the website later, so maybe we should create

[GitHub] zeppelin issue #1407: [ZEPPELIN-1412] add support multiline for pythonErrorI...

2016-09-06 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1407 @bzz thank you. This code works with both Python 3 and 2. In this code, the part that related to the Python version does not exist except `print (" Exception ( "test exception"))

[GitHub] zeppelin issue #1401: [ZEPPELIN-1409] Refactor RAT build on Travis.CI config...

2016-09-06 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1401 Looks good to me, merging if there is no further discussion. CI fails on single profile with 1 flaky test ``` [INFO] Zeppelin: Server ... FAILURE

[GitHub] zeppelin pull request #1401: [ZEPPELIN-1409] Refactor RAT build on Travis.CI...

2016-09-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1401 --- 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 #1406: [ZEPPELIN-1414] Remove CONTRIBUTING.md files

2016-09-06 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1406 @corneadoug There is one more "how to contribute" link - [http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/#100-opensource](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/#100-opensource) :)

[GitHub] zeppelin issue #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-06 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1334 After running it locally, I think it is great but the last thing that would be very nice to have, is to expand the example in the docs to be self-contained, meaning easily runnable by the the

[GitHub] zeppelin pull request #1408: [ZEPPELIN-808] Changed Permissions on Notebooks...

2016-09-06 Thread kavinkumarks
GitHub user kavinkumarks opened a pull request: https://github.com/apache/zeppelin/pull/1408 [ZEPPELIN-808] Changed Permissions on Notebooks do not provide helpful error to user ### What is this PR for? This is about showing information to the user when there are errors on

[GitHub] zeppelin pull request #1393: [ZEPPELIN-1142] Zeppelin allows two users to si...

2016-09-06 Thread kavinkumarks
GitHub user kavinkumarks reopened a pull request: https://github.com/apache/zeppelin/pull/1393 [ZEPPELIN-1142] Zeppelin allows two users to simultaneously edit the notebook permissions ### What is this PR for? This is about storing the owner information on creating a note so

[GitHub] zeppelin pull request #1393: [ZEPPELIN-1142] Zeppelin allows two users to si...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks closed the pull request at: https://github.com/apache/zeppelin/pull/1393 --- 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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 Found an build test failure, working on it. --- 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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-06 Thread kavinkumarks
Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1393 Reopening to trigger the CI build. --- 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 #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1265 @corneadoug I summarize all of unique combination. I think 'Per user' is the first criteria and 'Per note' is the second priority. And I also have a question. Can front-end know if shiro is

[GitHub] zeppelin pull request #1379: [ZEPPELIN-1116]send out more exception msg

2016-09-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1379 --- 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 #1379: [ZEPPELIN-1116]send out more exception msg

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1379 @passionke Can you tell me your jira id? --- 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 #1380: ZEPPELIN-1326: make profile to select dependency of ha...

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1380 @Leemoonsoo I've misunderstood your comment. 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

[GitHub] zeppelin issue #1379: [ZEPPELIN-1116]send out more exception msg

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1379 @passionke Sorry for the late reply. Thanks for the comment. 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

[GitHub] zeppelin issue #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] [ZEPPELIN-1338] ...

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1390 @echarles I've looked around your PR. I think your PR makes Zeppelin support separate users within a same instance. I have two questions regarding current behaviour. The first is about

[GitHub] zeppelin issue #1399: [ZEPPELIN-1182] Zeppelin should have Startup and Shutd...

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1399 Done or 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

Re: How to start Zeppelin By IDE

2016-09-06 Thread Jongyoul Lee
Hello, I'm using IntelliJ and there's my step below. 1. mvn clean pakcage -DskipTest 2. Debug ZeppelinServer Regards, Jongyoul On Fri, Sep 2, 2016 at 6:02 AM, moon soo Lee wrote: > Hi, > > Eclipse, i used to create eclipse project file by running "mvn > eclipse:eclipse". >

[GitHub] zeppelin issue #1406: [ZEPPELIN-1414] Remove CONTRIBUTING.md files

2016-09-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1406 Good! I like it! --- 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,

[jira] [Created] (ZEPPELIN-1416) Update Flink with latest version 1.1.1

2016-09-06 Thread Trevor Grant (JIRA)
Trevor Grant created ZEPPELIN-1416: -- Summary: Update Flink with latest version 1.1.1 Key: ZEPPELIN-1416 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1416 Project: Zeppelin Issue

[GitHub] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1384 looks fine to me, I'd defer to others on licensing. thanks for the great contribution! --- 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 #1409: [ZEPPELIN-1461] Update Flink with latest versio...

2016-09-06 Thread rawkintrevo
GitHub user rawkintrevo opened a pull request: https://github.com/apache/zeppelin/pull/1409 [ZEPPELIN-1461] Update Flink with latest version 1.1.2 ### What is this PR for? Flink has had two releases since 1.0.3, we are now on 1.1.2 This includes new functionality for

[GitHub] zeppelin issue #1409: [ZEPPELIN-1461] Update Flink with latest version 1.1.2

2016-09-06 Thread rawkintrevo
Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/1409 [Profile 2](https://travis-ci.org/apache/zeppelin/jobs/157948630) [Profile 3](https://travis-ci.org/apache/zeppelin/jobs/157948631) [Profile

[GitHub] zeppelin pull request #1402: [MINOR] Remove duplicated dependency declaratio...

2016-09-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1402 --- 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 #1399: [ZEPPELIN-1182] Zeppelin should have Startup and Shutd...

2016-09-06 Thread rajarajan-g
Github user rajarajan-g commented on the issue: https://github.com/apache/zeppelin/pull/1399 I am logging the configuration details of Zeppelin . I am not sure if we have any more configuration details to be logged. Please check PR and let me know if any more configuration needs to

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-06 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1369 I also checked the interpreter setting page like @astroshim did. (edit interpreter -> add `one:two:1.1.1` to the dependency table -> save ) Two things are different with master

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-06 Thread doanduyhai
Github user doanduyhai commented on the issue: https://github.com/apache/zeppelin/pull/1369 Let me investigate the root cause, it should be some Javascript subtle bug --- 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 #1406: [ZEPPELIN-1414] Remove CONTRIBUTING.md files

2016-09-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1406 Not sure if it's important, the thing with having a file called Contributing.md is that github actually automatically notify the person creating a PR: