fallintoplace opened a new pull request, #1066:
URL: https://github.com/apache/arrow-go/pull/1066

   ## What changed
   
   Remove the duplicate retain in `PreparedStatement.SetRecordReader`.
   
   ## Why
   
   The setter retained the supplied reader twice, while parameter replacement 
and statement close released it only once. This left one reference permanently 
owned by the statement.
   
   The regression test tracks the reader reference count through binding and 
statement close to verify the documented one-retain, one-release contract.
   
   ## Testing
   
   - `go test ./arrow/flight/flightsql`


-- 
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]

Reply via email to