You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
### Rationale for this change
Previously the ipc reader was leaking allocations.
### What changes are included in this PR?
Call `Clear()` on the memo table on final release of the ipc reader.
### Are these changes tested?
Yes
### Are there any user-facing changes?
* Closes: #36981
Authored-by: thorfour <me@thor-hansen.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
### Rationale for this change
Previously the ipc reader was leaking allocations.
### What changes are included in this PR?
Call `Clear()` on the memo table on final release of the ipc reader.
### Are these changes tested?
Yes
### Are there any user-facing changes?
* Closes: apache#36981
Authored-by: thorfour <me@thor-hansen.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
Describe the bug, including details regarding any error messages, version, and platform.
The ipc reader may perform allocations to read dictionaries.
arrow/go/arrow/ipc/reader.go
Line 47 in 0fb744c
However these dictionary allocations are never released
arrow/go/arrow/ipc/reader.go
Lines 150 to 163 in 0fb744c
Component(s)
Go
The text was updated successfully, but these errors were encountered: