amrishlal edited a comment on pull request #8446: URL: https://github.com/apache/pinot/pull/8446#issuecomment-1085214751
> @amrishlal Did you intentionally skip the `DocIdSetOperator`, or that one missing unexpectedly? Yes it was skipped intentionally but there wasn't any strong reason behind it. At the time, we just decided to do the initial merge and postpone improvements such as putting in `DocIdSetOperator` for later incremental PRs. We probably need to do some work on showing Broker side processing as well in the EXPLAIN PLAN. -- 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
