Document threading concerns in TaskSchedulerImpl

TaskSchedulerImpl has a couple of places threading concerns aren't clearly documented, which could improved a little. There is also a race in killTaskAttempt on taskIdToExecutorId (though I think nobody actually uses this api).