Remove task collector from JDC and Translator.

[bit-aloo]

In both JDC and tproxy, we utilize a task collector ([JDC](

stratum/roles/jd-client/src/lib/mod.rs

Line 79 in a306512

let task_collector = Arc::new(Mutex::new(vec![]));

) and [tproxy](

stratum/roles/translator/src/lib/mod.rs

Line 61 in a306512

let task_collector: Arc<Mutex<Vec<(AbortHandle, String)>>> =

)). The task collector is responsible for managing the child future handles associated with initial_jdc in JDC and internal_start in tproxy. This mechanism ensures that, in the event of an abort scenario, these child tasks can be terminated, allowing the parent future to spawn a new set of futures for reconnection.

Recently, we revised the reconnection logic for the parent future. Previously, it could become blocking during connection interruptions. To address this, we now spawn the parent future as a separate task, enabling the select blocks ([JDC](

stratum/roles/translator/src/lib/mod.rs

Line 91 in a306512

select! {

) and [tproxy](

stratum/roles/jd-client/src/lib/mod.rs

Line 117 in a306512

select! {

)) defined in the main blocking runtime thread to remain responsive.

With this change, the core concept behind the task collector can be simplified. We can now leverage a root future handler that, upon termination, can terminate all its child tasks in a single operation.

[bit-aloo]

The task collector was added in this PR: #1001

[bit-aloo]

cc: @Fi3 @plebhash @GitGab19 @jbesraa

[plebhash]

@Fi3 do you ACK removing task_collector?

[Fi3]

cACK I usally prefer having tasks that get aborted when dropped https://github.com/demand-open-source/demand-cli/blob/master/src/shared/utils.rs#L12