Jackie-Jiang commented on issue #8453: URL: https://github.com/apache/pinot/issues/8453#issuecomment-1085194696
The operator name is a little bit misleading. For this query, we will first evaluate the `type='PullRequestEvent'` predicate using inverted index, and get a bitmap; then pass the bitmap to the scan-based operator to evaluate the `repo_name like 'FeedScale%'` predicate by scanning the docs in the bitmap. The issue for this query is that we should not fully evaluate the scan-based predicate, but scan on demand, and stop at 100 documents. If the query is aggregation (requires all documents to be evaluated), then it is okay. We should add a flag to the filter operator denoting whether all documents need to be evaluated (lazy evaluation). -- 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
