canbinzheng created FLINK-6132:
----------------------------------

             Summary: Remove redundant code in CliFrontend.java
                 Key: FLINK-6132
                 URL: https://issues.apache.org/jira/browse/FLINK-6132
             Project: Flink
          Issue Type: Improvement
          Components: Client
            Reporter: canbinzheng
            Priority: Minor


Look at the switch case block in method parseParameters of CliFrontend.java. 

                // do action
                switch (action) {
                        case ACTION_RUN:
                                return CliFrontend.this.run(params);
                        case ACTION_LIST:
                                return list(params);
                        case ACTION_INFO:
                                return info(params);
                        case ACTION_CANCEL:
                                return cancel(params);
                        case ACTION_STOP:
                                return stop(params);
                        case ACTION_SAVEPOINT:
                                return savepoint(params);
It's better to change the first case return clause 
from:
      return CliFrontend.this.run(params);
to:
      return run(params);




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to