timsaucer commented on issue #15072:
URL: https://github.com/apache/datafusion/issues/15072#issuecomment-2798821873

   > > I did a little investigating, but I don't have time for a couple of days 
to dive in deeper. This appears to be related to 
[#15542](https://github.com/apache/datafusion/pull/15542) 
[@UBarney](https://github.com/UBarney) do you know why the sql unit tests 
needed to be changed to pass? It seems like we have a regression in 
`datafusion-python` related to this change.
   > 
   > I think [@andygrove](https://github.com/andygrove) filed a ticket for this 
one
   > 
   >     * [Regression in `last_value` functionality 
#15676](https://github.com/apache/datafusion/issues/15676)
   > 
   > 
   > I didn't fully follow the discussion -- but it seems like that issue has 
been closed
   
   I've read the discussion now and I think I'm in agreement that it's not an 
actual regression since the aggregation has no deterministic outcome without 
ordering assigned.


-- 
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