pitrou commented on PR #34972: URL: https://github.com/apache/arrow/pull/34972#issuecomment-1520360143
> @pitrou Do we need an equivalent for `Event` or just use a `void*` and let other backends use whatever they would need appropriately? Probably a `int64_t`, so that the event can fit in the field regardless of the backend (some backends may use a pointer, but some may use some kind of opaque handle). However, we should probably define what the field _means_ for each backend. Otherwise we'll have a half-spec that nobody can implement rigorously. Also, I think this means the device array must have its own release callback, to properly handle the lifetime of said event. -- 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]
