leonkozlowski commented on issue #1298:
URL: https://github.com/apache/arrow-go/issues/1298#issuecomment-5714584318
I tested a fix that adds an unexported `borrowed` flag to the datum structs
e.g. `NewDatumWithoutOwning` sets it and `Release()` becomes a noop when its
set so the datum can't drop a reference it never held. Curious if this
direction is acceptable, this would mean any calls with unkeyed literal stop
compiling `&compute.ArrayDatum{d}`. It just makes the diff alot bigger. Happy
to take this on if direction sounds right
--
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]
