[ZEPPELIN-915] New registration mechanism applied to JDBCInterpreter

[ralphgee]

What is this PR for?

This PR applies the new interpreter registration mechanism to the JDBCInterpreter.

What type of PR is it?

Improvement

Todos

• [x ] - Move interpreter registration properties from static block to interpreter-setting.json

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-915

How should this be tested?

1. apply patch
2. rm -r interpreter/jdbc
3. rm conf/interpreter.json
4. mvn clean package -DskipTests -pl jdbc
5. bin/zeppelin-daemon.sh start
6. run some paragraph with simple sql queries

Screenshots (if appropriate)

Questions:

• Does the licenses files need update? No
• Is there breaking changes for older versions? No
• Does this needs documentation? No

[jongyoul]

@ralphge Thanks for the contribution. I've tested and worked well. And CI failure is a temporary issue. Could you please re-trigger CI? It's OK to close and re-open this PR.

[jongyoul]

In addition to step for test, you should remove conf/interpreter.json before you start.

[ralphgee]

I re-triggered the CI. The build was successful.

[bzz]

@ralphge thank you for stepping up and contributing this refactoring!
Looks great to me.

[jongyoul]

LGTM. Merging if there's no more discussion.

[jongyoul]

@ralphge Could you please tell me your jira id? I cannot find you on jira with your github id.

https://issues.apache.org/jira/browse/ZEPPELIN-915

[ralphgee]

It is ralphg only.