[YARN-6169] container-executor message on empty configuration file can be improved

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.0
• Component/s: container-executor
• Labels: newbie pull-request-available trivial
• Target Version/s: 3.4.0
• Hadoop Flags: Reviewed
• Assignee: Riya Khandelwal Assign to me
• Reporter: Miklos Szegedi
• Created: 10/Feb/17 00:36
• Updated: 11/Feb/24 10:54
• Resolved: 04/Oct/22 03:32

Description

If the configuration file is empty, we get the following error message:
Invalid configuration provided in /root/etc/hadoop/container-executor.cfg
This is does not provide enough details to figure out what is the issue at the first glance. We should use something like 'Empty configuration file provided...'

  if (cfg->size == 0) {
    fprintf(ERRORFILE, "Invalid configuration provided in %s\n", file_name);
    exit(INVALID_CONFIG_FILE);
  }

Issue Links

• links to: GitHub Pull Request #4952