Merged PRs
dolt
- 10496: /{docker,go,integration-tests}: install git in docker containers, fix .git suffix on clone in sql-server
go-mysql-server
- 3427: Do not wrap hoisted subquery in Limit node until AFTER checking if it's a SubqueryAlias
Wrapping subquery nodes in aLimitnode before checking if it was aSubqueryAliaswas causing us to not findSubqueryAliasnodes. This was noticed when investigating #10472. We should probably be inspecting the whole node instead of simply checking the node type, but this is a quick fix.
filed #10494 to add better test coverage and address TODOs - 3426: Do not push filters into SubqueryAliases that wrap RecursiveCTEs
fixes #10472
All RecursiveCTEs are wrapped by a SubqueryAlias node. We currently don't push filters through a RecursiveCTE so pushing the filter below the SubqueryAlias just causes it to be awkwardly sandwiched in between and might make some queries less optimal if the filter contains outerscope columns since the SQA gets marked as uncacheable. We probably can push filters through a RecursiveCTE in some cases, but we should spend more time thinking about what that means (see #10490).
Also contains some refactors that I noticed along the way.
skipped tests will be fixed in #3427