Remove some inner classes from MasterService (#84448)

Today the `MasterService` has inner classes `TaskInputs`, `TaskOutputs`
and `BatchExecutionResult` that collect up various data needed for the
cluster state update computation. In fact they're not necessary: we
already have everything we need at the top level except the new
`ClusterState`, which we can return directly.

Author: DaveCTurner