minyoung commented on a change in pull request #12641:
URL: https://github.com/apache/arrow/pull/12641#discussion_r828322776
##########
File path: go/parquet/pqarrow/path_builder.go
##########
@@ -555,7 +555,6 @@ type multipathLevelResult struct {
}
func (m *multipathLevelResult) Release() {
- m.leafArr.Release()
Review comment:
`pathBuilder.addTerminalInfo` itself does call
`paths.primitiveArr.Retain` (and a corresponding `Release`), so I think this is
safe to remove? This would also explain why the leak only happens with nested
types (because they have an extra array not released by this `leafArr.Release`)
--
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]
