Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6140#discussion_r199724031
  
    --- Diff: 
flink-scala-shell/src/main/scala/org/apache/flink/api/scala/FlinkShell.scala ---
    @@ -255,14 +257,25 @@ object FlinkShell {
         yarnConfig.queue.foreach((queue) => args ++= Seq("-yqu", 
queue.toString))
         yarnConfig.slots.foreach((slots) => args ++= Seq("-ys", 
slots.toString))
     
    +    val customCommandLines = CliFrontend.loadCustomCommandLines(
    +      configuration,configurationDirectory)
    +    val commandOptions = CliFrontendParser.getRunCommandOptions
    +    val customCommandLineOptions = new Options()
    +    customCommandLines.asScala.foreach(cmd => {
    --- End diff --
    
    this is already done int he `CliFrontend` constructor. It may be sufficient 
to switch the initialization of `frontend` and `commandLine`.


---

Reply via email to