asolimando commented on PR #4668: URL: https://github.com/apache/calcite/pull/4668#issuecomment-3626202678
Since the PR has been approved and I have addressed the two pending comments, refine comments and add the new tests of `RelOptRuleTest` to `sub-query.iq` as well (minor modifications with no impact on the code), I labeled the PR as "LGTM will merge soon", and I will merge within 24/48 hours if no further comments arise in the meantime. Not sure what cancelled CI tasks, but I have re-run them manually and they all pass: https://github.com/apache/calcite/actions/runs/20024112894/job/57418490793?pr=4668 -- 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]
