[GitHub] [pinot] agavra commented on a diff in pull request #9560: [multistage] refactor traversals of stage nodes into visitor pattern

[GitBox <gi...@apache.org>]

agavra commented on code in PR #9560:
URL: https://github.com/apache/pinot/pull/9560#discussion_r993636290


##########
pinot-query-planner/src/main/java/org/apache/pinot/query/planner/stage/MailboxReceiveNode.java:
##########
@@ -32,23 +32,31 @@ public class MailboxReceiveNode extends AbstractStageNode {
   private RelDistribution.Type _exchangeType;
   @ProtoProperties
   private KeySelector<Object[], Object[]> _partitionKeySelector;
+  @ProtoProperties
+  private StageNode _sender;

Review Comment:
   👍 sounds good, I was a bit worried about when exactly this would be used or not but I'm happy to remove it!



-- 
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