Simplifying scheduler

Removing unnecessary code and variables
Please move the `if` to its own line. This is not an `} else if () {`.
Looks like your logic is hardcoding the number of namenodes to 2 here, but can clean that up with the rest of the state machine logic in a follow-up patch. Or you could use `if (offers.size() > 0 && clusterState.getNameNodes().size() < TOTAL_NAME_NODES) {` to get the same behavior as before, where we launch TNN-1 secondary NNs.