On Wed Jul 9 12:26:06 2025 +0000, Nikolay Sivov wrote:
I think adding new flags like that should be revised after recent changes like 08b8aeed163d3a2de3443fbfb66430fe74c775e2. Can we avoid them now?
I still think this is the cleanest way.
The only alternative I can think of is to break `COMMAND_STATE_RESTARTING_SOURCES` in to two separate command states: 1. for stopping the sources. This is what `COMMAND_STATE_RESTARTING_SOURCES` currently does; and 2. for starting the sources. This is what `COMMAND_STATE_STARTING_SOURCES` currently does; with the exception that we call `session_flush_sinks` when restarting.
Let me know if you have a preference. Leave it as is? Or modify the restarting/starting sources command state?