[ https://issues.apache.org/jira/browse/FLINK-2613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14948530#comment-14948530 ]
ASF GitHub Bot commented on FLINK-2613: --------------------------------------- Github user chiwanpark commented on a diff in the pull request: https://github.com/apache/flink/pull/1106#discussion_r41503700 --- Diff: flink-staging/flink-scala-shell/src/main/scala/org/apache/flink/api/scala/FlinkShell.scala --- @@ -68,38 +94,62 @@ object FlinkShell { def startShell( userHost : String, - userPort : Int, + userPort : Int, + executionMode : ExecutionMode.Value, externalJars : Option[Array[String]] = None): Unit ={ --- End diff -- Please remove space after parameter name like following: ```scala def startShell( userHost: String, userPort: Int, executionMode: ExecutionMode.value ``` > Print usage information for Scala Shell > --------------------------------------- > > Key: FLINK-2613 > URL: https://issues.apache.org/jira/browse/FLINK-2613 > Project: Flink > Issue Type: Improvement > Components: Scala Shell > Affects Versions: 0.10 > Reporter: Maximilian Michels > Assignee: Nikolaas Steenbergen > Priority: Minor > Labels: starter > Fix For: 0.10 > > > The Scala Shell startup script starts a {{FlinkMiniCluster}} by default if > invoked with no arguments. > We should add a {{--help}} or {{-h}} option to make it easier for people to > find out how to configure remote execution. Alternatively, we could print a > notice on the local startup explaining how to start the shell in remote mode. -- This message was sent by Atlassian JIRA (v6.3.4#6332)