[GitHub] zeppelin issue #1410: Fix download links

2016-09-06 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1410 @bzz Tested: * Locally * With Production Flag * With SVN doc folder LGTM, let's try to get it merged fast since it is a Hot Fix. Documentation change for that PR is

[GitHub] zeppelin pull request #1410: Fix download links

2016-09-06 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/zeppelin/pull/1410 Fix download links ### What is this PR for? In recent changes download links were broken on the project website ### What type of PR is it? Hot Fix ### Todos * [x] - upda

[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 @felixcheung Right, this message is showing if you have a CONTRIBUTING.md file in the root of the repository. But we are also using the PR_TEMPLATE that github supports, and we have pretty

[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 O I've found the issue, it's tricky. I have added a non-existing dependency to the angular interpreter and I can also see a spinning circle forever. With Web Dev Console, I can see

[GitHub] zeppelin issue #1404: ZEPPELIN-1411. UDF with pyspark not working - object h...

2016-09-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1404 CI failed because of selenium, I think --- 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 issue #1400: [ZEPPELIN-1407] Fix Scala 2.11 build

2016-09-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1400 CI green, any more 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 your project does not have this feature enabled

[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 @felixcheung Yeah right. It makes sense. Then how about putting root `CONTRIBUTING.md` back and providing this link [https://zeppelin.apache.org/contribution/contributions.html](https://zeppeli

[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: ![image](https://cloud.githubusercon

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

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

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

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

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

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

[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 #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 4](https://travis-ci.org/apache/zeppelin/job

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

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

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

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

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". > IDEA, just importin

[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 I found you as your email. --- 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

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

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

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

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

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

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

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

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

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

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

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

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

[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 : [https://github.com/apache/zeppelin/blob/gh-pages/contribution/documentation.md#deploy-to-asf-svnpubsub-infra](https://github.com/apache/zeppelin

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

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

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

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

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

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

[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

[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