Coordinator stopped on bad control command

[Hennzau]

Fix #648 and more

[haixuanTao]

Looks good overall :)

Thanks!

[haixuanTao]

Do we really need to change the stop_dataflow return type if we are not using it?

[Hennzau]

We are using it in the control loop for dora stop but not for the dora destroy command. But its more readable if we don't have 2 different functions to do that.

let dataflow = stop_dataflow(
    &mut running_dataflows,
    dataflow_uuid,
    &mut daemon_connections,
    clock.new_timestamp(),
    grace_duration,
)
.await;

match dataflow {
    Ok(dataflow) => {
        dataflow.reply_senders.push(reply_sender);
    }
    Err(err) => {
        let _ = reply_sender.send(Err(err));
    }
}

[haixuanTao]

Ah ok make sense! Thanks!