[YARN-4190] missing container information in FairScheduler preemption log.

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Later
• Affects Version/s: 2.7.1
• Component/s: fairscheduler
• Assignee: Zhihai Xu
• Reporter: Zhihai Xu
• Created: 19/Sep/15 00:43
• Updated: 19/Sep/15 01:02
• Resolved: 19/Sep/15 01:02

Description

Add container information in FairScheduler preemption log to help debug. Currently the following log doesn't have container information

LOG.info("Preempting container (prio=" + container.getContainer().getPriority() +
        "res=" + container.getContainer().getResource() +
        ") from queue " + queue.getName());

So it will be very difficult to debug preemption related issue for FairScheduler.
Even the container information is printed in the following code

        LOG.info("Killing container" + container +
            " (after waiting for premption for " +
            (getClock().getTime() - time) + "ms)");

But we can't match these two logs based on the container ID.
It will be very useful to add container information in the first log.

Issue Links

• depends upon: YARN-4134 FairScheduler preemption stops at queue level that all child queues are not over their fairshare