You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "yevgenypats (via GitHub)" <gi...@apache.org> on 2023/03/30 11:26:29 UTC
[GitHub] [arrow] yevgenypats opened a new issue, #34795: [Go]: Potential race condition around retain/release
yevgenypats opened a new issue, #34795:
URL: https://github.com/apache/arrow/issues/34795
### Describe the bug, including details regarding any error messages, version, and platform.
Maybe Im missing something around how memory management is handles in the arrow library but I think there is a potential race condition even though it is said that it is safe to call `Release/Retain` from multiple go-routines.
For example:
in [NewRecord](https://github.com/apache/arrow/blob/main/go/arrow/array/record.go#L135) we call `Retain` but what guarantee we have that this object wasn't released already by `retain`. There is no check in `Release` and I would assume that for go that won't be needed as GC is taking care of that but here we are trying to implement kinda "smarter" GC but even if an object was released after the call to `New` which is valid this can in theory introduce races.
### Component(s)
Go
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] yevgenypats closed issue #34795: [Go]: Potential race condition around retain/release
Posted by "yevgenypats (via GitHub)" <gi...@apache.org>.
yevgenypats closed issue #34795: [Go]: Potential race condition around retain/release
URL: https://github.com/apache/arrow/issues/34795
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] yevgenypats commented on issue #34795: [Go]: Potential race condition around retain/release
Posted by "yevgenypats (via GitHub)" <gi...@apache.org>.
yevgenypats commented on issue #34795:
URL: https://github.com/apache/arrow/issues/34795#issuecomment-1490256862
Sorry. Seems like a false alarm. Closing.
--
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