paradedb / pg_analytics

DuckDB-powered analytics for Postgres
https://paradedb.com
PostgreSQL License
197 stars 13 forks source link

Show users a better error message for unsupported timestamp ranges #5

Closed rebasedming closed 4 months ago

rebasedming commented 7 months ago

What Datafusion only supports queries with timestamps from 1677 to 2262 (https://github.com/apache/arrow-datafusion/blob/b7f477237cc91bbdf90e655db181ca4f0a64fc25/datafusion/functions/src/datetime/common.rs#L34). When users try to query with a timestamp outside of this range i.e. SELECT * FROM t WHERE a < '1650-01-01 01:01:01'::timestamp they get a confusing error message about nanosecond overflow. We should intercept this error message and show them a more helpful one.

Why

How

Abdi-29 commented 7 months ago

Hi, I'm excited to contribute to paradeDB for the first time. Before starting, I wanted to check if anyone else is already working on this. Looking forward to your feedback and suggestions!

philippemnoel commented 7 months ago

Hi, I'm excited to contribute to paradeDB for the first time. Before starting, I wanted to check if anyone else is already working on this. Looking forward to your feedback and suggestions!

It's all yours! I just assigned you

Abdi-29 commented 5 months ago

I fixed the issue in the arrow-rs and I'm waiting them to accept my pr

philippemnoel commented 5 months ago

https://github.com/apache/arrow-rs/pull/5730

philippemnoel commented 4 months ago

Closing these as we're archiving this repository for good