zstan commented on PR #3732: URL: https://github.com/apache/ignite-3/pull/3732#issuecomment-2109999626
honestly i don`t know why do we need planner tests here, let`s wait @ygerzhedovich thoughts. In my opinion planner tests can test propagated parameters or for example cases like awaiting predicates in scan node but describe one of possible plans is no good idea, as for me. -- 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]
