gortiz opened a new pull request, #18847:
URL: https://github.com/apache/pinot/pull/18847
## What
Adds an optional overload to `QueryExecutor.execute(...)` that accepts a
caller-supplied `PlanMaker`, used for that single execution instead of the
executor's configured plan maker:
```java
default InstanceResponseBlock execute(ServerQueryRequest queryRequest,
ExecutorService executorService,
@Nullable ResultsBlockStreamer streamer, @Nullable PlanMaker
planMakerOverride);
```
- The default implementation ignores the override, so it is backward
compatible for any existing `QueryExecutor` implementation.
- `ServerQueryExecutorV1Impl` honors it, threading the override down to plan
building (`planCombineQuery`). Passing `null` is equivalent to the existing
3-argument `execute(...)`.
## Why
This lets a caller plug in alternate segment-level plan nodes/operators for
a single execution without reconfiguring the executor (for example, to produce
a different result-block representation). It is a small, opt-in extensibility
hook.
All existing call sites pass `null`, so behavior is unchanged.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
