aloneguid / parquet-dotnet

Fully managed Apache Parquet implementation
https://aloneguid.github.io/parquet-dotnet/
MIT License
637 stars 153 forks source link

feat: DateTimeOffset #546

Open cliedeman opened 2 months ago

cliedeman commented 2 months ago

This is my follow up to #521

Rationale: I have many models that currently use DateTimeOffset to map entities and I would like to serialize these without creating mapping models.

I am aware that DateTimeOffset was previously removed due to confusion but I have added additional checks to try and mitigate this.

The Logical Timestamp helps with this as we can refuse to decode a Timestamp to DateTimeOffset when it is not in UTC. As this invites errors and can cause confusion. (DateTime avoids this by allowing you to stipulate the kind as Local).