gianm commented on code in PR #17057:
URL: https://github.com/apache/druid/pull/17057#discussion_r1759537858
##########
processing/src/main/java/org/apache/druid/query/groupby/GroupingEngine.java:
##########
@@ -469,13 +465,16 @@ public QueryRunner<ResultRow> mergeRunners(
* @param query the groupBy query
* @param cursorFactory cursor factory for the segment in question
* @param timeBoundaryInspector time boundary inspector for the segment in
question
+ * @param bufferPool processing buffer pool
+ * @param groupByQueryMetrics metrics instance, will be populated if
nonnull
*
* @return result sequence for the cursor factory
*/
public Sequence<ResultRow> process(
GroupByQuery query,
CursorFactory cursorFactory,
@Nullable TimeBoundaryInspector timeBoundaryInspector,
+ NonBlockingPool<ByteBuffer> bufferPool,
Review Comment:
Could provide the `GroupByResourcesReservationPool` to `process` also, I
suppose.
--
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]
