alexandre-normand opened a new pull request, #728: URL: https://github.com/apache/arrow-go/pull/728
### Rationale for this change Currently, the pqarrow FileWriter ignores the `closed` status of the FileWriter and write calls are attempted which can lead to surprising and hard to understand errors as noted in issue #727 . Returning a clear error stating the `FileWriter` is already closed should help nudge users about their misuse of the API (or provide an indicator of a potential race condition between invocations of Write calls and Close). . ### What changes are included in this PR? Adds a check on all `FileWriter` write methods to validate is a `FileWriter` is already closed to return a clear error and short-circuit the write execution. ### Are these changes tested? Yes, a unit tested was added to validate the behavior. ### Are there any user-facing changes? resolves #727 -- 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]
