GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/2805
[ZEPPELIN-3240] Zeppelin server fail to start if interpreter has mixed
properties
### What is this PR for?
I found that Zeppelin server is failing to start when there is a mix of
both kind of properties in interpreter;
```
"properties": {
"shell.command.timeout.millisecs": {
"type": "string",
"name": "shell.command.timeout.millisecs",
"value": "60000"
},
"shell.working.directory.user.home": {
"type": "checkbox",
"name": "shell.working.directory.user.home",
"value": false
},
"zeppelin.shell.auth.type": "KERBEROS",
"zeppelin.shell.keytab.location":
"/etc/security/keytabs/zeppelin.server.kerberos.keytab",
"zeppelin.shell.principal": "zeppe...@example.com"
}
```
Mix as in these two kind of properties:
- Where key is string and value is object
```
"shell.command.timeout.millisecs": {
"type": "string",
"name": "shell.command.timeout.millisecs",
"value": "60000"
}
```
- Where key and value both are string
```
"zeppelin.shell.auth.type": "KERBEROS"
```
### What type of PR is it?
[Bug Fix]
### What is the Jira issue?
* [ZEPPELIN-3240](https://issues.apache.org/jira/browse/ZEPPELIN-3240)
### How should this be tested?
* Place this
[interpreter.json](https://issues.apache.org/jira/secure/attachment/12910862/interpreter.json)
in Z-server conf folder and try to start Z-Server, with this patch it should
start.
### Screenshots (if appropriate)
### 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-3240
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2805.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 #2805
----
commit 8545ecd040c7641fc97062acfbf7c58a5b6f1a90
Author: Prabhjyot Singh <prabhjyotsingh@...>
Date: 2018-02-16T04:32:46Z
ZEPPELIN-3240: Zeppelin server fail to start if interpreter has mixed
properties
Change-Id: I63dfb66f34e8b50bbc9d2b1e48d70c7949ebfbf9
----
---
