waterWang commented on issue #24514:
URL: https://github.com/apache/datafusion/issues/24514#issuecomment-5366593113

   Hi! I submitted a fix for this in #24556.
   
   The root cause was the same as the binary order-key issue (#24327, fixed in 
#24357): `extract_window_frame_target_type` in 
`datafusion/optimizer/src/analyzer/type_coercion.rs` had no arm for 
`Time32`/`Time64`, so free range frames (the default `RANGE BETWEEN UNBOUNDED 
PRECEDING AND CURRENT ROW`) hit the internal error fallback.
   
   The fix adds `Time32`/`Time64` support to the free-range arm, and the 
existing offset-arithmetic guard now rejects `RANGE ... INTERVAL 
PRECEDING/FOLLOWING` over time keys with a proper planning error (time/interval 
arithmetic wraps around the 24h clock).


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to