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

    https://github.com/apache/flink/pull/5220#discussion_r160715198
  
    --- Diff: 
flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontend.java ---
    @@ -587,9 +526,7 @@ protected int cancel(String[] args) {
                        try {
                                jobId = new 
JobID(StringUtils.hexStringToByte(jobIdString));
                        } catch (Exception e) {
    -                           LOG.error("Error: The value for the Job ID is 
not a valid ID.");
    -                           System.out.println("Error: The value for the 
Job ID is not a valid ID.");
    -                           return 1;
    +                           throw new CliArgsException("The value for the 
JobID is not a valid ID: " + e.getMessage());
    --- End diff --
    
    Will change it.


---

Reply via email to