[PR] fix(pqarrow): return an error on pqarrow write calls if the writer is already closed [arrow-go]

["alexandre-normand (via GitHub)" <gi...@apache.org>]

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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] fix(parquet/pqarrow): return an error on pqarrow write calls if the writer is already closed [arrow-go]

["zeroshade (via GitHub)" <gi...@apache.org>]

zeroshade merged PR #728:
URL: https://github.com/apache/arrow-go/pull/728


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org