search

ZJONSSON / parquetjs

fully asynchronous, pure JavaScript implementation of the Parquet file format
MIT License
34 stars 61 forks source link

Fix BigInt timestamp_millis fail to read #91

Closed leiffoged closed 3 months ago

leiffoged commented 1 year ago

new Date(+value); => new Date(Number(value));

value is a BigInt. +value is a JS error ('Cannot convert a BigInt value to a number')

I believe the proper conversion is Number(value). Also used Number(value) instead of parseInt.