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
…43360)
### Rationale for this change
`ReadRowGroups` needs to support externally canceled contexts, e.g. for request-scoped contexts in servers like gRPC.
### What changes are included in this PR?
Additionnaly, `releaseColumns` needs to ignore columns with uninitialized data as it used in a `defer` statement.
### Are these changes tested?
Yes: a new test `TestArrowReaderCanceledContext` is included.
### Are there any user-facing changes?
None
* GitHub Issue: #43359
Authored-by: sebdotv <[email protected]>
Signed-off-by: Joel Lubinitsky <[email protected]>
Describe the bug, including details regarding any error messages, version, and platform.
Context
ReadTable
:ReadRowGroups
might panic when trying to read first column length because it was not initializedreleaseColumns
might also panic for the same reasonComponent(s)
Go
The text was updated successfully, but these errors were encountered: