[GitHub] zeppelin issue #1689: [ZEPPELIN-1709] Update checkstyle rule file, plugin

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1689
  
@bzz 

A test failed but it is irrelevant with this PR

```
Tests run: 4, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.156 sec 
<<< FAILURE! - in org.apache.zeppelin.dep.DependencyResolverTest
testLoad(org.apache.zeppelin.dep.DependencyResolverTest)  Time elapsed: 
1.142 sec  <<< ERROR!
org.sonatype.aether.RepositoryException: Cannot fetch dependencies for 
com.databricks:spark-csv_2.10:1.3.0
at 
org.apache.zeppelin.dep.DependencyResolver.getArtifactsWithDep(DependencyResolver.java:181)
at 
org.apache.zeppelin.dep.DependencyResolver.loadFromMvn(DependencyResolver.java:131)
at 
org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:79)
at 
org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:96)
at 
org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:88)
at 
org.apache.zeppelin.dep.DependencyResolverTest.testLoad(DependencyResolverTest.java:77)


Results :

Tests in error: 
  DependencyResolverTest.testLoad:77 » Repository Cannot fetch 
dependencies 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 #1741: [HOTFIX][ZEPPELIN-1779]Flaky test: PySparkInterpreterM...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1741
  
merging if no more comment - I'll wait to tomorrow morning.


---
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 #1748: Option in the Zeppelin GUI to clear the output in a sp...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1748
  
thanks for the contribution. since this is the notebook menu, it might not 
be very clear what that icon means or what that "button" does (for example, is 
it for erasing the notebook? deleting the notebook?) Do you think there is some 
ways or other places for this?



---
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 #1742: [ZEPPELIN-1781] Failed to run the script 'test_zeppeli...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1742
  
@cuspymd have you tested this with Python 2.x and Python 3.x? I'm not sure 
we have automated test covered for Python + Spark on both.



---
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 #1741: [HOTFIX][ZEPPELIN-1779]Flaky test: PySparkInterpreterM...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1741
  
+1 to merge it ASAP. It is a hotfix to fix the flaky test. 


---
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 #1742: [ZEPPELIN-1781] Failed to run the script 'test_zeppeli...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1742
  
So there were 2 runs

The older run https://travis-ci.org/apache/zeppelin/jobs/182613697 failed 
with Spark cluster tests
The newer run https://travis-ci.org/apache/zeppelin/builds/183250971 failed 
with cassandra tests

Given that and no code changes between, I think we could attribute the 
failures to flaky test.



---
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 #1741: [HOTFIX][ZEPPELIN-1779]Flaky test: PySparkInterpreterM...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1741
  
Is this a hotfix? CI passed so should this be merged ASAP?



---
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 #1741: [HOTFIX][ZEPPELIN-1779]Flaky test: PySparkInterpreterM...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1741
  
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 pull request #1750: [ZEPPELIN-1796] Add 'copy to clipboard' next to...

[AhyoungRyu]

GitHub user AhyoungRyu opened a pull request:

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

[ZEPPELIN-1796] Add 'copy to clipboard' next to the paragraph id

### What is this PR for?
There are lots of use cases that need to know each `Paragraph Id` in 
Zeppelin. 
e.g. [run paragraph using rest 
api](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/rest-api/rest-notebook.html#run-a-paragraph-asynchronously)
 / 
[`z.angularBind`](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/displaysystem/front-end-angular.html#basic-usage)
 / [export paragraphs using 
iframe](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/manual/publish.html) ...

So I added `Copy to clipboard` btn next the each `Paragraph Id` under 
paragraph setting menu using 
[angular-zeroclipboard](https://github.com/lisposter/angular-zeroclipboard). 
It'll be useful even if it's small feature :)

### What type of PR is it?
Improvement | Feature 

### Todos
* [ ] - Task

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

### How should this be tested?
1. `npm run build` under `zeppelin-web/`
2. `npm run start` to start dev mode and browse `http://localhost:9000`
3. click gear icon placed top of the each paragraph -> there must be https://cloud.githubusercontent.com/assets/10060731/21129033/96e56ec4-c142-11e6-9ffd-af75007e1f92.png;>
 btn next to the paragraph id 

4. click that button and paste the paragraph id anywhere to check it's 
copied well or not

### Screenshots (if appropriate)

![copy_pid](https://cloud.githubusercontent.com/assets/10060731/21128863/7ef02008-c141-11e6-8ddd-d2d89fe47a9a.gif)

### 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 add/copyToClipboard

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

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


commit a01cc4ad342f46b05da547e7f0a7a84a00973093
Author: AhyoungRyu 
Date:   2016-12-13T05:26:18Z

Add 'copy to clipboard' next to the paragraph id




---
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 #1717: [ZEPPELIN-1744] replace to movement for paragraph in j...

[cloverhearts]

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1717
  
Could you please review for 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.
---

[jira] [Created] (ZEPPELIN-1796) Add "copy to clipboard" next to the paragraph id

[Ahyoung Ryu (JIRA)]

Ahyoung Ryu created ZEPPELIN-1796:
-

 Summary: Add "copy to clipboard" next to the paragraph id 
 Key: ZEPPELIN-1796
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1796
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.6.2
Reporter: Ahyoung Ryu
Assignee: Ahyoung Ryu
Priority: Minor
 Fix For: 0.7.0


There are lots of use cases that need to know each "Paragraph Id".
It would be useful if we have "Copy to clipboard" btn next to the paragraph id. 



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

[GitHub] zeppelin pull request #1749: [ZEPPELIN-1298] Address invalid ticket warnings...

[khalidhuseynov]

GitHub user khalidhuseynov opened a pull request:

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

[ZEPPELIN-1298] Address invalid ticket warnings when server restarted

### What is this PR for?
When Zeppelin server restarted then all previously open tab tickets will be 
invalidated. It causes execessive `PING` logs and requires user to refresh the 
page. More details in comments of the issue. 

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

### Todos
* [x] - remove ping warning
* [x] - add popup

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

### How should this be tested?
1. start Zeppelin server and login
2. restart server
3. try to do something in previously open window

### Screenshots (if appropriate)
Before:

![ping_stack_before](https://cloud.githubusercontent.com/assets/1642088/21128483/b50a9400-c13e-11e6-84af-2384cdde7efc.gif)

After:

![ping_stack_after](https://cloud.githubusercontent.com/assets/1642088/21128484/bd49f76e-c13e-11e6-86a6-f6ec944d927c.gif)


### 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/khalidhuseynov/incubator-zeppelin 
fix/invalidTicket

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

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


commit 4decb3b2f47acaa65a5329f8438f14b0219c3b53
Author: Khalid Huseynov 
Date:   2016-12-13T05:06:47Z

remote ping warn, add popup




---
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 #1742: [ZEPPELIN-1781] Failed to run the script 'test_zeppeli...

[cuspymd]

Github user cuspymd commented on the issue:

https://github.com/apache/zeppelin/pull/1742
  
@felixcheung 
I opened the PR again, but there was errors elsewhere. Zeppelin test seems 
unstable.


---
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 #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

[astroshim]

GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1767] Blank paragraph should not be run.

### What is this PR for?
This PR fixes not to run blank paragraph.


### What type of PR is it?
Improvement


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


### How should this be tested?
please run blank paragraph.


### Screenshots (if appropriate)
- before
![12 -08-2016 
12-41-59](https://cloud.githubusercontent.com/assets/3348133/20996914/5d568e92-bd44-11e6-9462-aad47e4c08ca.gif)


### 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 skipBlankParagraph

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

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


commit 35f049790131c0a0f15fccff74f5733a1f36b822
Author: astroshim 
Date:   2016-12-08T03:29:59Z

skip blank to run paragraph.

commit b7c9d7237350b51790c3ee74776827103b7a887c
Author: astroshim 
Date:   2016-12-08T10:10:22Z

fix typo

commit c80fbebfcbea0cd4cbe812bcdde014b13711baf4
Author: astroshim 
Date:   2016-12-10T16:37:04Z

fix restapi issue

commit 68ca222ed68699001d33c558db02a943b0c589bf
Author: astroshim 
Date:   2016-12-11T08:54:31Z

add testcase

commit 14f9714aa55f7dd21e3c9780c20a1f5cde0a444e
Author: astroshim 
Date:   2016-12-11T13:49:00Z

Merge branch 'master' into skipBlankParagraph

commit ccd99823d460a7cf07d181f2aecfb5d45f879670
Author: astroshim 
Date:   2016-12-12T15:09:45Z

fix testcase




---
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 #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

[astroshim]

Github user astroshim closed the pull request at:

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


---
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-1795) Release 0.7.0

[Mina Lee (JIRA)]

Mina Lee created ZEPPELIN-1795:
--

 Summary: Release 0.7.0
 Key: ZEPPELIN-1795
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1795
 Project: Zeppelin
  Issue Type: Bug
Reporter: Mina Lee


This is ticket for tracking unresolved issues for 0.7.0 release.

Here is the scope of 0.7.0 release stated in [wiki 
page|https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+Roadmap].

0.7.0 - Enterprise-ready
 * Enterprise support
   ** multi user support
   ** Impersonation
 * Job management
 * Monitoring support (e.g. JMX)
 * Interpreter
   ** Improve JDBC / Python interpreter
   ** New Interpreters
 * Front end performance improvement
 * Pluggable visualization





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

[GitHub] zeppelin issue #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1538
  
@mfelgamal

I agree with you opinion. We can directly use `usr/local/zeppelin` instead 
of `start-zeppelin.sh`. 
I'v just created PR for removing `start-zeppelin.sh` related files, codes, 
docs. 

https://github.com/mfelgamal/zeppelin/pull/3/files 



---
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: Beginner issue

[Alexander Bezzubov]

Thank you Igor!

You was added to JIRA contributed group and from now on should be able to
assign issues.

Indeed, JIRA issues \w ''begginer'' label is a very good start!

--
Alex

On Tue, Dec 13, 2016, 07:02 moon  wrote:

> Done! Thanks you for taking care of the issue.
>
> Regards,
> moon
>
> On Mon, Dec 12, 2016 at 6:10 AM Igor Drozdov 
> wrote:
>
> > Hello,
> >
> > I would like to contribute to Zeppelin. Could anyone assign issue
> > https://issues.apache.org/jira/browse/ZEPPELIN-922 to me (drigor) in
> jira?
> >
> > Thank you,
> > Igor Drozdov
> >
> >
>

Re: Data Source Metadata Your Thoughts

[Alexander Bezzubov]

Datasource schema exploration is something very usefull and practical
indeed.

And thank you very much Peter for sharing PoC and a very nice video through
a new feature discussion - this really helps!

The only minor concern is - how to make user experience with such feature
consistent across the interpreters? What is the best way to fit it into the
"mental framework" of the notebook user?

As this is outside of the grammar of SQL and does not start with % as
paragraph meta-information about the interpreter, how would one discover
this feature in a way, that might be extensible enough to be supported by
other interpreters?

At the same time it's great not to over-generalise at first and make it
simple to make sure it's usefull. Something as simple as have it in the
docs for SQL interpreter might be a good start.

And then enhance it up to the point where it's general enough to become the
major part of "mental framework" on par with Note/paragraph concept, to
cover all interpreters and potentially event fit into the GUI.

What do you guys think?

--
Alex

On Tue, Dec 13, 2016, 11:10 Hyung Sung Shim  wrote:

> Hello Peter.
> Thank you for suggesting great function.
> It would be really useful function for zeppelin users!
>
>
> 2016년 12월 13일 (화) 오전 7:01, Peter McCaffrey 님이 작성:
>
> > Hey everyone,
> >  I wanted to send this out to gather some opinions before I submit
> any
> > PRs on this topic. I've been using zeppelin for about 7 months now, and
> it
> > has gained rapid adoption amongst my team.  Unfortunately, one sticking
> > point for our team is it's lack of some useful data source exploration
> > tools (if there is such functionality and I just missed it, please let me
> > know!).
> >
> > I made a simple change to the JDBC interpreter to add an "explore"
> feature
> > as shown in this video I recorded (https://s3.amazonaws.com/scre
> > enshots-mockups/embedvid.html).
> >
> > I understand that JDBC data source metadata can be gathered using SQL but
> > this works exclusively through the JDBC driver api and seems like a
> simple
> > and clean way to get tables and views while working on a query.
> >
> > I've tested this with PostgreSQL and MySQL and I just wanted to see if
> the
> > community thinks this is valuable. I see in the sourcecode something
> called
> > the "sqlCompleter" which appears to work with metadata but I'm not sure
> if
> > or how this pulls data source tables and views so please let me know if
> I'm
> > missing something big. If this feature seems useful, then once PR-1744 (
> > https://github.com/apache/zeppelin/pull/1744) is merged I'll open a PR
> for
> > this.
> >
> > Sincerely,
> > -Peter
> >
>

[GitHub] zeppelin pull request #1740: ZEPPELIN-1773. Remove method destroy() in Inter...

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1773. Remove method destroy() in Interpreter

### What is this PR for?
IMHO `destroy` is a little functional duplicated with `close`. And it seems 
currently only cassandra use destroy and it just calls `close()` 
https://github.com/apache/zeppelin/blob/master/cassandra/src/main/java/org/apache/zeppelin/cassandra/CassandraInterpreter.java#L231.
 So IMO, we could delete destroy method, also we can delete the destroy method 
in `InterpreterGroup`.  Besides, I move the terminate remoteInterpreterProcess 
in `close`.


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

### Todos
* [ ] - Task

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

### How should this be tested?
Just remove something so the existing test pass should be sufficient. 

### 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/zjffdu/zeppelin ZEPPELIN-1773

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

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


commit 4fd2e38db99c815da783eea4232a20fd9bd30178
Author: Jeff Zhang 
Date:   2016-12-09T02:12:51Z

ZEPPELIN-1773. Remove method destroy() in 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 pull request #1740: ZEPPELIN-1773. Remove method destroy() in Inter...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1745: ZEPPELIN-1785. Remove ZEPPELIN_NOTEBOOK_DIR and ZEPPEL...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1745
  
@Leemoonsoo  Please help 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.
---

[jira] [Created] (ZEPPELIN-1794) Flaky test: InterpreterIT

[Jeff Zhang (JIRA)]

Jeff Zhang created ZEPPELIN-1794:


 Summary: Flaky test: InterpreterIT
 Key: ZEPPELIN-1794
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1794
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.7.0
Reporter: Jeff Zhang


https://api.travis-ci.org/jobs/183237658/log.txt?deansi=true
{noformat}
13:12:21,028  INFO org.apache.zeppelin.integration.ZeppelinIT:189 - 
testCreateNotebook Test executed
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 217.56 sec - in 
org.apache.zeppelin.integration.ZeppelinIT
 
Results :
 
Tests in error: 
  InterpreterIT.testShowDescriptionOnInterpreterCreate:69 » ElementNotVisible 
El...
 
{noformat}



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

[GitHub] zeppelin issue #1744: Better output of JDBC Interpreter.

[bzz]

Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1744
  
Looks great to me, modulo minor issue above.

Also, in my experience if CI failure reason is summarised and posted here 
by PR author - it makes merging/reviewing process much faster.


---
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: Notebook example

[Alexander Bezzubov]

Done! Now you should be able to assign all further issues.

Thank you for contribution, the notebook looks great, and here is how it
looks like in case people want to see it "in action" [1]

Please, feel free to check other open issues in Zeppelin JIRA marked \w
'beginner' label [2] for good future contributions ideas!


 1.
https://www.zeppelinhub.com/viewer/notebooks/aHR0cHM6Ly9yYXcuZ2l0aHVidXNlcmNvbnRlbnQuY29tL0FsZXhhbmRlclNob3NoaW4vemVwcGVsaW4tZXhhbXBsZS9tYXN0ZXIvbm90ZWJvb2tzL0ZsaWdodHMuanNvbg

 2.
https://issues.apache.org/jira/browse/ZEPPELIN-1793?jql=project%20%3D%20ZEPPELIN%20AND%20status%20in%20(Open%2C%20%22In%20Progress%22)%20AND%20labels%20%3D%20beginner

--
Alex

On Mon, Dec 12, 2016 at 5:55 PM, Alexander Shoshin <
alexander_shos...@epam.com> wrote:

> I have created an issue:
> https://issues.apache.org/jira/browse/ZEPPELIN-1787
> but I can't assign it to myself. Could you assign it to me please?
>
> -Original Message-
>  From: moon soo Lee [mailto:leemoon...@gmail.com]
>  Sent: Friday, December 09, 2016 8:49 PM
>  To: dev@zeppelin.apache.org
>  Subject: Re: Notebook example
>
> > Althought it's not mendentory, creating issue helps track the changes
> such as when
> > code has chaged, which version of zeppelin includes the changes, etc.
> So, please
> > create if you don't mind.
> >
> > Thanks,
> > moon
> > On Fri, 9 Dec 2016 at 11:05 AM Alexander Shoshin <
> alexander_shos...@epam.com>
> > wrote:
> >
> > > Thanks for reply.
> > >
> > > And do I need to create an issue first? Or I can just make a pull
> > > request for adding a note to tutorial examples?
> > >
> > > Thanks,
> > > Alexander
> > >
> > > > -Original Message-
> > > > From: moon soo Lee [mailto:m...@apache.org]
> > > > Sent: Thursday, December 08, 2016 11:30 PM
> > > > To: dev@zeppelin.apache.org
> > > > Subject: Re: Notebook example
> > > >
> > > > Hi,
> > > >
> > > > Appreciate for sharing a notebook and considering contribution.
> > > > You can contribute your note as a tutorial under
> > > > https://github.com/apache/zeppelin/tree/master/notebook by making
> > > > pullrequest. So Zeppelin distribution can include notebook. Just
> > > > sharing on the mailing list greatly helps as well if you prefer.
> > > >
> > > > Don't hesitate making pullrequest or asking questions.
> > > >
> > > > Thanks,
> > > > moon
> > > >
> > > > On Thu, Dec 8, 2016 at 10:26 AM Alexander Shoshin <
> > > > alexander_shos...@epam.com> wrote:
> > > >
> > > > > Hi everyone.
> > > > >
> > > > > I want to become a Zeppelin contributor. Before starting to fix
> > > > > issues I decided to make an example of a notebook to become
> > > > > understand Zeppelin better :). I chose Apache Flink as a main
> > > > > Interpreter for my example because there are not so many examples
> of "Zeppelin on Flink"
> > > > > usage. I think it also may be helpful for your community according
> > > > > to https://issues.apache.org/jira/browse/ZEPPELIN-684, so here is
> > > > > the link:
> > > > > https://github.com/AlexanderShoshin/zeppelin-example
> > > > >
> > > > > I am ready to make any kind of corrections in this example.
> > > > >
> > > > > Regards,
> > > > > Alexander
> > > > >
>

[jira] [Created] (ZEPPELIN-1793) Import\export between Zeppelin and Jupyter notebook formats

[Alexander Bezzubov (JIRA)]

Alexander Bezzubov created ZEPPELIN-1793:


 Summary: Import\export between Zeppelin and Jupyter notebook 
formats
 Key: ZEPPELIN-1793
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1793
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Alexander Bezzubov
 Fix For: 0.7.0


As a user doing data analytics in Notebook environment, I would like not to be 
locked up with a single notebook implementation.

Right now, Apache Zeppelin has it's own notebook format {{note.json}}, and it 
is different from a Jupyther one, so user is locked up.

In order to avoid it, Zeppelin user should be able to:
 - import new Jupyter\IPython {.ipnb} notebook to Zeppelin, so it gets 
converted to a {note.json} and saved under {{./notebooks/}}
 - export existing Zeppelin notebook as Jupyter format, so it gets converted 
and downloaded as Jupyter {{.ipnb}} file

This can be further broken down to sub-tasks (in order of priority):
 - backend: implement an {{.ipnb <-> note.json}} convertor, \w UnitTests
 - backend: implement REST API for import\export notebooks in Zeppelin
 - frontend: add GUI for Import/Export to Zeppelin webapp

As soon as backend is ready, it will be easy for find people working on 
frontend, willing to contribute Import/Export GUI.



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

Re: Data Source Metadata Your Thoughts

[Hyung Sung Shim]

Hello Peter.
Thank you for suggesting great function.
It would be really useful function for zeppelin users!


2016년 12월 13일 (화) 오전 7:01, Peter McCaffrey 님이 작성:

> Hey everyone,
>  I wanted to send this out to gather some opinions before I submit any
> PRs on this topic. I've been using zeppelin for about 7 months now, and it
> has gained rapid adoption amongst my team.  Unfortunately, one sticking
> point for our team is it's lack of some useful data source exploration
> tools (if there is such functionality and I just missed it, please let me
> know!).
>
> I made a simple change to the JDBC interpreter to add an "explore" feature
> as shown in this video I recorded (https://s3.amazonaws.com/scre
> enshots-mockups/embedvid.html).
>
> I understand that JDBC data source metadata can be gathered using SQL but
> this works exclusively through the JDBC driver api and seems like a simple
> and clean way to get tables and views while working on a query.
>
> I've tested this with PostgreSQL and MySQL and I just wanted to see if the
> community thinks this is valuable. I see in the sourcecode something called
> the "sqlCompleter" which appears to work with metadata but I'm not sure if
> or how this pulls data source tables and views so please let me know if I'm
> missing something big. If this feature seems useful, then once PR-1744 (
> https://github.com/apache/zeppelin/pull/1744) is merged I'll open a PR for
> this.
>
> Sincerely,
> -Peter
>

[jira] [Created] (ZEPPELIN-1792) Cannot start Zeppelin on Windows 7

[Arthur Zubarev (JIRA)]

Arthur Zubarev created ZEPPELIN-1792:


 Summary: Cannot start Zeppelin on Windows 7
 Key: ZEPPELIN-1792
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1792
 Project: Zeppelin
  Issue Type: Bug
Reporter: Arthur Zubarev


Hi,
Followed the installation instructions for the regular distribution and at the 
stage where I need to issue command {code:none}
bin\zeppelin.cmd 
{code}
it reports on error 
 {code:none}
The system cannot find the path specified.
{code}
It is possible this is happening on the last line in the aforesaid file:
{code:sql}
"%ZEPPELIN_RUNNER%" %JAVA_OPTS% -cp %CLASSPATH% %ZEPPELIN_SERVER% "%*"
{code}
perhaps %ZEPPELIN_RUNNER% is undefined?



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

code generation: paragraph generates another paragraph

[Ruslan Dautkhanov]

We'd like to have paragraph's code generated by a preceding paragraph.

For example, one of the use cases we have
is when %pyspark generates Hive DDLs.
(can't run those in Spark in some cases)
Any chance an output of a paragraph can be redirected to a following
paragraph?

I was thinking something like this could be used
https://zeppelin.apache.org/docs/latest/rest-api/rest-notebook.html#create-a-new-paragraph
http://[zeppelin-server]:[zeppelin-port]/api/notebook/[notebookId]/paragraph
But not sure if there is a easy way to call Zeppelin API directly through
"z" variable?

Something like z.addParagraph(...)

In most cases a paragraph generates a SQL code that can't be run directly
as Spark SQL
and has to be run by a different engine, for example, by Hive.


Thanks,
Ruslan

[jira] [Created] (ZEPPELIN-1791) Bar Chart - Stacked setting isn't get persisted

[Ruslan Dautkhanov (JIRA)]

Ruslan Dautkhanov created ZEPPELIN-1791:
---

 Summary: Bar Chart - Stacked setting isn't get persisted
 Key: ZEPPELIN-1791
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1791
 Project: Zeppelin
  Issue Type: Bug
  Components: Core, front-end, Interpreters
Affects Versions: 0.7.0
 Environment: 0.7 from master/snapshot
Reporter: Ruslan Dautkhanov


Bar Chart - Stacked setting isn't get persisted.

Every time Bar Chart is getting switched to 
Grouped 
from Stacked.

We have some particular visualizations that only need Stacked Bar Charts.
It's very incovenient to switch back to Stack every time.

Please make Bar Chart - Stacked setting persisted.

See screenshot.



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

Data Source Metadata Your Thoughts

[Peter McCaffrey]

Hey everyone,
 I wanted to send this out to gather some opinions before I submit any
PRs on this topic. I've been using zeppelin for about 7 months now, and it
has gained rapid adoption amongst my team.  Unfortunately, one sticking
point for our team is it's lack of some useful data source exploration
tools (if there is such functionality and I just missed it, please let me
know!).

I made a simple change to the JDBC interpreter to add an "explore" feature
as shown in this video I recorded (https://s3.amazonaws.com/scre
enshots-mockups/embedvid.html).

I understand that JDBC data source metadata can be gathered using SQL but
this works exclusively through the JDBC driver api and seems like a simple
and clean way to get tables and views while working on a query.

I've tested this with PostgreSQL and MySQL and I just wanted to see if the
community thinks this is valuable. I see in the sourcecode something called
the "sqlCompleter" which appears to work with metadata but I'm not sure if
or how this pulls data source tables and views so please let me know if I'm
missing something big. If this feature seems useful, then once PR-1744 (
https://github.com/apache/zeppelin/pull/1744) is merged I'll open a PR for
this.

Sincerely,
-Peter

[jira] [Created] (ZEPPELIN-1790) Restart in Spark Interpreter should be able to kill a particular session instead of all sessions

[HARISYAM SUKUMARAN NAIR (JIRA)]

HARISYAM SUKUMARAN NAIR created ZEPPELIN-1790:
-

 Summary: Restart in Spark Interpreter should be able to kill a 
particular session instead of all sessions
 Key: ZEPPELIN-1790
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1790
 Project: Zeppelin
  Issue Type: Improvement
  Components: Interpreters
Affects Versions: 0.6.2
Reporter: HARISYAM SUKUMARAN NAIR
Priority: Critical


Currently in the Interpreter - Spark Interpreter the restart will kill all the 
open sessions by other users .Instead we should be able to kill particular 
session so that it wont affect other running sessions



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

[jira] [Created] (ZEPPELIN-1789) Improve data download file name from data.csv to ${paragraph_titile}.csv

[Ruslan Dautkhanov (JIRA)]

Ruslan Dautkhanov created ZEPPELIN-1789:
---

 Summary: Improve data download file name from data.csv to 
${paragraph_titile}.csv
 Key: ZEPPELIN-1789
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1789
 Project: Zeppelin
  Issue Type: Improvement
  Components: front-end, zeppelin-server
Affects Versions: 0.7.0
Reporter: Ruslan Dautkhanov


Currently data downloads are name just data.csv.

It's really confuzing when users have to download a lot of tables from 
different paragraphs.

Suggested naming 
- ${paragraph_name}.csv
- ${notebook_name}-${paragraph_name}.csv
- ${notebook_name}-${paragraph_name}-{paragraph_finishedAtTime}.csv




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

[GitHub] zeppelin pull request #1748: Option in the Zeppelin GUI to clear the output ...

[PetroKostyuk]

GitHub user PetroKostyuk opened a pull request:

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

Option in the Zeppelin GUI to clear the output in a specific note fro…

### What is this PR for?
It allowes user to clear notes input from menu. It resolves problem in 
issue 1503


### What type of PR is it?
Feature

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

### How should this be tested?
Clicking on new icon should clear output of all notes in given notebook

### Screenshots (if appropriate)
![screenshot from 2016-12-12 
21-01-45](https://cloud.githubusercontent.com/assets/11884177/21114854/e0e0bafe-c0ae-11e6-8893-2c979283ffb3.png)

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

$ git pull https://github.com/PetroKostyuk/zeppelin ZEPPELIN-1503

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

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


commit a5db929f21cf171ce2cb253f9d08a56a63a0e488
Author: Petro Kostyuk 
Date:   2016-12-12T19:55:38Z

Option in the Zeppelin GUI to clear the output in a specific note from the 
main menu




---
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 #1747: Minor changes

[bitchelov]

GitHub user bitchelov opened a pull request:

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

Minor changes

### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - 
https://zeppelin.apache.org/contribution/contributions.html


### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. 
[ZEPPELIN-533]

### How should this be tested?
Outline the steps to test the PR here.

### Screenshots (if appropriate)

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

Remove this unused import,  naming convention

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

$ git pull https://github.com/bitchelov/zeppelin 
hdfsFileInterpreterSmallChanges

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

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


commit 6d27bb8d9350ba1830d3b1c618838b8631cfed16
Author: Unknown 
Date:   2016-12-12T14:14:12Z

Minor changes

Remove this unused import,  naming convention




---
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 #1733: [ZEPPELIN-1753] Fix blank notename issue

[marchpig]

Github user marchpig closed the pull request at:

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


---
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 #1733: [ZEPPELIN-1753] Fix blank notename issue

[marchpig]

GitHub user marchpig reopened a pull request:

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

[ZEPPELIN-1753] Fix blank notename issue

### What is this PR for?
Note ID is displayed on action bar after pressing enter key with blank 
notename.
This is because the scope variable for notename is used directly as the 
model of input field and it doesn't revert the blank notename back to its 
existing name even though the notename was actually not updated to the blank 
value. (Temporarily lost notename from scope)
To resolve this issue, I added a variable to `ElasticInputCtrl` for the 
model of input field.
This change can always make the notename valid.

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

### Todos

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

### How should this be tested?
Input blank notename on action bar and..
- Check the notename on action bar
- Click 'Export this note' button and check the downloaded json filename.

### Screenshots (if appropriate)

![blank-notename](https://cloud.githubusercontent.com/assets/17305893/20969808/07b6a970-bcce-11e6-9457-e264a2bb0f92.gif)

### 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/marchpig/zeppelin ZEPPELIN-1753

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

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


commit 25c2ebf6df01b28a551ff7a0f3fc2af1de19db52
Author: Sangwoo Lee 
Date:   2016-12-07T13:34:03Z

Fix blank notename issue




---
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 #1742: [ZEPPELIN-1781] Failed to run the script 'test_...

[cuspymd]

GitHub user cuspymd reopened a pull request:

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

[ZEPPELIN-1781] Failed to run the script 'test_zeppelin_pr.py' on Windows 10

### What is this PR for?
Fix failure of script execution on Windows 10


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

### Todos

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

### How should this be tested?
Run 'test_zeppelin_pr.py' on Windows 10.

### 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/cuspymd/zeppelin test-pr

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

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


commit eb4b5670ab3c2f31d188e5ee806290d666f26bb8
Author: Myoungdo Park 
Date:   2016-12-09T14:50:20Z

Fix failure of script execution on Windows 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.
---

[GitHub] zeppelin pull request #1742: [ZEPPELIN-1781] Failed to run the script 'test_...

[cuspymd]

Github user cuspymd closed the pull request at:

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


---
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 #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl...

[cuspymd]

GitHub user cuspymd reopened a pull request:

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

[ZEPPELIN-1239] Google WebFont: goog-webfont-dl Grunt task warning and 
build error

### What is this PR for?
It is to fix a zeppelin-web build error in Windows 10.
To fix it, I changed google webfont download module of grunt.

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

### Todos


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

### How should this be tested?
zeppelin-web build in Windows 10

### Screenshots (if appropriate)

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

'spawn UNKNOWN' error occurs on zeppelin-web build processing in windows 10.
It due to the function 'grunt.util.spawn()' which is called by the module 
'grunt-goog-webfont-dl'.
So I changed this module to 'grunt-google-fonts'.

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

$ git pull https://github.com/cuspymd/zeppelin google-fonts

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

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


commit 1fea68fe08b06f7c5215dc54c5601d52d9386bbf
Author: Myoungdo Park 
Date:   2016-12-06T16:12:59Z

Change google font downloader to fix a build error in win 7

Curruntly 'spawn UNKNOWN' error occurs on zeppelin-web build processing in 
windows 7.
It due to the function 'grunt.util.spawn()' which is called by the module 
'grunt-goog-webfont-dl'.
So I changed this module to 'grunt-google-fonts'.




---
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 #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl...

[cuspymd]

Github user cuspymd closed the pull request at:

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


---
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 #1745: ZEPPELIN-1785. Remove ZEPPELIN_NOTEBOOK_DIR and...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1740: ZEPPELIN-1773. Remove method destroy() in Inter...

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1773. Remove method destroy() in Interpreter

### What is this PR for?
IMHO `destroy` is a little functional duplicated with `close`. And it seems 
currently only cassandra use destroy and it just calls `close()` 
https://github.com/apache/zeppelin/blob/master/cassandra/src/main/java/org/apache/zeppelin/cassandra/CassandraInterpreter.java#L231.
 So IMO, we could delete destroy method, also we can delete the destroy method 
in `InterpreterGroup`.  Besides, I move the terminate remoteInterpreterProcess 
in `close`.


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

### Todos
* [ ] - Task

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

### How should this be tested?
Just remove something so the existing test pass should be sufficient. 

### 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/zjffdu/zeppelin ZEPPELIN-1773

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

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


commit 4fd2e38db99c815da783eea4232a20fd9bd30178
Author: Jeff Zhang 
Date:   2016-12-09T02:12:51Z

ZEPPELIN-1773. Remove method destroy() in 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 pull request #1740: ZEPPELIN-1773. Remove method destroy() in Inter...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1746: Collection.isEmpty() should be used to test for...

[bitchelov]

GitHub user bitchelov opened a pull request:

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

Collection.isEmpty() should be used to test for emptiness

### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - 
https://zeppelin.apache.org/contribution/contributions.html


### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. 
[ZEPPELIN-533]

### How should this be tested?
Outline the steps to test the PR here.

### Screenshots (if appropriate)

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

Using Collection.size() to test for emptiness works, but using 
Collection.isEmpty() makes the code more readable and can be more performant.



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

$ git pull https://github.com/bitchelov/zeppelin severalChangesCollection

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

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


commit e2dae2fc97057c90e51ffaa199c695925266db2c
Author: sergey_sokur 
Date:   2016-12-12T11:50:29Z

Collection.isEmpty() should be used to test for emptiness




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

Beginner issue

[Igor Drozdov]

Hello,

I would like to contribute to Zeppelin. Could anyone assign issue 
https://issues.apache.org/jira/browse/ZEPPELIN-922 to me (drigor) in jira?

Thank you,
Igor Drozdov

[jira] [Created] (ZEPPELIN-1788) don't modify CLASSPATH environment variable

[Esteban Donato (JIRA)]

Esteban Donato created ZEPPELIN-1788:


 Summary: don't modify CLASSPATH environment variable
 Key: ZEPPELIN-1788
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1788
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.1
Reporter: Esteban Donato


scripts install-interpreter.sh, interpreter.sh, zeppelin-daemon.sh and 
zeppelin.sh modify the environment variable CLASSPATH appending either 
ZEPPELIN_CLASSPATH or ZEPPELIN_INTP_CLASSPATH value to it. Since 
zeppelin-deamon.sh launches zeppelin server which it then launches the 
different interpreters via interpreter.sh, if CLASSPATH is set as a exported 
variable, the CLASSPATH value modified by  zeppelin-daemon.sh (which appends 
the zeppelin server dependencies) is then propagated to the interpreter.sh 
execution which result in having the zeppelin server dependencies appended to 
the interpreters classpath. This cause not just to have unnecessary libraries 
in the interpreters classpath but also compatibility issues in the 
spark-interpreter since spark is compiled for Scala 2.10 while zeppelin server 
includes Scala 2.11 as one of its dependencies and both end up being part of 
the spark interpreter classpath.

my proposal: use CLASSPATH variable in the scripts as a read-only variable to 
append libraries to the different classpaths and just modify  
ZEPPELIN_CLASSPATH or ZEPPELIN_INTP_CLASSPATH on case by case to append just 
the necessarily libraries to the different processes.

If this proposal makes sense I can provide a PR with this change. 



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

Re: Integrating SSO to Zeepelin Shiro

[Vinay Shukla]

Hi Niraj,

AFAIK, right now Zeppelin does not support SSO integration. There is a
possiblity to integrate Zeppelin with Apache Knox to take advantage of SSO
features and integrate any SAML based SSO provider.

If you are willing to right custom code, I support SSO integration can be
done with current Zeppelin versions.

Thanks,
Vinay

On Sat, Dec 10, 2016 at 9:46 PM, Niraj Kumar  wrote:

> Hi All,
>
>
>
> I am trying to integrate SSO to Zeppelin using shiro. Identity provider is
> Siteminder and protocol is SAML.
>
> Could you please send me or point me to link where I find the procedure to
> do this.?
>
>
>
> Thanks in advance
>
> Niraj
>
> Disclaimer :
> This email communication may contain privileged and confidential
> information and is intended for the use of the addressee only. If you are
> not an intended recipient you are requested not to reproduce, copy
> disseminate or in any manner distribute this email communication as the
> same is strictly prohibited. If you have received this email in error,
> please notify the sender immediately by return e-mail and delete the
> communication sent in error. Email communications cannot be guaranteed to
> be secure & error free and Incedo Inc. is not liable for any errors in
> the email communication or for the proper, timely and complete transmission
> thereof.
>

[GitHub] zeppelin pull request #1538: [ZEPPELIN-1386] Docker images for running Apach...

[mfelgamal]

GitHub user mfelgamal reopened a pull request:

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

[ZEPPELIN-1386] Docker images for running Apache Zeppelin releases

### What is this PR for?

This PR is for making docker images for zeppelin releases. It contains a 
script for building image for each release. Another script is used for 
publishing images to zeppelin Dockerhub account.

This repo, https://github.com/mfelgamal/zeppelin-dockers, is a 
demonstration of this PR. It contains zeppelin-base image and an image for each 
zeppelin release.
### What type of PR is it?

[Feature]
### Todos
- Review Comments
- Documentation
### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1386
### How should this be tested?
- run create_release script or publish_release script.
### Screenshots (if appropriate)
### Questions:
- Does the licenses files need update? no
- Is there breaking changes for older versions? no
- Does this needs documentation? yes


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

$ git pull https://github.com/mfelgamal/zeppelin zeppelin-dockers

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

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


commit 48d0a01741e4a44ec76fa39e404a6d56a1f1aab0
Author: mfelgamal 
Date:   2016-09-07T12:28:40Z

Merge pull request #1 from apache/master

merge latest master updates

commit ac06f3a0d9d8f5563890cf370c4f22aa8d7e3eca
Author: mahmoudelgamal 
Date:   2016-09-24T12:42:57Z

Make docker image for zeppelin release

commit 231a414c3ca26bca3a6cbdb85ec0636213beeebf
Author: mahmoudelgamal 
Date:   2016-10-18T10:13:01Z

Add zeppelin-base image

commit e6423098786f52c2600745c3b60a15d3938300c0
Author: mahmoudelgamal 
Date:   2016-10-21T23:18:12Z

Add documentation and some modifications

commit e731cb4515c06b04b54c0cffa6dfc39115df9456
Author: mahmoudelgamal 
Date:   2016-10-23T21:59:33Z

Add java-cacerts to zeppelin-base

commit e1d4b7733523e7242b0d539301ac6a6c674e177b
Author: mahmoudelgamal 
Date:   2016-11-03T17:35:58Z

add R and python to zeppelin-base

commit fd2397057dfc263c1c2a8723935e09706abf1f99
Author: mahmoudelgamal 
Date:   2016-11-05T10:53:20Z

remove bash erorr message.

commit d2c744e3ce732fc1df529e6ee956331345cc61d1
Author: mahmoudelgamal 
Date:   2016-11-12T21:29:57Z

add scala to zeppelin-base

commit 1f093d47d0d26d68191cce6518e278ad45a110dd
Author: mahmoudelgamal 
Date:   2016-11-18T09:24:39Z

Add script start-zeppelin to zeppelin-base

commit b64c68008ab39ab929cda36adfe1ad78636a9d47
Author: mahmoudelgamal 
Date:   2016-12-01T15:34:23Z

Remove gcc and g++ for decreasing the size




---
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 #1538: [ZEPPELIN-1386] Docker images for running Apach...

[mfelgamal]

Github user mfelgamal closed the pull request at:

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


---
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 #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

[mfelgamal]

Github user mfelgamal commented on the issue:

https://github.com/apache/zeppelin/pull/1538
  
@bzz @1ambda are there further 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 #1722: [ZEPPELIN-1776] substitute null check for the refactor...

[khalidhuseynov]

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1722
  
ping


---
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 #1675: [ZEPPELIN-1190] Visit and switch notebook revisions

[khalidhuseynov]

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1675
  
@Leemoonsoo CI is finally green, I think it's ready to go


---
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: Notebook example

[Alexander Shoshin]

I have created an issue:
https://issues.apache.org/jira/browse/ZEPPELIN-1787
but I can't assign it to myself. Could you assign it to me please?

-Original Message-
 From: moon soo Lee [mailto:leemoon...@gmail.com] 
 Sent: Friday, December 09, 2016 8:49 PM
 To: dev@zeppelin.apache.org
 Subject: Re: Notebook example

> Althought it's not mendentory, creating issue helps track the changes such as 
> when
> code has chaged, which version of zeppelin includes the changes, etc. So, 
> please 
> create if you don't mind.
>
> Thanks,
> moon
> On Fri, 9 Dec 2016 at 11:05 AM Alexander Shoshin 
> wrote:
>
> > Thanks for reply.
> >
> > And do I need to create an issue first? Or I can just make a pull 
> > request for adding a note to tutorial examples?
> >
> > Thanks,
> > Alexander
> >
> > > -Original Message-
> > > From: moon soo Lee [mailto:m...@apache.org]
> > > Sent: Thursday, December 08, 2016 11:30 PM
> > > To: dev@zeppelin.apache.org
> > > Subject: Re: Notebook example
> > >
> > > Hi,
> > >
> > > Appreciate for sharing a notebook and considering contribution.
> > > You can contribute your note as a tutorial under
> > > https://github.com/apache/zeppelin/tree/master/notebook by making 
> > > pullrequest. So Zeppelin distribution can include notebook. Just 
> > > sharing on the mailing list greatly helps as well if you prefer.
> > >
> > > Don't hesitate making pullrequest or asking questions.
> > >
> > > Thanks,
> > > moon
> > >
> > > On Thu, Dec 8, 2016 at 10:26 AM Alexander Shoshin <
> > > alexander_shos...@epam.com> wrote:
> > >
> > > > Hi everyone.
> > > >
> > > > I want to become a Zeppelin contributor. Before starting to fix 
> > > > issues I decided to make an example of a notebook to become 
> > > > understand Zeppelin better :). I chose Apache Flink as a main 
> > > > Interpreter for my example because there are not so many examples of 
> > > > "Zeppelin on Flink"
> > > > usage. I think it also may be helpful for your community according 
> > > > to https://issues.apache.org/jira/browse/ZEPPELIN-684, so here is 
> > > > the link:
> > > > https://github.com/AlexanderShoshin/zeppelin-example
> > > >
> > > > I am ready to make any kind of corrections in this example.
> > > >
> > > > Regards,
> > > > Alexander
> > > >

[jira] [Created] (ZEPPELIN-1787) Add example of Flink Notebook

[Alexander Shoshin (JIRA)]

Alexander Shoshin created ZEPPELIN-1787:
---

 Summary: Add example of Flink Notebook
 Key: ZEPPELIN-1787
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1787
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.7.0
Reporter: Alexander Shoshin
Priority: Minor


It will be great to add an example of *Flink* Notebook to default Zeppelin 
notebooks. There are only *Spark*, *Python* and *R* examples at the moment.




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