feat: 1414: adds support for time_bucket()

[devanbenz]

Signed-off-by: Devan [email protected]# Ticket(s) Closed

• Closes #

What

Why

How

Tests

[rebasedming]

Nice! Not sure if you're done with this PR yet, but you'll need to test that time_bucket queries when pushed down to DuckDB yield the same results as those not pushed down and fall back to your time_bucket UDF.

On another note, we're moving pg_lakehouse to its own repo and renaming to pg_analytics. I would recommend opening PRs to that repo, it will make our migration process easier. https://github.com/paradedb/pg_analytics

[devanbenz]

Nice! Not sure if you're done with this PR yet, but you'll need to test that time_bucket queries when pushed down to DuckDB yield the same results as those not pushed down and fall back to your time_bucket UDF.

On another note, we're moving pg_lakehouse to its own repo and renaming to pg_analytics. I would recommend opening PRs to that repo, it will make our migration process easier. paradedb/pg_analytics

Ah yeah I'm not quite finished with this yet. Still working on tests! :) Sounds good I'll go ahead and re-open this there tomorrow/during the weekend.

[devanbenz]

@rebasedming I'm assuming that this is not pushing down to duckdb per the query plan:

[rebasedming]

Nice! Not sure if you're done with this PR yet, but you'll need to test that time_bucket queries when pushed down to DuckDB yield the same results as those not pushed down and fall back to your time_bucket UDF.
On another note, we're moving pg_lakehouse to its own repo and renaming to pg_analytics. I would recommend opening PRs to that repo, it will make our migration process easier. paradedb/pg_analytics

Ah yeah I'm not quite finished with this yet. Still working on tests! :) Sounds good I'll go ahead and re-open this there tomorrow/during the weekend.

Not necessarily. EXPLAIN doesn't work well, there's a separate open issue for that too. Best way to see if it's being pushed down is adding break points in the executor hook.

[devanbenz]

Opened in new repo: paradedb/pg_analytics#32 Closing this one!