Skip to content

MSQ: Use MapBinder for QueryKit implementations.#18875

Merged
gianm merged 3 commits intoapache:masterfrom
gianm:msq-querykit-extensions
Jan 1, 2026
Merged

MSQ: Use MapBinder for QueryKit implementations.#18875
gianm merged 3 commits intoapache:masterfrom
gianm:msq-querykit-extensions

Conversation

@gianm
Copy link
Contributor

@gianm gianm commented Dec 29, 2025

This allows extensions to provide QueryKit implementations.

This allows extensions to provide QueryKit implementations.
@github-actions github-actions bot added Area - Batch Ingestion Area - MSQ For multi stage queries - https://github.com/apache/druid/issues/12262 labels Dec 29, 2025
@gianm gianm merged commit 2720cfb into apache:master Jan 1, 2026
104 of 106 checks passed
@gianm gianm deleted the msq-querykit-extensions branch January 1, 2026 17:37
gianm added a commit to gianm/druid that referenced this pull request Jan 13, 2026
PR apache#18875 caused a regression in WindowOperatorQuery planning, by
causing non-outermost WindowOperatorQuery to be planned without
windowFunctionOperatorTransformation (see apache#17443). This happened because
only the outermost query has the windowFunctionOperatorTransformation
parameter.

This patch fixes the problem by propagating the context parameter from
the outermost queries to any inner WindowOperatorQuery instances.
gianm added a commit that referenced this pull request Jan 13, 2026
PR #18875 caused a regression in WindowOperatorQuery planning, by
causing non-outermost WindowOperatorQuery to be planned without
windowFunctionOperatorTransformation (see #17443). This happened because
only the outermost query has the windowFunctionOperatorTransformation
parameter.

This patch fixes the problem by propagating the context parameter from
the outermost queries to any inner WindowOperatorQuery instances.
@kgyrtkirk kgyrtkirk added this to the 36.0.0 milestone Jan 19, 2026
kgyrtkirk pushed a commit that referenced this pull request Jan 20, 2026
PR #18875 caused a regression in WindowOperatorQuery planning, by
causing non-outermost WindowOperatorQuery to be planned without
windowFunctionOperatorTransformation (see #17443). This happened because
only the outermost query has the windowFunctionOperatorTransformation
parameter.

This patch fixes the problem by propagating the context parameter from
the outermost queries to any inner WindowOperatorQuery instances.

(cherry picked from commit 07de2ff)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area - Batch Ingestion Area - MSQ For multi stage queries - https://github.com/apache/druid/issues/12262

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants