lidavidm commented on issue #38837: URL: https://github.com/apache/arrow/issues/38837#issuecomment-2123732250
One nit: > Its release is NULL It probably shouldn't be NULL since either (1) caller may expect it to be non-NULL and try to call it or (2) caller may assume NULL means that the array was already released It would be safer to have a release() that just does nothing. -- 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]
