fix: release the Coordinator segment response in sys.segments

[kgyrtkirk]

• fixes to places where the closeable was not closed due to various reasons
  • these close calls will lead to the coordinator client to shut down - and release the buffer
• uses ResourceHolder instead CloseableIterator

note: recommend to enable ignore whitespace while viewing the changes

needs #202521 to be present for a test which is failing right now to pass

[FrankChen021]

🟢 Approval recommended

No actionable issues found. The response holder closes both the lazy JSON iterator and its underlying input stream, including when closed before the first read. Metadata polling closes the holder on successful and failed iteration, uncached scans carry ownership through datasource filtering, and sys.segments propagates early close through its enumerable. The bound interpreter is released on completion, early exit, and enumeration failure.

Reviewed 12 of 12 changed files, including production changes, regression tests, benchmark and embedded-client adaptations, plus relevant callers, sequence cleanup, and Calcite 1.42.0 iterator/interpreter implementations.

Validation: git diff --check 3e93651 aff397e passed. Static review only; tests and builds were not run.

---

This is an automated review by Codex GPT-5.6-Luna(max)