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

   Column chunk cleanup retained both the current page and page reader after 
Close, so repeated cleanup could release or close the same resources twice. 
Page-reader replacement also discarded close errors and installed the 
replacement anyway.
   
   This change detaches resources before cleanup, makes Close idempotent, 
returns and retains replacement cleanup errors, and uses the checked 
replacement path from the Arrow reader before continuing to seek or read.
   
   Tests:
   - go test ./parquet/file -run 
'TestPrimitiveReaderSuite|TestColumnChunkReader'
   - go test ./parquet/... -run '^$'


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