joechenrh opened a new pull request, #485: URL: https://github.com/apache/arrow-go/pull/485
### Rationale for this change Previously, only data page v2 in the reader will use the memory allocated from allocator. And seems there exists some misuage of the allocator. ### What changes are included in this PR? Use memory allocator to allocate buffer for `serializedPageReader`, and introduce `Close` function to make sure all the memory will be freed after usage. ### Are these changes tested? ### Are there any user-facing changes? If user are using some customized memory allocator, they should remember to call `ColumnChunkReader.Close` after reading after chunk. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
