daniellansun opened a new pull request, #3167: URL: https://github.com/apache/groovy/pull/3167
## Summary Peels the window-function IR / renderer extension-point work out of #3155 into a dedicated PR, per GEP-26 follow-up scope. **Depends on:** [#3155](https://github.com/apache/groovy/pull/3155) (GEP-26 SQL/jOOQ reference continuation). Branch is forked from the current #3155 tip (`gep26-sql` @ `3edd995fd0`). Related: **GEP-26**. ## What this does - First-class IR: `SqlWindowCall`, `SqlWindow`, `SqlWindowFrame`, `SqlWindowBound` - `SqlFunctionMapping.Kind.WINDOW` factory (separate from scalar / aggregate) - Native + jOOQ renderers emit ANSI `OVER` when IR is present - `SqlFunctionRegistry.standard()` stays sealed with **no** window registrations — product path still rejects `over` with the existing deferred message; mixed mode still evaluates in memory - Tests: `SqlWindowTranslationTest` (+ registry coverage) ## Out of scope (later PR) - Registering GINQ window functions on the built-in registry - Full product-path `OVER` push-down / window semantics ## Tip - Branch: `gep26-sql-window` - Tip: `0d104b0084bb` (Author/Committer: Daniel Sun `<[email protected]>`) - Parent: `3edd995fd0` (#3155 head after peel) ## Known CI Lint gate was red on this tip (FullyQualifiedNameInJavadoc / UnnecessaryFullyQualifiedName). Fix landing on this branch separately before merge consideration. Auto-merge paused; do not merge until Daniel notifies resume + review. -- 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]
