[YARN-10797] Logging parameter issues in scheduler package

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.0
• Component/s: capacity scheduler
• Target Version/s: 3.4.0
• Hadoop Flags: Reviewed
• Assignee: Szilard Nemeth
• Reporter: Szilard Nemeth
• Created: 01/Jun/21 10:14
• Updated: 11/Feb/24 14:16
• Resolved: 01/Jun/21 14:06

Description

1. There is a LOG.error call in org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.QueueConfigurationAutoRefreshPolicy#editSchedule that provides logging arguments without a placeholder in the message.

  LOG.error("Failed to reload capacity scheduler config file - " +
                "will use existing conf.", e.getMessage());

2. There is a LOG.debug call in org.apache.hadoop.yarn.server.resourcemanager.scheduler.common.fica.FiCaSchedulerApp#moveReservation that has a placeholder in the logging message but the argument is an instance of Throwable so the message does not require a placeholder.

 } catch (IllegalStateException e) {
    LOG.debug("Reserve on target node failed, e={}", e);
    return false;
}