Merged PRs
dolt
- 7535: Allow importing tables when the columns contain characters that require escaping.
dolt table import
was previously generating invalid SQL statements for importing these tables. - 7531: bump sqllogictest version
- 7516: Less docker layers and more compact Docker image
Install both packages on one Docker layer and delete package cache after installation.
Docker image size decreased from 259 MB to 210 MB. - 7514: Bump actions/setup-go and use Go version from go.mod file
Changes:- Bump actions/setup-go
- Use Go version from go.mod file
- Move actions/checkout before actions/setup-go
go-mysql-server
- 2343: fix json bool comparison
JSON Boolean values are special; they do not behave like normal when converted/compared against other types.
fixes #7528 - 2336: fix rangetree removal of largest node without left child
When removing a right child, a missing nil guard was causing us to improperly propagate the new max range.
This resulted in overlapping ranges.
This fixes 1 sqllogictest. - 2335: Schema pruning considers database name
Fix some inconsistencies with project and join hints. New test added for new project pruning changes. - 2334: fix char to bool conversion
This PR adds missingnil
guards in a few conversion methods.
fixes #7515 - 2333: [planbuilder] Skip post-aggregate projection when there are no aliases in target scope
re: #6982
This takes the last big chunk of CPU pressure off of TPC-C execution, which is mostly disk limited now:
- 2332: [time] shortcut for no timezone conversion
This function reads files from disk to try to convert a timezone to itself. There are other optimizations we could do to maybe try checking for the integer offset before doing syscalls.
re: #6982, ~10% TPC-C boost - 2331: fix
NOT
expression in conjunction withWHERE EXISTS(<subquery>)
The analyzer ruleunnest_exists_subqueries
was accidentally droppingNOT
expressions when hoisting subqueries fromWHERE EXISTS...
clauses.
This should fix 8 sqllogictests.
Correctness: #7510 - 2327: memo/planbuilder: Off by one error, and issue importing statistics for testing
The original bug was a one-off error whereconjCollector
was checking for the wrong equivalence column. That spiraled into:- statistics were not properly being used for some tests because PRIMARY case sensitivity in
ANALYZE ... UPDATE HISTOGRAM
statements - one integration test had a weird edge case where OSX and linux compute slightly differently float values, leading to different plans locally vs CI. One option was to truncate the specificity of all cost values with a multiply/round to avoid numerical weirdness...picking the first lowest plan also seems to work.
- variety of other small bugs in costing
- update join bug where the projections after a rearrangement lost expression id/nullability info. This was exposed by the changes above
- statistics were not properly being used for some tests because PRIMARY case sensitivity in