microbluey commented on PR #5125: URL: https://github.com/apache/calcite/pull/5125#issuecomment-5260004210
You're right, and I'm sorry for the noise — there is no bug on main. The failure was mine. `core/build/diffrepo/.../RelOptRulesTest_actual.xml` was a leftover from 27 July, and `DiffRepository` compares the reference XML against that file. Main had gained 9 new `TestCase` entries since then (including `testEmptyCalc`), so the stale actual was missing them and the comparison failed. I had also run a `--tests` subset earlier, which is what left a partial file behind. After `rm -rf core/build/diffrepo core/build/test-results/test` and a `--rerun-tasks` run: - `origin/main` — `BUILD SUCCESSFUL`, no failures - this branch — 930 tests, 0 failures, 0 errors So nothing to file. Sorry for holding up the merge over 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
