[GitHub] zeppelin issue #1421: [ZEPPELIN-1422][zeppelin-interpreter] fix InterpreterP...

2016-09-09 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1421
  
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.
---


[jira] [Created] (ZEPPELIN-1423) Allow users to specify pre/post-execute callbacks for interpreters

2016-09-09 Thread Alex Goodman (JIRA)
Alex Goodman created ZEPPELIN-1423:
--

 Summary: Allow users to specify pre/post-execute callbacks for 
interpreters
 Key: ZEPPELIN-1423
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1423
 Project: Zeppelin
  Issue Type: New Feature
  Components: Interpreters
Affects Versions: 0.6.1
Reporter: Alex Goodman
Assignee: Alex Goodman
 Fix For: 0.7.0


Currently there is no easy way to specify 

With ZEPPELIN-1344, one of the requirements for interactive inline matplotlib 
plotting involves displaying images without explicitly calling a show() 
function. Eg,

{code}
import matplotlib.pyplot as plt
plt.plot([1, 2, 3])
{code}

should display an inline image even without calling {{plt.show()}} if mpl's 
interactive flag is enabled.

Currently this could be done in the pyspark interpreter by binding the figure's 
byte-string representation to the AngularDisplay system (since that can easily 
be updated through the matplotlib backend) but closing the figures (ie freeing 
up memory) would have to manually be done by the user. Thus we would need a 
post-execute callback that displays / cleans up all the active figure objects.

I will attempt to implement this using a new InterpreterCallback interface 
which includes support for two event types, pre and post-execute (relative to 
the paragraph). This would ideally lay down the foundation to make it possible 
for users to specify their callback code to the interpreter. 



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


[GitHub] zeppelin issue #1416: Rewrite of "Quick Start" page.

2016-09-09 Thread rfilmyer
Github user rfilmyer commented on the issue:

https://github.com/apache/zeppelin/pull/1416
  
Thanks for the feedback! If I want to make more contributions, should I do 
it in this PR, or should I make a new one.


---
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 #830: [MINOR] Set log4j.configuration into JAVA_OPTS and JAVA...

2016-09-09 Thread aarcro
Github user aarcro commented on the issue:

https://github.com/apache/zeppelin/pull/830
  
I had to remove this to get my %sql interpreter running again (on EMR). 
From what I can tell on the yarn resource manager, something owned by the 
hadoop user was trying to write to zeppelin's log file. I assume zeppelin's 
log4j config, shouldn't get used by the spark proc behind it, right?


---
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-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1404
  
@zjffdu 

Once you build zeppelin,
```
mvn package [your profiles] -DskipTests
```

Then you can run this test, like
```
mvn package -pl 'zeppelin-interpreter,zeppelin-zengine,zeppelin-server' 
-Dtest=ZeppelinSparkClusterTest -DfailIfNoTests=false -Drat.skip=true
```

Let me know if it does not work.


---
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 #1419: ZEPPELIN-1420. java.util.ConcurrentModificationExcepti...

2016-09-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1419
  
LGTM and merge if there're 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 #1421: [ZEPPELIN-1422][zeppelin-interpreter] fix InterpreterP...

2016-09-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1421
  
LGTM and merge if there're 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 #1359: [ZEPPELIN-966] job manager change information collecti...

2016-09-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1359
  
@cloverhearts Could you trigger CI again and see if test passes? especially 
first 3 test profiles didn't pass past 3 CI run in this branch.


---
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 #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-09 Thread mfelgamal
Github user mfelgamal commented on the issue:

https://github.com/apache/zeppelin/pull/1334
  
@bzz 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 pull request #1422: Allow matplotlib plot display in pyspark contex...

2016-09-09 Thread hafizur-rahman
GitHub user hafizur-rahman opened a pull request:

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

Allow matplotlib plot display in pyspark context

### What is this PR for?
Similar to %python context, allow matplotlib plot display in %pyspark 
context

### What type of PR is it?
Improvement

### How should this be tested?
The following code should display a graph:

```
%pyspark

import numpy as np
import matplotlib

matplotlib.use('Agg')

import matplotlib.pyplot as plt

x = np.linspace(0, 1)
y = np.sin(4 * np.pi * x) * np.exp(-5 * x)

plt.figure()

plt.fill(x, y, 'r')
plt.grid(True)

z.show(plt, fmt="svg")
plt.close()
```

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

$ git pull https://github.com/hafizur-rahman/zeppelin patch-1

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

https://github.com/apache/zeppelin/pull/1422.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 #1422


commit 36ea45b277aadc3e06b11c3713af5a19e9e7b32c
Author: Hafizur Rahman 
Date:   2016-09-09T16:51:26Z

Allow matplotlib plot display in pyspark context




---
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 #1333: [ZEPPELIN-1334] Environment variable defined in interp...

2016-09-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1333
  
Merge it to master and branch-0.6 if there're 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 #1409: [ZEPPELIN-1461] Update Flink with latest version 1.1.2

2016-09-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1409
  
Merge to master and branch-0.6, if there're no more dicussions


---
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 #928: [ZEPPELIN-116][wip] Add Apache Mahout Interpreter

2016-09-09 Thread rawkintrevo
Github user rawkintrevo commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
@bzz, thanks! That is exactly the idea- and I can do the same thing for 
Flink.

The cool thing there is, the exact same code* will run on %mahoutSpark and 
%mahoutFlink, so that will bring the 'mahout interpreter' to users even if they 
are running Spark 2.0, before Mahout gets bumped up to run on Spark 2.





---
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-09 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue:

https://github.com/apache/zeppelin/pull/1371
  
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on 
project zeppelin-interpreter: Compilation failure
[ERROR] 
/Users/yizhang/zeppelin/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java:[35,43]
 package org.apache.zeppelin.interpreter.dev does not exist
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute 
goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile 
(default-compile) on project zeppelin-interpreter: Compilation failure

/Users/yizhang/zeppelin/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java:[35,43]
 package org.apache.zeppelin.interpreter.dev does not exist

at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:212)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:116)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:80)
at 
org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:51)
at 
org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:128)
at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:307)
at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:193)
at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:106)
at org.apache.maven.cli.MavenCli.execute(MavenCli.java:863)
at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:288)
at org.apache.maven.cli.MavenCli.main(MavenCli.java:199)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)
Caused by: org.apache.maven.plugin.compiler.CompilationFailureException: 
Compilation failure

/Users/yizhang/zeppelin/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java:[35,43]
 package org.apache.zeppelin.interpreter.dev does not exist

at 
org.apache.maven.plugin.compiler.AbstractCompilerMojo.execute(AbstractCompilerMojo.java:858)
at 
org.apache.maven.plugin.compiler.CompilerMojo.execute(CompilerMojo.java:129)
at 
org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:134)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:207)
... 20 more

Not sure whether this is some problem with my own settings?


---
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 #928: [ZEPPELIN-116][wip] Add Apache Mahout Interpreter

2016-09-09 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
@rawkintrevo thank you for sharing the ideas and keeping it up! 
This sounds like very reasonable approach to me, at least until mahout is 
Spark2 compliant.

So the idea is to provide an automation inside Zeppelin that would add a 
pre-configure interpreter, for Spark 1.5-1.6 with Mahout dependencies, update 
the docs on how to use it and provide an example notebook?


---
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 #1421: [ZEPPELIN-1422][zeppelin-interpreter] fix Inter...

2016-09-09 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request:

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

[ZEPPELIN-1422][zeppelin-interpreter] fix InterpreterProperty equals 
methods, add null check

### What is this PR for?
in PR https://github.com/apache/zeppelin/pull/1382,
there is a small bug that `InterpreterProperty.equals` do not check `null`.
jdk lib request the `null` check otherwise it has the risk causing java 
collections operation throw exception.
so I fix it.

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

### Todos
N/A

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

### How should this be tested?
existing test.

### Screenshots (if appropriate)

### 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/WeichenXu123/zeppelin 
fix_InterpreterProperty.equals

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

https://github.com/apache/zeppelin/pull/1421.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 #1421


commit eb8d4ee87c5f681caf8a1c9b4c03c90b71ff851f
Author: WeichenXu 
Date:   2016-09-08T04:46:27Z

fix_InterpreterProperty.equals




---
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-1422) fix InterpreterProperty equals methods, add null check

2016-09-09 Thread Weichen Xu (JIRA)
Weichen Xu created ZEPPELIN-1422:


 Summary: fix InterpreterProperty equals methods, add null check
 Key: ZEPPELIN-1422
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1422
 Project: Zeppelin
  Issue Type: Bug
  Components: zeppelin-interpreter
Affects Versions: 0.7.0
Reporter: Weichen Xu


there is a small bug that `InterpreterProperty.equals` do not check `null`.
jdk lib request the `null` check otherwise it has the risk causing java 
collections operation throw exception.



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


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

2016-09-09 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue:

https://github.com/apache/zeppelin/pull/1371
  
@corneadoug fetched branch-0.6 but compile failed.


---
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 #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1408
  
@khalidhuseynov thanks for sharing the inputs on this.Basically I like your 
idea of propagating the specific error to the user instead of the generic error 
message so the user would be more specific about the issue.If we need to handle 
other errors we can think about them in the future if there is need.

Here is my thought to the approach.I will change 
org.apache.zeppelin.notebook.repo.VFSNotebookRepo.save(Note, 
AuthenticationInfo) to throw the specific exception with the error message and 
handle the exception in 
org.apache.zeppelin.socket.NotebookServer.runParagraph(NotebookSocket, 
HashSet, Notebook, Message) and then broadcast the error message to the 
user.

Thanks,
Kavin
MailTo: kavin.ku...@imaginea.com


---
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 #1381: [ZEPPELIN-1390] SparkInterpreter does not work for Spa...

2016-09-09 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1381
  
Thanks everyone!


---
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 #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1420
  
@AhyoungRyu thanks for the information but we can use the path 
https://github.com/apache/zeppelin/blob/master/docs/assets/themes/zeppelin/img/docs-img/shell-example.png
 to access the image as used in other interpreters.There is no need to use 
BASE_PATH variable.What do you say?

Thanks,
Kavin
MailTo: kavin.ku...@imaginea.com


---
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 #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1420
  
@kavinkumarks 
The shell interpreter docs in github is just a source not a real docs page. 
The real one is deployed at 
[https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/shell.html](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/shell.html).
 

The below tag is for the broken image.
```

```
Since it contains `{{BASE_PATH}}`, github can't recognize this variable. 
That's the reason why the image is broken. Zeppelin is using 
[Jekyll](https://jekyllrb.com/) to build the documentation site. (See 
[here](https://zeppelin.apache.org/contribution/documentation.html) for more 
details). Only Jekyll can recognize the variable. 
Just FYI, that kind of variables(e.g. `BASE_PATH`, `ASSET_PATH`) are 
defined in 
[docs/_config.yml](https://github.com/apache/zeppelin/blob/master/docs/_config.yml)
 :)


---
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 #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1420
  
LGTM with the dead link in README.md

Also, shouldn't we need to fix the broken image in shell interpreter doc?


---
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 #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1420
  
@zjffdu Ah it's wired. In `0.7.0-SNAPSHOT`, all the link in the dropdown 
menu are broken. It should be 
[https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/install/install.html](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/install/install.html)
 not 
[https://zeppelin.apache.org/install/install.html](https://zeppelin.apache.org/install/install.html).
 


---
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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-09 Thread astroshim
GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

### What is this PR for?
This PR is for refactoring code for JDBCInterpreter.
There is no putting 'Connection' to 'propertyKeyUnusedConnectionListMap' 
anywhere in the original code.


### What type of PR is it?
Improvement 


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


### 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/astroshim/zeppelin ZEPPELIN-1405

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

https://github.com/apache/zeppelin/pull/1396.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 #1396


commit 12dd7cb239cd22cb85d55221a18bbf157665a45a
Author: astroshim 
Date:   2016-09-02T09:05:50Z

remove propertyKeyUnusedConnectionListMap map

commit 4dabbcc98d5f8e8013502ba786ec7f60f115efff
Author: astroshim 
Date:   2016-09-04T18:08:36Z

wip) changing to use dbcp

commit 9f1e368fe16ef889715b8a58e9592cb0393ae464
Author: astroshim 
Date:   2016-09-05T15:58:25Z

replace ConnectionPool

commit 4d4f85cf1e59aaecbabe025b6c261349c94abe4a
Author: astroshim 
Date:   2016-09-05T16:10:41Z

refactoring code of close()

commit 21217a7340438bbd39b75a1dd3756885c4d2d911
Author: astroshim 
Date:   2016-09-07T02:06:50Z

fix indentation.




---
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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

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

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


---
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: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-09 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1396
  
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.
---


[GitHub] zeppelin issue #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1420
  
LGTM. BTW, I found the docs of 0.7.0-SNAPSHOT is broken, is it a known 
issue ?
e.g. 
https://zeppelin.apache.org/install/install.html


---
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 #1420: [ZEPPELIN-1421] Fix dead link in docs/README.md

2016-09-09 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request:

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

[ZEPPELIN-1421] Fix dead link in docs/README.md

### What is this PR for?
There is a dead link in 
[docs/README.md](https://github.com/apache/zeppelin/blob/master/docs/README.md).
 

It should be `https://zeppelin.apache.org/docs/latest/` not 
`https://zeppelin.apache.org/docs/latest`

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

### What is the Jira issue?
[ZEPPELIN-1421](https://issues.apache.org/jira/browse/ZEPPELIN-1421)

### How should this be tested?
 - Before 
[https://zeppelin.apache.org/docs/latest](https://zeppelin.apache.org/docs/latest)
 - After 
[https://zeppelin.apache.org/docs/latest/](https://zeppelin.apache.org/docs/latest/)

### Screenshots (if appropriate)

### 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/AhyoungRyu/zeppelin ZEPPELIN-1421

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

https://github.com/apache/zeppelin/pull/1420.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 #1420


commit 311e2eff0831f1174cf99e99dbfb05f7f6d86e1a
Author: AhyoungRyu 
Date:   2016-09-09T08:10:38Z

Fix dead link in docs/README.md




---
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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-09 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1393
  
@kavinkumarks i believe some of the committers or pmcs will take a look 
into this pr before merging


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


Re: [GitHub] zeppelin pull request #1419: ZEPPELIN-1420. java.util.ConcurrentModification...

2016-09-09 Thread Andrea Santurbano
Mm

Il ven 9 set 2016 08:45 zjffdu  ha scritto:

> GitHub user zjffdu opened a pull request:
>
> https://github.com/apache/zeppelin/pull/1419
>
> ZEPPELIN-1420. java.util.ConcurrentModificationException caused by
> calling remove inside foreach loop
>
> ### What is this PR for?
> We should use Iterator to iterate the list when we want to remove
> items in the middle of iteration, Otherwise will get the
> ConcurrentModificationException
>
>
> ### What type of PR is it?
> [Bug Fix]
>
> ### Todos
> * [ ] - Task
>
> ### What is the Jira issue?
> * https://issues.apache.org/jira/browse/ZEPPELIN-1420
>
> ### How should this be tested?
> No test added
>
> ### 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-1420
>
> Alternatively you can review and apply these changes as the patch at:
>
> https://github.com/apache/zeppelin/pull/1419.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 #1419
>
> 
> commit 5431de5b78e2fe82a578cc21b1777f7fa5ee9ef8
> Author: Jeff Zhang 
> Date:   2016-09-09T06:41:26Z
>
> ZEPPELIN-1420. java.util.ConcurrentModificationException caused by
> calling remove inside foreach loop
>
> 
>
>
> ---
> 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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-09 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1393
  
Thanks @khalidhuseynov ! Can we get this merged?

-Kavin
MailTo: kavin.ku...@imaginea.com


---
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-09 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
@corneadoug I just re-based 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 issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

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

https://github.com/apache/zeppelin/pull/1302
  
Tested LGTM
@astroshim can you rebase? It 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 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.
---


Re: DOCs-Zeppelin

2016-09-09 Thread Rajarajan Ganesan
Hey you can look into Zeppelin website for docs
https://zeppelin.apache.org/docs/0.6.1/

You can also check below pdf

http://events.linuxfoundation.org/sites/events/files/slides/Zeppelin_ApacheCon2015_0.pdf


Regards,
*Rajarajan*
*Imaginea*

On Thu, Sep 8, 2016 at 1:58 PM, Abhisar Mohapatra <
abhisar.mohapa...@inmobi.com> wrote:

> Hi ,
> Is there ant doc on design and implementation of Zeppelin. ? If it is can
> any body please share ?
>
> --
> _
> The information contained in this communication is intended solely for the
> use of the individual or entity to whom it is addressed and others
> authorized to receive it. It may contain confidential or legally privileged
> information. If you are not the intended recipient you are hereby notified
> that any disclosure, copying, distribution or taking any action in reliance
> on the contents of this information is strictly prohibited and may be
> unlawful. If you have received this communication in error, please notify
> us immediately by responding to this email and then delete it from your
> system. The firm is neither liable for the proper and complete transmission
> of the information contained in this communication nor for any delay in its
> receipt.
>


[jira] [Created] (ZEPPELIN-1421) Broken link/image in zeppelin docs

2016-09-09 Thread Kavin Kumar (JIRA)
Kavin Kumar created ZEPPELIN-1421:
-

 Summary: Broken link/image in zeppelin docs
 Key: ZEPPELIN-1421
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1421
 Project: Zeppelin
  Issue Type: Bug
  Components: documentation
Affects Versions: 0.6.1
Reporter: Kavin Kumar
Priority: Minor


The link https://zeppelin.apache.org/docs/latest in 
https://github.com/apache/zeppelin/tree/master/docs leads to 404 error and the 
example image in 
https://github.com/apache/zeppelin/blob/master/docs/interpreter/shell.md is 
broken.



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