amrishlal commented on a change in pull request #7568:
URL: https://github.com/apache/pinot/pull/7568#discussion_r759890644
##########
File path:
pinot-core/src/main/java/org/apache/pinot/core/operator/DocIdSetOperator.java
##########
@@ -34,6 +37,7 @@
*/
public class DocIdSetOperator extends BaseOperator<DocIdSetBlock> {
private static final String OPERATOR_NAME = "DocIdSetOperator";
+ private static final String EXPLAIN_NAME = null;
Review comment:
There isn't any big reason behind it, except that we didn't think this
would be useful enough to show in the EXPLAIN PLAN output. @siddharthteotia ?
In either case, I would suggest that we not get into particular operators of
EXPLAIN PLAN and output display format etc. in this PR. As the PR states
EXPLAIN PLAN output is only for information purposes and we are not
guaranteeing that the output will remain backward compatible. So we can change
what is displayed in the EXPLAIN PLAN output anytime as needed.
--
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
