[GitHub] zeppelin pull request #2856: ZEPPELIN-3291. Throw exception instead of retur...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---

[GitHub] zeppelin pull request #2856: ZEPPELIN-3291. Throw exception instead of retur...

[zjffdu]

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-3291. Throw exception instead of return null for 
InterpreterFactory.getInterpreter

### What is this PR for?
For some cases we throw runtime exception in InterpreterFactory, and for 
some other cases, just return null. It doesn't make sense to do that. This 
ticket is trying to refactor it to throw exception for all the cases to make 
code robust and consistent. 

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

### Todos
* [ ] - Task

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

### 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 ZEPPELIN-3291

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

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


commit bee319ccc069df3a76f27996d5447bf112da15ba
Author: Jeff Zhang 
Date:   2018-03-05T04:58:19Z

ZEPPELIN-3291. Throw exception instead of return null for 
InterpreterFactory.getInterpreter




---