Upgrade window UDF api

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

datafusion is in process of removing their built-in window-functions, which our current window function relies on. sum has already been removed which brakes the q11_important_stock_identification.py tpch example.

Additional context

AggregateFunction::Sum enum variant is still defined, but can not be used.

I suspect that the proper solution is to register the new UDAFs with the function registry.

As a workaround for releasing 39, I explicitly match on name == "sum" and redirect to the UDAF.

UDAF Migration Epic: apache/datafusion#8709

marked q11 tpch example as expected fail

@timsaucer

@timsaucer could you take a look at this once #728 is merged?

Upgrade datafusion 39 (#728)

I’ll be happy to but on vacation so it will be a couple weeksOn Jun 14, 2024, at 1:49 PM, Michael J Ward ***@***.***> wrote: @timsaucer could you take a look at this once #728 is merged? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

provides workaround for half-migrated UDAF `sum`

UDAF `sum` workaround (#741)

@Michael-J-Ward

@Michael-J-Ward I know this issue was already closed but I have proposed a method to remove the work around.

Navigation Menu

Repository navigation

Upgrade window UDF api #730

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Upgrade window UDF api #730

Description

Activity

Michael-J-Ward commented on Jun 14, 2024

timsaucer commented on Jun 14, 2024

timsaucer commented on Jul 1, 2024

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions