gortiz commented on PR #10372: URL: https://github.com/apache/pinot/pull/10372#issuecomment-1458498923
Ok, but that is not achieving that much from what I've got. I mean, yes, with your option the window size can be changed in other cases. I still think that the buffer should be managed by the caller, but your option require less changes, so lets use that. In my specific case I know the custom window size when the Operator is created, so it doesn't hurt me. But in my use case I would also need to be able to `applyAnd` using an iterator. I have this `applyAnd` that receives ` BatchIterator` but does not include the batch size -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
