Re: [PR] [KYUUBI #5884] PVM should inherit MultiInstance and wrap with new exprId [kyuubi]

["jersy (via GitHub)" <gi...@apache.org>]

jersy commented on PR #5885:
URL: https://github.com/apache/kyuubi/pull/5885#issuecomment-1972451999

   @AngersZhuuuu 
   
   Brother, I have a question that's been bothering me. 
   If PermanentViewMarker inherits LeafNode and the RuleApplyPermanentViewMarker rule is registered during the resolution phase. If the PermanentViewMarker LogicalPlan is generated during the resolution phase with an empty children property, will it affect other rules that read the children property? For example, RuleApplyRowFilter.
   


-- 
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: notifications-unsubscribe@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org