Alena0704 commented on code in PR #1865:
URL: https://github.com/apache/cloudberry/pull/1865#discussion_r3658834415
##########
src/backend/commands/explain.c:
##########
@@ -1557,13 +1557,12 @@ ExplainNode(PlanState *planstate, List *ancestors,
int motion_recv;
int motion_snd;
ExecSlice *parentSlice = NULL;
- /*
- * Guard if subtree on QE lives in another slice
- * and not instantiated here.
- */
+ /*
+ * Guard against the case where a subtree on the QE lives in another
slice
+ * and is not instantiated in this slice.
Review Comment:
The comment doesn't say where the decision not to instantiate the subtree is
made, so a reader has no starting point. I think you should add a pointer to
ExecInitMotion here:
/*
* Guard against the case where a subtree on the QE lives in another
slice
* and is not instantiated in this slice (see ExecInitMotion).
*/
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
