After this is merged, I would suggest [Obsolete]ing the Task<IList<T>> DeserializeAsync<T>(Stream, ParquetOptions, CancellationToken) over load at the earliest opportunity. Removing it will reduce confusion between the new method and this one, and will make it easer to optimize DeserializeRowGroupAsync.
Fixes #386 .
After this is merged, I would suggest
[Obsolete]
ing theTask<IList<T>> DeserializeAsync<T>(Stream, ParquetOptions, CancellationToken)
over load at the earliest opportunity. Removing it will reduce confusion between the new method and this one, and will make it easer to optimizeDeserializeRowGroupAsync
.