Re: [PR] fix: allow free RANGE window frames over Duration and Interval ORDER BY types [datafusion]

["edubraqd (via GitHub)" <gi...@apache.org>]

edubraqd commented on PR #24916:
URL: https://github.com/apache/datafusion/pull/24916#issuecomment-5541170366

   Thanks, all three addressed:
   
   - **All ORDER BY expressions are now checked** for the free-range fallback, not just the first one: each key has to be an arithmetic type or one accepted by `supports_free_range_frame`. `window.slt` gains `ORDER BY duration, map` (rejected at planning time) and `ORDER BY duration, duration` (runs).
   - **Title and description** now say Duration / Interval, and the PR is "Part of #24915" rather than closing it; `Struct` / `Map` are left for a follow-up on the comparators.
   - **semver check**: the missing `coerce_asof_join` was only the branch being behind `main`; rebased on current `main`.
   
   On the number of open PRs: understood. I left #24916, #24903 and #24897 ready for review and converted the other eleven to drafts; I'll promote one draft at a time as these land.
   


-- 
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-unsubscribe@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org