[release-1.12] Revert "Scheduler: Use a "scheduler" task for thread sleep (#57544)"

[kpamnany]

This reverts commit 640c4e1.

See discussion.

Cc: @KristofferC to confirm that this PR is done correctly (to release-1.12).

Cc: @Keno to confirm whether reverting this on 1.12 alone is fine or if you want this reverted on master.

[Keno]

I'm ok with 1.12 only

[kpamnany]

From the discussion in #58689:

Do we not properly try/catch/restart that task?

I was thinking of adding exactly this as an alternative to backing this out entirely. This patch can be considered a bugfix as it did close 2 issues.

We don't, but even if we did that, that would just result in us swallowing all InterruptExceptions.

The scheduler task should only be running if there are no other runnable tasks. If we add a try/catch to the scheduler task and swallow InterruptExceptions, then that seems to me like an adequate stopgap solution until there is action on #52291.

Thoughts @Keno and @vtjnash?

[gbaraldi]

Can you open a PR that does the try catch behaviour? Just so that we can see how it behaves?

[kpamnany]

Can you open a PR that does the try catch behaviour? Just so that we can see how it behaves?

See #58765.

[Keno]

I would pretty strongly advocate for a straight revert here. The problem with underdefined behavior is that people will have written their existing code to whatever happened to work, so even small changes are likely to break people's workflows. I don't see that the motivation to let the task be GC'ed slightly earlier is sufficiently strong to force this through and we don't have enough time left to really let people try a new behavior to assess impcat.