{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/446","labels_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/446/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/446/comments","events_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/446/events","html_url":"https://github.com/apache/incubator-zeppelin/pull/446","id":117614685,"number":446,"title":"ZEPPELIN-441 Make SparkInterpreter work even if Spark version is not listed","user":{"login":"Leemoonsoo","id":1540981,"avatar_url":"https://avatars.githubusercontent.com/u/1540981?v=3","gravatar_id":"","url":"https://api.github.com/users/Leemoonsoo","html_url":"https://github.com/Leemoonsoo","followers_url":"https://api.github.com/users/Leemoonsoo/followers","following_url":"https://api.github.com/users/Leemoonsoo/following{/other_user}","gists_url":"https://api.github.com/users/Leemoonsoo/gists{/gist_id}","starred_url":"https://api.github.com/users/Leemoonsoo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Leemoonsoo/subscriptions","organizations_url":"https://api.github.com/users/Leemoonsoo/orgs","repos_url":"https://api.github.com/users/Leemoonsoo/repos","events_url":"https://api.github.com/users/Leemoonsoo/events{/privacy}","received_events_url":"https://api.github.com/users/Leemoonsoo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-11-18T15:46:00Z","updated_at":"2015-11-19T00:20:32Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-zeppelin/pulls/446","html_url":"https://github.com/apache/incubator-zeppelin/pull/446","diff_url":"https://github.com/apache/incubator-zeppelin/pull/446.diff","patch_url":"https://github.com/apache/incubator-zeppelin/pull/446.patch"},"body":"https://issues.apache.org/jira/browse/ZEPPELIN-441\r\n\r\nCurrently when spark version is not listed in SparkVersion.java, it throws \r\n\r\n```\r\njava.lang.IllegalArgumentException\r\n    at org.apache.zeppelin.spark.SparkVersion.fromVersionString(SparkVersion.java:58)\r\n    at org.apache.zeppelin.spark.SparkInterpreter.open(SparkInterpreter.java:477)\r\n```\r\n\r\nThis PR changes SparkVersion from 'enum' to 'class' and make Spark Interpreter work with unlisted version."},"comment":{"url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/comments/157906874","html_url":"https://github.com/apache/incubator-zeppelin/pull/446#issuecomment-157906874","issue_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/446","id":157906874,"user":{"login":"Leemoonsoo","id":1540981,"avatar_url":"https://avatars.githubusercontent.com/u/1540981?v=3","gravatar_id":"","url":"https://api.github.com/users/Leemoonsoo","html_url":"https://github.com/Leemoonsoo","followers_url":"https://api.github.com/users/Leemoonsoo/followers","following_url":"https://api.github.com/users/Leemoonsoo/following{/other_user}","gists_url":"https://api.github.com/users/Leemoonsoo/gists{/gist_id}","starred_url":"https://api.github.com/users/Leemoonsoo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Leemoonsoo/subscriptions","organizations_url":"https://api.github.com/users/Leemoonsoo/orgs","repos_url":"https://api.github.com/users/Leemoonsoo/repos","events_url":"https://api.github.com/users/Leemoonsoo/events{/privacy}","received_events_url":"https://api.github.com/users/Leemoonsoo/received_events","type":"User","site_admin":false},"created_at":"2015-11-19T00:20:32Z","updated_at":"2015-11-19T00:20:32Z","body":"That make sense.\r\n\r\nWhich do you prefer? When Spark is future version, we can either print some warning log or raise exception to not working with it."}}