zeroshade opened a new pull request, #793: URL: https://github.com/apache/arrow-go/pull/793
### Rationale for this change Instead of relying on a finalizer, make the `nativeCRecordBatchReader` use atomic Retain and Release to make releasing deterministic. ### What changes are included in this PR? Remove the finalizer, call C.ArrowArrayRelease and C.ArrowArrayStreamRelease based on refcount. ### Are these changes tested? Yes, tests cover this already. ### Are there any user-facing changes? Only that retain and release now properly control the determinism of releasing the C memory instead of relying on a finalizer. -- 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]
