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

Reply via email to