xuzifu666 commented on PR #4641: URL: https://github.com/apache/calcite/pull/4641#issuecomment-3556407103
> @dssysolyatin Yes, it does seem challenging to write this test. As an alternative, could we first perform a manual audit to check if all necessary RelNode classes have implemented the visit method? If the audit confirms they do, would it be acceptable to merge that PR #4620 first? Because it is indeed a defect. @dssysolyatin This is indeed a problem. Do you agree with this suggestion? -- 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]
