You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2022/03/31 23:04:21 UTC

[GitHub] [pinot] amrishlal commented on pull request #8446: Add DocIdSetOprator to explain plan

amrishlal commented 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.


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org