[GitHub] zeppelin issue #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1371 I wanted to merge this in 0.6 also, there was some conflicts, and something went wrong the cherry-pick. Now I can't push to a different branch with our merge tool --- If your project is set u

[GitHub] zeppelin issue #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1371 @Peilin-Yang Could you eventually open a PR with those changed targeting branch-0.6? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

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

2016-09-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1265 Agree on we should make PR keep small for review. Otherwise we should make subtask. --- 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 #1397: [ZEPPELIN-1406] modify to load method artifact local p...

2016-09-05 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1397 @cloverhearts I can't reproduce this bug. Here is what I did. ## 1. Build the latest `master` branch with below command and restart Zeppelin daemon ``` mvn clean package -Pb

[GitHub] zeppelin pull request #1403: HotFix for ZEPPELIN-1374

2016-09-05 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1403 HotFix for ZEPPELIN-1374 ### What is this PR for? This is for fixing bug introduced in #1365. I may introduce this when rebasing the PR. ### What type of PR is it? [Bug Fix

[GitHub] zeppelin issue #1403: HotFix for ZEPPELIN-1374

2016-09-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1403 @minahlee Please help review. Thanks --- 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 enab

[GitHub] zeppelin issue #1395: [ZEPPELIN-1404] invalid html structure for bootstrap i...

2016-09-05 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1395 Tested and it definitely looks better! It's my personal preference, can we put more space between "Option" and the dropdown menu? It would be better "Options" and the dropdown menu has same

[jira] [Created] (ZEPPELIN-1410) ERROR] SLF4J: Class path contains multiple SLF4J bindings.

2016-09-05 Thread suresh (JIRA)
suresh created ZEPPELIN-1410: Summary: ERROR] SLF4J: Class path contains multiple SLF4J bindings. Key: ZEPPELIN-1410 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1410 Project: Zeppelin Iss

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

2016-09-05 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1384 @corneadoug Thanks! I'v installed github markdown css and added it to `bower.json` using the command you provided. It is definitely better to use bower if possible :) --- If your pr

[GitHub] zeppelin issue #1387: [ZEPPELIN-1398] Use relative path for search_data.json

2016-09-05 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1387 All the issues are addressed i think. IDK the exact reason, but some CI tests are failed with the below unrelated reason. ``` No output has been received in the last 10 m

[GitHub] zeppelin issue #1397: [ZEPPELIN-1406] modify to load method artifact local p...

2016-09-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1397 @AhyoungRyu I'll check 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 enab

[GitHub] zeppelin issue #1395: [ZEPPELIN-1404] invalid html structure for bootstrap i...

2016-09-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1395 @AhyoungRyu Thank you for your feedback. --- 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 fea

[jira] [Created] (ZEPPELIN-1411) UDF with pyspark not working

2016-09-05 Thread Sojan James (JIRA)
Sojan James created ZEPPELIN-1411: - Summary: UDF with pyspark not working Key: ZEPPELIN-1411 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1411 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #1403: HotFix for ZEPPELIN-1374

2016-09-05 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1403 Good catch, tested. I am merging it as a hotfix. --- 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 #1403: HotFix for ZEPPELIN-1374

2016-09-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1403 --- 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 #1390: [WIP] [ZEPPELIN-1339] [ZEPPELIN-1000] [ZEPPELIN-1338] ...

2016-09-05 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1390 I have introduced a parameter in the URL `?runAs=...` that allows to impersonate from an authenticated user another user. The needed R/W authorization are respected. An alternative would

[GitHub] zeppelin issue #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...

2016-09-05 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1330 If the current single `notebook-authorization.json` is managed by user (with multiple files, and no more with a single one), how would you handle it? --- If your project is set up for it, you c

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

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai Could you check the infinitely downloading library problem(like following screen shot)? Strangely this problem happens all interpreter except spark. Is this problem happen only

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

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 \cc @jongyoul I changed some codes to use `Connection Pool` that is `org.apache.commons.dbcp2`, please review. --- If your project is set up for it, you can reply to this email and have your re

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

2016-09-05 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1401 @bzz this has now been updated to follow the same pattern used for maven tests, enable RAT to run by default, but support disabling it with -DskipRat. Travis CI will run RAT once, on the RAT bui

[jira] [Created] (ZEPPELIN-1412) Python interpreter cannot handle Exception that has multi-line message

2016-09-05 Thread Peng Cheng (JIRA)
Peng Cheng created ZEPPELIN-1412: Summary: Python interpreter cannot handle Exception that has multi-line message Key: ZEPPELIN-1412 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1412 Project: Z

[GitHub] zeppelin issue #1387: [ZEPPELIN-1398] Use relative path for search_data.json

2016-09-05 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1387 @lresende Can you close and reopen this 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

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

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1390 @echarles can you explain more about the `?runAs=`? I don't really see what this is used for. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] zeppelin issue #1387: [ZEPPELIN-1398] Use relative path for search_data.json

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1387 @AhyoungRyu we don't have any Selenium tests on the docs, so it should be unrelated --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] zeppelin issue #1387: [ZEPPELIN-1398] Use relative path for search_data.json

2016-09-05 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1387 @corneadoug I just wanna see the green light :) I'm merging this one if there is no more discussions. --- If your project is set up for it, you can reply to this email and have your r

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

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1401 Then how about we use -DskipRat on all building profiles, and have a profile that run only RAT without building? Build time is pretty long, so it would be nice to separate some easy che

[GitHub] zeppelin issue #1395: [ZEPPELIN-1404] invalid html structure for bootstrap i...

2016-09-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1395 fixed. https://github.com/apache/zeppelin/pull/1395/commits/cd6db05768523071462457205b0fcf20fc4f7865 ``` ``` ![2016-09-06 11 31 00](https://cloud.githubusercontent.c

[GitHub] zeppelin pull request #1404: ZEPPELIN-1411. UDF with pyspark not working - o...

2016-09-05 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1404 ZEPPELIN-1411. UDF with pyspark not working - object has no attribute 'parseDataType' ### What is this PR for? The root cause is that SQLContext's signature changes in spark 2.0. Spark 1.

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

2016-09-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1404 \cc @Leemoonsoo Please help review, thanks --- 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 featur

[GitHub] zeppelin issue #1395: [ZEPPELIN-1404] invalid html structure for bootstrap i...

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1395 @cloverhearts actually there is also the space of the dependencies section. It would be better to simply modify the `table-striped` class. [at this line](https://github.com/apache/zeppe

[jira] [Created] (ZEPPELIN-1413) Pyspark, can not create data frame by using sqlContext

2016-09-05 Thread robinlin (JIRA)
robinlin created ZEPPELIN-1413: -- Summary: Pyspark, can not create data frame by using sqlContext Key: ZEPPELIN-1413 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1413 Project: Zeppelin Iss

[GitHub] zeppelin pull request #1356: [ZEPPELIN-1346] Add contribution guidelines to ...

2016-09-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1356 --- 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-05 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1401 @corneadoug This is exactly what is implemented on 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 no

[GitHub] zeppelin issue #1395: [ZEPPELIN-1404] invalid html structure for bootstrap i...

2016-09-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1395 @corneadoug Okay, i has fixed it. 1. removed style `table style="margin-top: 0px"` 2. remove margin-top css in paragraph.css --- If your project is set up for it, you can r

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

2016-09-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1359 @Leemoonsoo please, check to this pr. Thank you. --- 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 #1359: [ZEPPELIN-966] job manager change information collecti...

2016-09-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1359 change done. --- 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 #1401: [ZEPPELIN-1409] Refactor RAT build on Travis.CI config...

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1401 @lresende My bad, when I first read I understood that RAT would be included in one profile that would still build everything. (just like we do with the -DskipTests) --- If your project is set

[GitHub] zeppelin issue #1395: [ZEPPELIN-1404] invalid html structure for bootstrap i...

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1395 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 wishes so, or if the

[GitHub] zeppelin pull request #1152: ZEPPELIN-531 ] Job management - Filter and Run/...

2016-09-05 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1152 --- 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 #1152: ZEPPELIN-531 ] Job management - Filter and Run/stop co...

2016-09-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1152 i will divide 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 feature enabled and w

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

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @corneadoug I fixed it. please review. Thank you! --- 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 #1401: [ZEPPELIN-1409] Refactor RAT build on Travis.CI config...

2016-09-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1401 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 wishes so, or if the

[GitHub] zeppelin issue #1402: [MINOR] Remove duplicated dependency declaration

2016-09-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1402 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 wishes so, or if the

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

2016-09-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1404 LGTM It'll be great to add some test for pyspark interpreter? --- 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 projec

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

2016-09-05 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1404 It guess it is because PythonInterpreter depends on python environment, so there's no test for it yet. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin pull request #1405: Remove --safe from building instruction

2016-09-05 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/zeppelin/pull/1405 Remove --safe from building instruction ### What is this PR for? Usually in order to push jekyll code to github pages, it will compile using the `--safe` One drawback is that you canno

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

2016-09-05 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1404 right. probably another PR, but I think we could use travis' addons support to install python via apt-get https://docs.travis-ci.com/user/installing-dependencies/ as we have for R. ---

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

2016-09-05 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1390 @echarles I also believe having global/single NotebookAuthorization module would be more relevant --- If your project is set up for it, you can reply to this email and have your reply appe

[jira] [Created] (ZEPPELIN-1414) Remove Contributing Markdown file from project source code

2016-09-05 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-1414: Summary: Remove Contributing Markdown file from project source code Key: ZEPPELIN-1414 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1414 Project: Zeppel

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

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 I tested `mysql`, `postgresql` and `hive`. --- 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 pull request #1406: [ZEPPELIN-1414] Remove CONTRIBUTING.md files

2016-09-05 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 pull request #1407: [ZEPPELIN-1412] add support multiline for pytho...

2016-09-05 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-05 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