fix(sql) disable idle timeout when still processing data

[cirospaciari]

What does this PR do?

Fix: #16892

Disable idle timeout when processing data, fix short read handling, remove unnecessary/dead code and small refactor

• Documentation or TypeScript types (it's okay to leave the rest blank in this case)
• Code changes

How did you verify your code works?

Test

[robobun]

^{Updated 2:51 AM PT - Feb 2nd, 2025}

❌ @cirospaciari, your commit 8ab60c0 has 3 failures in Build #11014:

• test/js/bun/util/filesink.test.ts - timeout on 🪟 2019 x64
• test/js/bun/util/filesink.test.ts - timeout on 🪟 2019 x64-baseline
• test/js/web/fetch/fetch.test.ts - 1 failing on 🍎 14 aarch64
• test/cli/install/bun-install-registry.test.ts - 1 failing on 🪟 2019 x64

---

🧪   try this PR locally:

bunx bun-pr 16984

[Jarred-Sumner]

maybe instead it'd be better to reset the connection timeout at both the start and end of the onData callback?

[cirospaciari]

maybe instead it'd be better to reset the connection timeout at both the start and end of the onData callback?

I just disable the idle timeout when processing the data, if we are processing data we are active/busy not idle.