alamb commented on PR #12857: URL: https://github.com/apache/datafusion/pull/12857#issuecomment-2417308020
> @alamb Before we merge this PR, I think it will be better to merge the bug fix done in `BuiltInWindowFunction::{Lead,Lag}` here: > > * [Fix: handle NULL input in lead/lag window function #12811](https://github.com/apache/datafusion/pull/12811) > > I can port the bug fix over to this same PR after #12811 is merged. After that we can merge this to main. > > Is that ok? Sounds like a plan. I just merged #12811 -- just let me know when this PR is ready -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org