[ZEPPELIN-4041] SparkVersion.java refactoring

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.8.2, 0.9.0
• Assignee: Leejeyeon
• Reporter: Leejeyeon
• Created: 07/Mar/19 16:14
• Updated: 29/Sep/19 08:41
• Resolved: 11/Mar/19 01:40

Description

I found some code which could be refactored after https://issues.apache.org/jira/projects/ZEPPELIN/issues/ZEPPELIN-4005 merged for checking versions.

[See code|

https://github.com/apache/zeppelin/blob/master/spark/interpreter/src/main/java/org/apache/zeppelin/spark/SparkVersion.java#L110

]

Method could use semantic majorVersion, minorVersion and patchVersion.

I think It could be more readable.

I hope it would be helpful.

Issue Links

• links to: GitHub Pull Request #3323