[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: Zeppelin
  Issue Type: Improvement
Reporter: CORNEAU Damien
Assignee: CORNEAU Damien
 Fix For: 0.6.2


Following https://github.com/apache/zeppelin/pull/1356, we move the 
contribution documentation to the website.

We now can remove the contributing files from the repository, and add proper 
links to that documentation



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[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 appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 cannot use any plugins.

After working on PR #1356 and introduction a plugin, using the build line 
with `--safe` specified in the README is failing.

However in our case, we are not forced to use that flag, since we are 
sending the compiled website directly to the Apache SVN, and I was able to 
confirm that fact when I sent the updated website to the SVN and it was 
rendered correctly

### What type of PR is it?
Documentation

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/corneadoug/incubator-zeppelin 
update/buildCommandLine

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/1405.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1405


commit acd0b04117cedae54939d71119650d0cdb480698
Author: Damien CORNEAU 
Date:   2016-09-06T05:23:47Z

Remove --safe from building instruction




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
  Issue Type: Bug
  Components: Core, zeppelin-interpreter
Affects Versions: 0.6.1
 Environment: docker container, ubuntu 14.04, spark 2.0, python 2.7
Reporter: robinlin


I got problem when I try to use sqlContext to create a data frame in Zeppelin 
notebook, but works fine in pysprak console.
{code:java}
l = [('alice', 1)]
sqlContext.createDataFrame(l).collect()
{code}
The Error message
{code:java}
Traceback (most recent call last):
  File "/tmp/zeppelin_pyspark-8877291249558119831.py", line 266, in 
raise Exception(traceback.format_exc())
Exception: Traceback (most recent call last):
  File "/tmp/zeppelin_pyspark-8877291249558119831.py", line 264, in 
exec(code)
  File "", line 2, in 
  File "/opt/spark/python/pyspark/sql/context.py", line 299, in createDataFrame
return self.sparkSession.createDataFrame(data, schema, samplingRatio)
  File "/opt/spark/python/lib/py4j-0.10.1-src.zip/py4j/java_gateway.py", line 
933, in __call__
answer, self.gateway_client, self.target_id, self.name)
  File "/opt/spark/python/pyspark/sql/utils.py", line 63, in deco
return f(*a, **kw)
  File "/opt/spark/python/lib/py4j-0.10.1-src.zip/py4j/protocol.py", line 316, 
in get_return_value
format(target_id, ".", name, value))
Py4JError: An error occurred while calling o49.createDataFrame. Trace:
py4j.Py4JException: Method createDataFrame([class java.util.ArrayList, null, 
null]) does not exist
at py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:318)
at py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:326)
at py4j.Gateway.invoke(Gateway.java:272)
at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:128)
at py4j.commands.CallCommand.execute(CallCommand.java:79)
at py4j.GatewayConnection.run(GatewayConnection.java:211)
at java.lang.Thread.run(Thread.java:745)
{code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[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/zeppelin/blob/master/zeppelin-web/src/app/notebook/paragraph/paragraph.css#L505)

I checked and that class is used only in a few cases (Credential, 
Configuration, Interpreter), and removing that space is making them look better)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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.6
```
def __init__(self, sparkContext, sqlContext=None):
```
Spark 2.0
```
def __init__(self, sparkContext, sparkSession=None, jsqlContext=None):
```
So we need to create SQLContext using named parameters, otherwise it would 
take intp.getSQLContext() as sparkSession which cause the issue.

### What type of PR is it?
[Bug Fix]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-1411

### How should this be tested?
Tested using the example code in ZEPPELIN-1411. 

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18260139/9bd702c0-741d-11e6-8b23-946c38a794c3.png)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1411

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/1404.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1404


commit ae12ea03737d943ee20ac6398346852046c260d4
Author: Jeff Zhang 
Date:   2016-09-06T02:26:05Z

ZEPPELIN-1411. UDF with pyspark not working - object has no attribute 
'parseDataType'




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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.com/assets/10525473/18260108/c340eb02-7425-11e6-829d-9734316dfb45.png)




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 check 
that do not need compilation or specific settings separated, we would keep the 
CI feedback loop shorter and it would also be easier to know what failed 
without going to the log.

It could probably also be applied to things like Style Check or Unit Tests 
later



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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: Zeppelin
  Issue Type: Bug
  Components: python-interpreter
Affects Versions: 0.7.0
 Environment: ubuntu 16.04 + python 2.7.12
Reporter: Peng Cheng


Python interpreter can only catch exception according to last line output, this 
will cause the following test case to fail:

```
  @Test
  public void multiRowErrorFails() {
//given
PythonInterpreter realPython = new PythonInterpreter(
PythonInterpreterTest.getPythonTestProperties());
realPython.open();

//when
String QQQ = "\"\"\"\n";
String code = "raise Exception (\n" +
QQQ +
"wrong\n" +
"syntax\n" +
QQQ +
")";
InterpreterResult ret = realPython.interpret(code, null);

//then
assertNotNull("Interpreter returned 'null'", ret);
//System.out.println("\nInterpreter response: \n" + ret.message());
assertEquals(InterpreterResult.Code.ERROR, ret.code());
assertTrue(ret.message().length() > 0);
  }
```

Please advice how to fix/bypass this error. (using stderr stream?)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[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 build, and disable RAT checks on all 
other build profiles.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 to me? 


![w](https://cloud.githubusercontent.com/assets/3348133/18251495/63dc30ec-73c4-11e6-9a84-8a15a29ffb63.gif)



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 be to intercalate the user in the URL: 
http://localhost/#/userName/noteId

Regarding multiple owners, I would simply drop this feature (just like a 
file, a note belongs to a unique user).

Apart minor navigation glitches, it works pretty well here.

I continue to follow #1265 and try to keep this PR orthogonal to what is 
discussed/implemented in there.

I will add a note regarding ZEPPELIN-1144 Zeppelin home page should only 
list notebooks with read or write permission #1330 - If listing all notes with 
read access is mandatory, I would have to fallback to the single 
`notebook-authorization.json`, or making the NotebookAuthorization service 
independent from the Notebook object.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 minutes, this potentially 
indicates a stalled build or something wrong with the build itself.

The build has been terminated
```

and in the Selenium test

```
INFO] 

[INFO] Reactor Summary:
[INFO] 
[INFO] Zeppelin: Interpreter .. SUCCESS [ 
15.898 s]
[INFO] Zeppelin: Zengine .. SUCCESS [  
5.505 s]
[INFO] Zeppelin: Display system apis .. FAILURE [  
2.439 s]
[INFO] Zeppelin: Spark dependencies ... SKIPPED
[INFO] Zeppelin: Spark  SKIPPED
[INFO] Zeppelin: Server ... SKIPPED
[INFO] 

[INFO] BUILD FAILURE
[INFO] 

[INFO] Total time: 24.675 s
[INFO] Finished at: 2016-09-02T23:15:27+00:00
[INFO] Final Memory: 37M/419M
[INFO] 

[ERROR] Failed to execute goal 
org.scalatest:scalatest-maven-plugin:1.0:test (test) on project 
zeppelin-display_2.10: There are test failures -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, 
please read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the 
command
[ERROR]   mvn  -rf :zeppelin-display_2.10
```


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
  Issue Type: Bug
  Components: build
Affects Versions: 0.6.1
 Environment: amazon linux
Reporter: suresh
 Fix For: 0.6.1


mvn install -DskipTests


[INFO] --- cobertura-maven-plugin:2.7:instrument (cobertura) @ 
zeppelin-interpreter ---
[INFO] Cobertura 2.1.1 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file

[ERROR] SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in 
[jar:file:/home/suresh/.m2/repository/ch/qos/logback/logback-classic/1.0.13/logback-classic-1.0.13.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in 
[jar:file:/home/suresh/.m2/repository/org/slf4j/slf4j-log4j12/1.7.10/slf4j-log4j12-1.7.10.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type 
[ch.qos.logback.classic.util.ContextSelectorStaticBinder]




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[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 gap 
with "Properties" and the below property table. I mean 

https://cloud.githubusercontent.com/assets/10060731/18241497/6ecb441c-738c-11e6-90ae-ab05626716e5.png;
 width="360px">



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-1374]

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18240526/00996468-737f-11e6-91db-276c73bf70cf.png)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin HotFix_ZEPPELIN-1374

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/1403.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1403


commit 032bafaf84008d620129f3f48f75102645791c1d
Author: Jeff Zhang 
Date:   2016-09-05T07:44:31Z

HotFix for ZEPPELIN-1374




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 -Pbuild-distr -Pexamples -DskipTests
```

## 2. Create a new notebook and run 
```
%spark

import java.util.Date
new Date();
```

## 3. I can see the Helium icon and the clock without this patch
![screen shot 2016-09-05 at 4 47 15 
pm](https://cloud.githubusercontent.com/assets/10060731/18240728/73c19e20-7388-11e6-9a0c-3f57fd6530b2.png)

Did I miss something?





---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1371: [ZEPPELIN-1372]Automatically Detect the data ty...

2016-09-05 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/1371


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 Tested again, merging 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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
Maybe a spinner just like the save button after editing an interpreter 
would be enough then.
Its just a matter of having a small feedback.
https://cloud.githubusercontent.com/assets/710411/18238936/7f06a16a-737c-11e6-88f4-4b65434488f0.png;>





---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2016-09-05 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1265
  
This is because small changes are easy to review and intuitive, thus I 
always try to make my PR as small as I can.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2016-09-05 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1265
  
Thus my idea to handle issues about multi-tenancy is dealt with separately 
and then will combine them together after finishing them. How about 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, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2016-09-05 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1384
  
@1ambda Usually to include external source css, we would use the 
`assets/styles` folder.

However in this case, you can use bower to have it part of the build 
instead of the source code, by using `bower install --save github-markdown-css`



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---