cryptoe commented on code in PR #17405:
URL: https://github.com/apache/druid/pull/17405#discussion_r1820093994
##########
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/querykit/WindowOperatorQueryFrameProcessorFactory.java:
##########
@@ -56,26 +56,26 @@
@JsonTypeName("window")
public class WindowOperatorQueryFrameProcessorFactory extends
BaseFrameProcessorFactory
{
- private final WindowOperatorQuery query;
+ private final QueryContext queryContext;
private final List<OperatorFactory> operatorList;
private final RowSignature stageRowSignature;
@JsonCreator
public WindowOperatorQueryFrameProcessorFactory(
- @JsonProperty("query") WindowOperatorQuery query,
+ @JsonProperty("queryContext") QueryContext queryContext,
@JsonProperty("operatorList") List<OperatorFactory> operatorFactoryList,
@JsonProperty("stageRowSignature") RowSignature stageRowSignature
)
{
- this.query = Preconditions.checkNotNull(query, "query");
+ this.queryContext = Preconditions.checkNotNull(queryContext, "query");
this.operatorList = Preconditions.checkNotNull(operatorFactoryList, "bad
operator");
this.stageRowSignature = Preconditions.checkNotNull(stageRowSignature,
"stageSignature");
}
- @JsonProperty("query")
- public WindowOperatorQuery getQuery()
+ @JsonProperty("queryContext")
Review Comment:
Window functions are GA in druid 31. We should not break backward
compatibility in the next release.
The way to do it now is to
1. Mark the older queryContext deprecated. Add a release note in the
developer section basically mentioning that we will remove this parameter in
druid 33.
2. The current code handles both query/query context and takes certain
assumptions. Preference given to query
3. We remove the deprecated param and the scafolding code in druid 33.
--
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]
