[GitHub] zeppelin pull request #2320: [MINOR] Move remoteinterpreter into zengine

[jongyoul]

Github user jongyoul closed the pull request at:

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


---
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 #2320: [MINOR] Move remoteinterpreter into zengine

[jongyoul]

GitHub user jongyoul reopened a pull request:

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

[MINOR] Move remoteinterpreter into zengine

### What is this PR for?
RemoteInterpreter is only used in the server side then zeppelin-interpreter 
doesn't have to include this class. Moving this class helps to reduce 
interpreter binary size and change RemoteInterpreter without adding more 
dependencies if we want


### What type of PR is it?
[Refactoring]

### Todos
* [x] - Move RemoteInterpreter and related files out of 
zeppelin-interpreter module

### What is the Jira issue?
N/A

### How should this be tested?
N/A

### 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/jongyoul/zeppelin 
minor/move-remoteinterpreter-into-zengine

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

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


commit 7e8721592d13c782d9028b6c52f4360980eb7d02
Author: Jongyoul Lee 
Date:   2017-05-03T16:39:21Z

move some files of remote packages from zeppelin-interpreter to 
zeppelin-zengine

commit b7e0b9436a55a2251af637b976222b7333556296
Author: Jongyoul Lee 
Date:   2017-05-04T08:15:45Z

moved some files related remote interpreter and fix some minor things

commit 0a881c1b3869e5005a99f55eae177caf5a85ee13
Author: Jongyoul Lee 
Date:   2017-05-04T08:20:28Z

Removed unused package imported
Removed unnecessary classes imported

commit 5ac8dfbbd9eaa127137a4767cddd8e60ddd2f290
Author: Jongyoul Lee 
Date:   2017-05-05T15:38:11Z

Moved RemoteInterpreterServer to zeppelin-interpreter

commit 99c093229655883a1d3aade5fab756f06a39285e
Author: Jongyoul Lee 
Date:   2017-05-05T15:40:10Z

Made DummyInterpreter

commit e1425dfa86b4cea07362694ad80212bcb513b0dc
Author: Jongyoul Lee 
Date:   2017-05-05T16:40:40Z

Adopted DummyInterpreter

commit 80979913c73acca792b0038cd735f5060321b7a3
Author: Jongyoul Lee 
Date:   2017-05-05T17:28:31Z

Removed author tag




---
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-2509) Zeppelin Spark Interpreter fails on Windows if zeppelin.spark.useHiveContext=true

[Christian Fries (JIRA)]

Christian Fries created ZEPPELIN-2509:
-

 Summary: Zeppelin Spark Interpreter fails on Windows if 
zeppelin.spark.useHiveContext=true 
 Key: ZEPPELIN-2509
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2509
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.7.1, 0.8.0
 Environment: Windows 10, Windows 7
Reporter: Christian Fries


On Windows, if zeppelin.spark.useHiveContext=true in the Spark interpreter's 
settings, the interpreter will fail.
The error message displaced in Zeppelin is an InvocationTargetException (as 
always), which is quite unhelpful. A look at the log / console output reveals 
multiple exceptions
{code}
java.sql.SQLException: Failed to start database 'metastore_db' with class 
loader org.apache.spark.sql.hive.client.IsolatedClientLoader$$anon$1@58611960, 
see the next exception for details.
 at org.apache.derby.impl.jdbc.SQLExceptionFactory.getSQLException(Unknown 
Source)
 at 
org.apache.derby.impl.jdbc.SQLExceptionFactory40.wrapArgsForTransportAcrossDRDA(Unknown
 Source)
{code}
or
{code}
Caused by: java.sql.SQLException: Another instance of Derby may have already 
booted the database Z:\Documents\zeppelin-0.8.0-SNAPSHOT\bin\metastore_db.
{code}
etc.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin issue #2284: [ZEPPELIN-2451]: Add JDBC config option for calling co...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2284
  
Looks like this PR changes default behavior.

Previously, `connection.commit()` is executed if 
`connection.getAutoCommit()` is false.
But after this patch, `connection.commit()` will be executed if 
`connection.getAutoCommit()` is false and `zeppelin.jdbc.autocommit` property 
is true.

And `zeppelin.jdbc.autocommit` property is false by default.

Shell we keep default behavior?


---
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 #2306: [ZEPPELIN-2479] Fix incorrect tooltip view insi...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #2319: [ZEPPELIN-2437] Executions of notebooks are blocked af...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2319
  
Could you try include 4a4aea3 only?

\cc @jongyoul 



---
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 #2318: [ZEPPELIN-2492] Use single Dockerfile for each tag

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2318
  
Thanks @1ambda for taking care of it.
LGTM and merge to master if no further discussions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. 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 #2284: [ZEPPELIN-2451]: Add JDBC config option for calling co...

[randerzander]

Github user randerzander commented on the issue:

https://github.com/apache/zeppelin/pull/2284
  
Looks like recent commits conflict with this fix. I'll re-synch with 
master, but @Leemoonsoo any comments on the approach before I do that?


---
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 #2300: [ZEPPELIN-2467] Restoring second travis job functional...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2300
  
Travis build green. 
https://travis-ci.org/AlexanderShoshin/zeppelin/builds/228261473

LGTM and merge to master if no further discussions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. 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 #2310: [ZEPPELIN-2484] Fix NullPointerException in che...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #2305: [ZEPPELIN-2382] Fixed windows compatibility of interpr...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2305
  
LGTM and merge to master and branch-0.7 if no further comments


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled 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 #2320: [MINOR] Move remoteinterpreter into zengine

[jongyoul]

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/2320
  
Will merge it sooner


---
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 #2320: [MINOR] Move remoteinterpreter into zengine

[jongyoul]

Github user jongyoul closed the pull request at:

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


---
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 #2320: [MINOR] Move remoteinterpreter into zengine

[jongyoul]

GitHub user jongyoul reopened a pull request:

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

[MINOR] Move remoteinterpreter into zengine

### What is this PR for?
RemoteInterpreter is only used in the server side then zeppelin-interpreter 
doesn't have to include this class. Moving this class helps to reduce 
interpreter binary size and change RemoteInterpreter without adding more 
dependencies if we want


### What type of PR is it?
[Refactoring]

### Todos
* [x] - Move RemoteInterpreter and related files out of 
zeppelin-interpreter module

### What is the Jira issue?
N/A

### How should this be tested?
N/A

### 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/jongyoul/zeppelin 
minor/move-remoteinterpreter-into-zengine

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

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


commit 7e8721592d13c782d9028b6c52f4360980eb7d02
Author: Jongyoul Lee 
Date:   2017-05-03T16:39:21Z

move some files of remote packages from zeppelin-interpreter to 
zeppelin-zengine

commit b7e0b9436a55a2251af637b976222b7333556296
Author: Jongyoul Lee 
Date:   2017-05-04T08:15:45Z

moved some files related remote interpreter and fix some minor things

commit 0a881c1b3869e5005a99f55eae177caf5a85ee13
Author: Jongyoul Lee 
Date:   2017-05-04T08:20:28Z

Removed unused package imported
Removed unnecessary classes imported

commit 5ac8dfbbd9eaa127137a4767cddd8e60ddd2f290
Author: Jongyoul Lee 
Date:   2017-05-05T15:38:11Z

Moved RemoteInterpreterServer to zeppelin-interpreter

commit 99c093229655883a1d3aade5fab756f06a39285e
Author: Jongyoul Lee 
Date:   2017-05-05T15:40:10Z

Made DummyInterpreter




---
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 #2314: [minor] Fix JDBC doc after #2229

[prabhjyotsingh]

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2314
  
Thank you @AhyoungRyu, @felixcheung for the review.
Will merge this to master, if no more discussion.


---
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 #2313: ZEPPELIN-2268. Adding png and jpg support for helium m...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2313
  
LGTM and merge to master if no further discussions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. 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 #2125: [ZEPPELIN-2222] Add Network Visualization

[conker84]

Github user conker84 commented on the issue:

https://github.com/apache/zeppelin/pull/2125
  
It should be ok now


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled 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 #2298: [ZEPPELIN-2466] Chart resize problem

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2298
  
Was this merged into master already?


---
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 #2298: [ZEPPELIN-2466] Chart resize problem

[cloverhearts]

Github user cloverhearts closed the pull request at:

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


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