[SPARK-3456] YarnAllocator on alpha can lose container requests to RM

True but it seems kind of weird to have it pass count but then grab a global for pending. It would also be easier to unit test taking it as paramater (if we had unit tests for this)