You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Thomas Thornton (Jira)" <ji...@apache.org> on 2020/11/19 01:45:00 UTC
[jira] [Commented] (FLINK-17827) scala-shell.sh should fail early
if no mode is specified, or have default logging settings
[ https://issues.apache.org/jira/browse/FLINK-17827?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17235100#comment-17235100 ]
Thomas Thornton commented on FLINK-17827:
-----------------------------------------
Hi, [~chesnay], I want to gain consensus on the implementation. I suggest we add an else condition [here|https://github.com/apache/flink/blob/master/flink-scala-shell/start-script/start-scala-shell.sh#L93] that will exit with an error that the mode is not set properly and should be one of local, remote, or yarn. This will maintain consistency with both the [Scala requirements|https://github.com/apache/flink/blob/master/flink-scala-shell/src/main/scala/org/apache/flink/api/scala/FlinkShell.scala#L206-L209] and maintain consistency with the [documentation|https://ci.apache.org/projects/flink/flink-docs-stable/ops/scala_shell.html#full-reference]. If this sounds reasonable to you, then feel free to assign the ticket to me.
> scala-shell.sh should fail early if no mode is specified, or have default logging settings
> ------------------------------------------------------------------------------------------
>
> Key: FLINK-17827
> URL: https://issues.apache.org/jira/browse/FLINK-17827
> Project: Flink
> Issue Type: Improvement
> Components: Scala Shell
> Affects Versions: 1.11.0
> Reporter: Chesnay Schepler
> Priority: Critical
> Labels: starter
>
> The scala-shell has multiple modes it can run in: local, remote and yarn.
> It is mandatory to specify such a mode, but this is only enforced on the scala side, not in the bash script.
> The problem is that the scala-shell script derives the log4j properties from the mode, and if no mode is set, then the log4j properties are empty.
> This leads to a warning from slf4j that no logger was defined and all that.
> Either scala-shell.sh should fail early if no mode is specified, or it should have some default logging settings (e.g., the ones for local/remote).
--
This message was sent by Atlassian Jira
(v8.3.4#803005)