zjffdu commented on pull request #3773: URL: https://github.com/apache/zeppelin/pull/3773#issuecomment-630276162
@alexott Overall I agree with using `spark.master` instead of `master`. But I think we should keep compatibility because lots of users are still using `master` today. And we should update this function to take `spark.master` precedence over `master` (https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/launcher/SparkInterpreterLauncher.java#L326) What do you think ? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
