dbtsai opened a new pull request, #291: URL: https://github.com/apache/spark-connect-rust/pull/291
The container-input capture fixture left its session active after terminating its mock server. Since `getOrCreate()` reuses the active/default session regardless endpoint, later captures could reuse a session attached to the stopped server. Give the fixture a fresh session with `create()`, stop it before the server, and retain process/pipe cleanup if session shutdown raises. Validation: Python syntax and `git diff --check` pass. The two existing list/tuple values-and-error cases pass using a privately copied older native extension; this is supplementary fixture evidence, not an exact current-target build. This PR is a draft. The required genuine Claude reviews read source but could not return final verdicts because the configured gateway rejects the current source IP (HTTP 403 IP ACL). Additional Codex source review is recorded separately. Exact native build and required Rust compile/test/fmt/clippy and official parity/API gates are also pending because the local toolchain is unavailable. No review/validation convergence is claimed. Follow-up to #129. This pull request and its description were written by Isaac. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
