Merged PRs
dolt
- 7901: Zachmu/schemas2 merge
- 7899: Properly add database collation change when using
-a
option indolt commit
This PR fixes a case where we don't properly handle database collation changes with the-a
option indolt commit
.
fixes #7897 - 7888: [sort] index build streams sorted edits
Use sorting to skip more steps building a prolly map. Shaves maybe 20-25% off of external index rebuilds.
This also fixes a bug where we were incorrectly using only the prefix descriptor to sort secondary index keys.
go-mysql-server
- 2512: Spooling shortcut for one/zero return schemas
Nodes that return zero or one row don't need a beefy channel/wait group setup to execute. They just need to grab the first row and close the iterator. There are several nodes that incorrectly reported their schemas previously, which I've updated to be more accurate. There are some nodes that optionally return rows, which I've simplified to return an empty schema that can be differentiated from the nil schema. We could make the distinction more explicit, also.
bump with perf here: #7894
vitess
- 348: Allowing caching plugin to be specified in string quotes
TheCREATE USER ... IDENTIFIED WITH
syntax (MySQL ref) allows the caching plugin to be specified in string quotes, but our parser only supported identifier quotes.
This came up as part of binlog replication testing – MySQL was sending aCREATE USER
statement from the primary to a Dolt replica, but Dolt wasn't able to parse the statement because of the use of string quotes around the caching plugin name. - 347: Added InjectedStatement
This is the same asInjectedExpr
, except for statements instead of expressions.