search

JuliaIO / Parquet.jl

Julia implementation of Parquet columnar file format reader
Other
112 stars 32 forks source link

Update writer.jl usage of Tables.Columns #158

Closed quinnj closed 2 years ago

quinnj commented 2 years ago

From the change in https://github.com/JuliaData/Tables.jl/pull/255, we're considering the fact that Tables.Columns didn't call Tables.columns a bug, so it's being changed and you don't need to call Tables.columns yourself anymore when using Tables.Columns. Luckily, I'm not aware of any table inputs where this actually makes a difference (i.e. a table where Tables.columns(x) !== x), so this shouldn't be very disruptive.

quinnj commented 2 years ago

This should bump the required Tables.jl version to 1.6 once JuliaRegistries/General#45984 is merged