Tim Saucer

As a new user, I like this change very much. I found creating lead and lag functions to be difficult to use. Suppose I wanted to call lead but provide...

I'm looking at the arguments we end up passing. I think `cume_dist` takes no arguments. Lead and lag take anywhere between 1 and 3. `nth_value` takes two. I think it...

I'm willing to work on this, but I'd like to wrap up the examples I have going into datafusion-python first. I think those will have more impact. I only have...

I should have time to work on this near the end of the week or early next week. I'm in nearing the end of a large PR for `datafusion-python` and...

Commenting here for myself to continue looking further, but I suspect this is happening in `exprlist_to_fields` in `datafusion/expr/src/utils.rs` where we look towards the input fields of aggregates. Because this type...

Quickly took a look at another aggregate function that can return a different type. By calling `count_distinct` instead of `array_agg` and updating the input file to have strings instead of...

Great! I've rebased @alamb 's branch and added the changes I suggested. I was about to start testing the code and then I was going to write up the unit...

Oh, great. Have you been able to run the [example code above](https://github.com/apache/datafusion/issues/6747#issuecomment-2090260284) using the new easy interface?

Thank you. I pulled your branch and many of the tests are failing for me even though **the functions are returning correct values** when I add additional debug statements. I...

The one thing I think we're missing is the other variants for these. I don't think it's covered in other unit tests that I can find. So for example, for...