Unmodifiable interpreter properties

2018-03-02 Thread Jhon Anderson Cardenas Diaz
Hi fellow Zeppelin users.

I would like to know if is there a way in zeppelin to set interpreter
properties
that can not be changed by the user from the graphic interface.

An example use case in which this can be useful is if we want that zeppelin
users can not kill jobs from the spark ui; for this you must modify the
property "spark.ui.killEnabled" with value FALSE. The problem is that the
user can change this property from the interpreter screen and enable the
feature again.

I am wondering if maybe is there some attribute in the properties
registered in interpreter-setting.json that makes that property readonly
(for end user) or something like that.

Thank you in advance!.


[jira] [Created] (ZEPPELIN-3282) pyspark as a Default Interpreter

2018-03-02 Thread Ruslan Dautkhanov (JIRA)
Ruslan Dautkhanov created ZEPPELIN-3282:
---

 Summary: pyspark as a Default Interpreter
 Key: ZEPPELIN-3282
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3282
 Project: Zeppelin
  Issue Type: New Feature
  Components: front-end, zeppelin-server
Affects Versions: 0.8.0
Reporter: Ruslan Dautkhanov
 Attachments: image-2018-03-02-13-26-39-913.png

I can't choose PySpark as a default Interpreter when creating a new note.. is 
this a new bug or am I missing some configuration?

 

!image-2018-03-02-13-26-39-913.png!



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] zeppelin pull request #2828: [ZEPPELIN-3283] Remove redundant method Interpr...

2018-03-02 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request:

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

[ZEPPELIN-3283] Remove redundant method 
InterpreterOutput.getInterpreterResultMessages()

### What is this PR for?
Remove Redundant method `InterpreterOutput.getInterpreterResultMessages()`


### What type of PR is it?
Refactoring

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

### 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/Leemoonsoo/zeppelin ZEPPELIN-3283

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

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


commit 82bda9c02b3d470d8bff4d86468717fc1564
Author: Lee moon soo 
Date:   2018-03-02T23:40:28Z

Remove redundant method getInterpreterResultMessages()

commit 12771a1454638e5c4aae428764d595d0c6c1c895
Author: Lee moon soo 
Date:   2018-03-02T23:47:35Z

remove unused import




---


[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors

2018-03-02 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2827
  
Thanks @prabhjyotsingh , I have already started one hotfix PR for building 
issue of branch-0.8, there's more issues except code style, do you mind to 
close this PR and I will fix it in one PR.


https://github.com/zjffdu/zeppelin/commit/0ef79d1807599d2fbc74e8eeb3e18d0997aa7b85


---


[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...

2018-03-02 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request:

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

[ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python 
interpreter

### What is this PR for?

```
%python
import time
print("Hello")
time.sleep(0.5) # in case of Ipython kernel, print may not immediately 
flushed and cleared.
z.getInterpreterContext().out().clear()
print("world")
```

Expected to print `world` only.
This worked in zeppelin-0.7.x and it'll be nice keep this feature in the 
future version.

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

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

### How should this be tested?
* First time? Setup Travis CI as described on 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed 
behavior
* Outline any manual 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?


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

$ git pull https://github.com/Leemoonsoo/zeppelin ZEPPELIN-3284

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

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


commit 224a2df8e64d203a5270f1948e013c14f78b52cb
Author: Lee moon soo 
Date:   2018-03-03T00:50:22Z

make z.getInterpreterContext().out().clear() work in PythonInterpreter

commit e0dcda61fa55d794b8de2f74a61454b46506f5ed
Author: Lee moon soo 
Date:   2018-03-03T01:42:41Z

make z.getInterpreterContext().out().clear() work in  IPythonInterprete




---


[GitHub] zeppelin pull request #2825: [ZEPPELIN-1967]: New pull request

2018-03-02 Thread sanjaydasgupta
Github user sanjaydasgupta closed the pull request at:

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


---


[GitHub] zeppelin issue #2825: [ZEPPELIN-1967]: New pull request

2018-03-02 Thread sanjaydasgupta
Github user sanjaydasgupta commented on the issue:

https://github.com/apache/zeppelin/pull/2825
  
Created from wrong branch in source repo. Need to correct branch and 
reissue.


---


[GitHub] zeppelin pull request #2830: [HOTFIX] Fix branch-0.8 build

2018-03-02 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

[HOTFIX] Fix branch-0.8 build

### What is this PR for?
This is a hotfix PR for branch-0.8 build


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

### Todos
* [ ] - Task

### What is the Jira issue?
* No jira created

### How should this be tested?
* CI pass

### 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 Hotfix_0.8

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

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


commit 59c2af4577d9bde3b503a251de1836f71c71e026
Author: Jeff Zhang 
Date:   2018-02-27T13:23:36Z

[HOTFIX] Fix branch-0.8 build




---


[GitHub] zeppelin pull request #2831: ZEPPELIN-3281. Apply getRelativePath when it is...

2018-03-02 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

ZEPPELIN-3281. Apply getRelativePath when it is LocalConfigStorage

### What is this PR for?
When it is LocalConfigStorage, we should use getRelativePath to get the 
config path.


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

### Todos
* [ ] - Task

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

### How should this be tested?
* Unit test added

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

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

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


commit 34385776b7946e2615b1e4d9feaa9efa338df820
Author: Jeff Zhang 
Date:   2018-03-02T10:57:10Z

ZEPPELIN-3281. Apply getRelativePath when it is LocalConfigStorage




---


[GitHub] zeppelin pull request #2831: ZEPPELIN-3281. Apply getRelativePath when it is...

2018-03-02 Thread asfgit
Github user asfgit closed the pull request at:

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


---


[jira] [Created] (ZEPPELIN-3286) Run All Paragraphs stops if there is a disabled paragraph mid-run

2018-03-02 Thread Ruslan Dautkhanov (JIRA)
Ruslan Dautkhanov created ZEPPELIN-3286:
---

 Summary: Run All Paragraphs stops if there is a disabled paragraph 
mid-run
 Key: ZEPPELIN-3286
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3286
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.7.3, 0.8.0, 0.9.0
Reporter: Ruslan Dautkhanov


Run All Paragraphs runs fine until it reaches a disabled paragraph in the 
middle of a run.

How it should be processed: disabled paragraphs should be just skipped.

What happens: disabled paragraph doesn't run (as expected), but it doesn't run 
non-disabled paragraphs after that as well (this is the bug). 

Also it seems make Zeppelin not able to run any paragraphs at all (even 
manually).. it's in some stuck state.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (ZEPPELIN-3283) InterpreterOutput.toInterpreterResultMessage() and InterpreterOutput.getInterpreterResultMessages() are redundant

2018-03-02 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-3283:
--

 Summary: InterpreterOutput.toInterpreterResultMessage() and 
InterpreterOutput.getInterpreterResultMessages() are redundant
 Key: ZEPPELIN-3283
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3283
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Lee moon soo


org.apache.zeppelin.interpreter.InterpreterOutput.toInterpreterResultMessage() 
and 
org.apache.zeppelin.interpreter.InterpreterOutput.getInterpreterResultMessages()
 are redundant



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (ZEPPELIN-3284) z.getInterpreterContext().out().clear() broken in Python interpreter

2018-03-02 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-3284:
--

 Summary: z.getInterpreterContext().out().clear() broken in Python 
interpreter
 Key: ZEPPELIN-3284
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3284
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.8.0
Reporter: Lee moon soo
 Attachments: image-2018-03-02-15-54-38-942.png

Zeppelin 0.7.x, %spark and %python interpreter able to clear output using 
z.getInterpreterContext().out().clear()

!image-2018-03-02-15-54-38-942.png!

 

However, master branch and branch-0.8, PythonInterpreter does not provide the 
same capability.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (ZEPPELIN-3285) Unnecessary margin before the title of note

2018-03-02 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-3285:
--

 Summary: Unnecessary margin before the title of note
 Key: ZEPPELIN-3285
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3285
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.8.0
Reporter: Lee moon soo
 Attachments: image-2018-03-02-15-59-39-827.png

There's unnecessary margin before the title.

!image-2018-03-02-15-59-39-827.png!

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] zeppelin issue #2817: [ZEPPELIN-3264] Notebook Snapshot feature.

2018-03-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2817
  
Interesting feature, thanks @SleepyThread for the contribution.


---


[GitHub] zeppelin issue #2763: [ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe interpre...

2018-03-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2763
  
Thanks @masyan for great contribution. Looks good to me.
I'll merge this to master and branch-0.8 if no further comments.


---


[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors

2018-03-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2827
  
LGTM


---


[GitHub] zeppelin pull request #2827: [HOTFIX] fix checkstyle errors

2018-03-02 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at:

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


---


[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors

2018-03-02 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2827
  
@zjffdu sure, no problem.


---


[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...

2018-03-02 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request:

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

[ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python 
interpreter

### What is this PR for?

```
%python
import time
print("Hello")
time.sleep(0.5) # in case of Ipython kernel, print may not immediately 
flushed and cleared.
z.getInterpreterContext().out().clear()
print("world")
```

Expected to print `world` only.
This worked in zeppelin-0.7.x and it'll be nice keep this feature in the 
future version.

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

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

### How should this be tested?
* First time? Setup Travis CI as described on 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed 
behavior
* Outline any manual 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?


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

$ git pull https://github.com/Leemoonsoo/zeppelin ZEPPELIN-3284

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

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


commit 224a2df8e64d203a5270f1948e013c14f78b52cb
Author: Lee moon soo 
Date:   2018-03-03T00:50:22Z

make z.getInterpreterContext().out().clear() work in PythonInterpreter

commit e0dcda61fa55d794b8de2f74a61454b46506f5ed
Author: Lee moon soo 
Date:   2018-03-03T01:42:41Z

make z.getInterpreterContext().out().clear() work in  IPythonInterprete




---


[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...

2018-03-02 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at:

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


---


[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...

2018-03-02 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at:

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


---


[GitHub] zeppelin pull request #2829: [ZEPPELIN-3284] z.getInterpreterContext().out()...

2018-03-02 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request:

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

[ZEPPELIN-3284] z.getInterpreterContext().out().clear() broken in Python 
interpreter

### What is this PR for?

```
%python
import time
print("Hello")
time.sleep(0.5) # in case of Ipython kernel, print may not immediately 
flushed and cleared.
z.getInterpreterContext().out().clear()
print("world")
```

Expected to print `world` only.
This worked in zeppelin-0.7.x and it'll be nice keep this feature in the 
future version.

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

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

### How should this be tested?
* First time? Setup Travis CI as described on 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed 
behavior
* Outline any manual 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?


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

$ git pull https://github.com/Leemoonsoo/zeppelin ZEPPELIN-3284

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

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


commit 224a2df8e64d203a5270f1948e013c14f78b52cb
Author: Lee moon soo 
Date:   2018-03-03T00:50:22Z

make z.getInterpreterContext().out().clear() work in PythonInterpreter

commit e0dcda61fa55d794b8de2f74a61454b46506f5ed
Author: Lee moon soo 
Date:   2018-03-03T01:42:41Z

make z.getInterpreterContext().out().clear() work in  IPythonInterprete




---


[GitHub] zeppelin pull request #2826: [ZEPPELIN-3276] Upgrade Firefox version in Trav...

2018-03-02 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at:

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


---


[jira] [Created] (ZEPPELIN-3281) Only apply getRelativePath when it is LocalConfigStorage

2018-03-02 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-3281:


 Summary: Only apply getRelativePath when it is LocalConfigStorage
 Key: ZEPPELIN-3281
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3281
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Jeff Zhang






--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] zeppelin pull request #2826: [ZEPPELIN-3276] Upgrade Firefox version in Trav...

2018-03-02 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[ZEPPELIN-3276] Upgrade Firefox version in Travis CI

### What is this PR for?
Upgrade Firefox version in Travis CI

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

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

### How should this be tested?
* CI should be green.

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


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

$ git pull https://github.com/prabhjyotsingh/zeppelin ZEPPELIN-3276

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

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


commit ffb03e10ad1d90dad9057bb008e4b930fda4bbce
Author: Prabhjyot Singh 
Date:   2018-03-01T21:15:51Z

upgrade FF to 58




---


[GitHub] zeppelin issue #2816: [ZEPPELIN-3263] Do not use Firefox >=55 for integratio...

2018-03-02 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2816
  
I understand Selenium, Gecko, WebExtensions are constantly evolving. 
However, with this (https://github.com/apache/zeppelin/pull/2826/files), I'm 
able to make all the Selenium test case pass on FF-58. This PR is still work in 
progress. 

Also, there is stuff that will get fixed with FF-59.


---


[GitHub] zeppelin issue #2827: [HOTFIX] fix checkstyle errors

2018-03-02 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2827
  
@r-kamath @zjffdu  Please help review this.


---


[GitHub] zeppelin pull request #2827: [HOTFIX] fix checkstyle errors

2018-03-02 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[HOTFIX] fix checkstyle errors

### What is this PR for?
There are some checkstyle errors in branch-0.8, this is to fix all those 
errors.

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


### How should this be tested?
* CI should be green.


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


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

$ git pull https://github.com/prabhjyotsingh/zeppelin 
hotfix/branch-0.8_checkstyle_errors

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

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


commit 1c94e5cd9d81ae18861b375056db74f5dfe36597
Author: Prabhjyot Singh 
Date:   2018-03-02T15:22:20Z

fix checkstyle errors

Change-Id: I0b24f9287f4a8d63d18d55863505370b70c3659f




---