[GitHub] zeppelin issue #1434: [MINOR] Hide "Currently there are no properties ~" msg...

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1434 @AhyoungRyu Test, Nice Improvements Can you just rebase to have a green CI? After that, Merging if there is no more discussions --- If your project is set up for it, you can reply to th

[GitHub] zeppelin issue #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1431 Tested LGTM @astroshim Can you just rebase? That should make the CI green --- 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 #1392: [ZEPPELIN-1437] Multi-user note management - user awar...

2016-09-20 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1392 rebased and CI is green --- 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 a

[GitHub] zeppelin issue #1442: ZEPPELIN-1459: Zeppelin JDBC URL properties mangled

2016-09-20 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1442 Thank you @randerzander, will merge this soon. --- 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

[GitHub] zeppelin issue #1437: ZEPPELIN-1454: Wrong property value on interpreter pag...

2016-09-20 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1437 Will merge this in both master and 0.6. --- 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 #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1265 @corneadoug I'm fixing some conflicts during rebasing 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

[GitHub] zeppelin issue #1444: ZEPPELIN-1456: Flaky Test: AuthenticationIT

2016-09-20 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1444 Thank you @corneadoug. --- 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 issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1384 Awesome, Merging if there is no more discussions --- 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 th

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

2016-09-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 Finally, CI is green! :) --- 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 wishe

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

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1384 Usually we prefer rebase since it doesn't pollute the commit history. But its fine, the end result is the same. We sometimes have flaky tests, I will check once the build is finished --- If y

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

2016-09-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 Open PR to re-trigger CI --- 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 wishe

[GitHub] zeppelin pull request #1384: [ZEPPELIN-1387] Support table syntax in markdow...

2016-09-20 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1384 --- 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-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 Close PR to re-trigger CI --- 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 wish

[GitHub] zeppelin pull request #1384: [ZEPPELIN-1387] Support table syntax in markdow...

2016-09-20 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1384 [ZEPPELIN-1387] Support table syntax in markdown interpreter ### What is this PR for? Support table markdown syntax issued by [ZEPPELIN-1387](https://issues.apache.org/jira/browse/ZEPPE

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

2016-09-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 Now CI is failing on the third build job. :( I checked out `zeppelin/master` and merged it on `1ambda/fix-zeppelin-1387`. Should I have use git `rebase` command instead? I thought usi

[jira] [Created] (ZEPPELIN-1465) Add an option to allow S3 server-side encryption

2016-09-20 Thread Liu Xiaohui (JIRA)
Liu Xiaohui created ZEPPELIN-1465: - Summary: Add an option to allow S3 server-side encryption Key: ZEPPELIN-1465 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1465 Project: Zeppelin Is

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

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1384 @1ambda You can rebase now :) --- 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 pull request #1444: ZEPPELIN-1456: Flaky Test: AuthenticationIT

2016-09-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1444 --- 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 #1443: [ZEPPELIN-1462] Zeppelin-Web Good Practices #3

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1443 @AhyoungRyu Since you always find a few mistakes, can you make a quick spell check like always? :) --- If your project is set up for it, you can reply to this email and have your reply appear

[jira] [Created] (ZEPPELIN-1464) Flaky Test -

2016-09-20 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-1464: Summary: Flaky Test - Key: ZEPPELIN-1464 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1464 Project: Zeppelin Issue Type: Bug Repo

[GitHub] zeppelin issue #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1441 @cloverhearts Thanks for the contribution, it looks cool. I will take a look :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1265 @jongyoul Could you try rebasing this PR? --- 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 feat

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

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1384 That CI failure is being handled in #1444, you should be able to rebase and have a green CI soon --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1444: ZEPPELIN-1456: Flaky Test: AuthenticationIT

2016-09-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1444 CI failure on this PR is irrelevant and failing on only build profile. We might want to open a JIRA issue for it, because that build profile normally have `-DskipTest` But seems to fail

[GitHub] zeppelin issue #1442: ZEPPELIN-1459: Zeppelin JDBC URL properties mangled

2016-09-20 Thread randerzander
Github user randerzander commented on the issue: https://github.com/apache/zeppelin/pull/1442 I tested the PR and it works as expected. Thanks @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 proje

[GitHub] zeppelin issue #1439: [ZEPPELIN-1423] Allow users to specify pre/post-execut...

2016-09-20 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1439 Ok, TODO's are now done. @AhyoungRyu @Leemoonsoo @bzz Please review, I myself am unsure if some of the errors I am seeing in some of the Travis CI builds are related to these changes. --- If your

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

2016-09-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1399 CI failure is irrelevant. LGTM and merge if there're no further discussions. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

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

2016-09-20 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1384 @1ambda Yeah tested locally and looks good. Thanks for your quick response! Now it's really ready to be merged I think 👍 The other PRs' CI are failing in Selenium test because of the sam

[GitHub] zeppelin issue #1392: [ZEPPELIN-1437] Multi-user note management - user awar...

2016-09-20 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1392 this is 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 enabled a

[GitHub] zeppelin pull request #1392: [ZEPPELIN-1437] Multi-user note management - us...

2016-09-20 Thread khalidhuseynov
GitHub user khalidhuseynov reopened a pull request: https://github.com/apache/zeppelin/pull/1392 [ZEPPELIN-1437] Multi-user note management - user aware reload broadcast ### What is this PR for? This PR addresses part of multi-user note management in Zeppelin. One of the tasks n

[GitHub] zeppelin pull request #1392: [ZEPPELIN-1437] Multi-user note management - us...

2016-09-20 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at: https://github.com/apache/zeppelin/pull/1392 --- 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 feat

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

2016-09-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 https://cloud.githubusercontent.com/assets/4968473/18679870/5ae08dec-7f9c-11e6-8c61-91e87bad5ed9.png";> CI is failing on only the last job. --- If your project is set up for it, you

[GitHub] zeppelin issue #1444: ZEPPELIN-1456: Flaky Test: AuthenticationIT

2016-09-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1444 Thanks @prabhjyotsingh! 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 enabl

[GitHub] zeppelin issue #1444: ZEPPELIN-1456: Flaky Test: AuthenticationIT

2016-09-20 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1444 makes sense, 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 wish

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

2016-09-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 @AhyoungRyu I'v resolved the merge conflict with 8f344db. Could you try again? :) --- 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 #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-20 Thread mfelgamal
Github user mfelgamal commented on the issue: https://github.com/apache/zeppelin/pull/1334 @bzz @felixcheung Thank you for your reviews. --- 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 f

[GitHub] zeppelin pull request #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

2016-09-20 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1441 [ ZEPPELIN-967 ] Implement search bar in job manager ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the c

[GitHub] zeppelin pull request #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

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

2016-09-20 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1384 @1ambda Sadly the conflict needs to be resolved first before merging. But only for `docs/interpreter/markdown.md` this file :) --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1444: ZEPPELIN-1456: Flaky Test: AuthenticationIT

2016-09-20 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1444 ZEPPELIN-1456: Flaky Test: AuthenticationIT ### What is this PR for? This started happening after ZEPPELIN-1144, https://github.com/apache/zeppelin/pull/1330. This test (testGroupP

[GitHub] zeppelin pull request #1443: [ZEPPELIN-1462] Zeppelin-Web Good Practices #3

2016-09-20 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/zeppelin/pull/1443 [ZEPPELIN-1462] Zeppelin-Web Good Practices #3 ### What is this PR for? This Good Practice Guide will focus on how to write a controller following the controller as vm concept, and how to

[jira] [Created] (ZEPPELIN-1463) Error on 'mvn install'

2016-09-20 Thread Kwang-in (Dennis) JUNG (JIRA)
Kwang-in (Dennis) JUNG created ZEPPELIN-1463: Summary: Error on 'mvn install' Key: ZEPPELIN-1463 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1463 Project: Zeppelin Issue T

Re: Looking for a mentor - Andrii P.

2016-09-20 Thread DuyHai Doan
Hello Andrii Welcome to the Apache Zeppelin community. We're always happy to help people getting onboard with the project. First, I would suggest to read the online documentation to get familiar with Zeppelin features: http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/ About Zeppelin Architecture,

[GitHub] zeppelin pull request #1442: ZEPPELIN-1459: Zeppelin JDBC URL properties man...

2016-09-20 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1442 ZEPPELIN-1459: Zeppelin JDBC URL properties mangled ### What is this PR for? While creating connection `DriverManager.getConnection(url, properties);` for JDBC interpreter, phoenix dri

[GitHub] zeppelin pull request #1435: ZEPPELIN-1368. interpreter-setting.json may be ...

2016-09-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1435 --- 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