Closed Qqwy closed 1 year ago
Patch coverage: 100.00
% and project coverage change: +0.01
:tada:
Comparison is base (
99e30d3
) 83.58% compared to head (7662b90
) 83.60%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Before, passing an already-released stream to
ArrowArrayStreamReader::try_new()
would result in a call to.get_schema()
(and later on when the reader is used possibly to.get_next()
), which goes against the spec, as those callbacks should not be called on released streams.This PR adds an extra check to the start of
try_new()
to safeguard against already-released streams.