Merged PRs
dolt
- 7217: Nonsense config warning prints to stderr
Updates the config warning to print tostderr
instead ofstdout
so as to not clutter up output
go-mysql-server
- 2224: empty string is 0.0 decimal
fix #7222 - 2223: Fix multi db views with overlapping name
fixes #7223 - 2221: fix decimal max precision check
- 2220: guard ctx functions with nil check
fixes #7216 - 2219: Implement fmt.Stringer for JsonDocument.
GMS side of an upcoming Dolt PR.
I'm improving error handling in schema merge tests by having the tests print the de-serialized values in incorrect tables instead of just their bytes. Since table values are passed around as aninterface{}
, it seems reasonable to have them implement Stringer. - 2204: fix foreign key panic on update
When resolving ForeignKeys,ForeignKeyReferenceHandler.CheckTable()
callsPartition()
and iterates over all rows in the referenced table to ensure that the key is valid (doesn't contain any values that are not in the referenced table).
WriteableIndexedDoltTable
prevents us from reading every value throughPartition()
as it is only supposed to return rows that match the index range. Additionally, there are some differences between the way MySQL and dolt handle certain ddl and dml operations during delayed resolution of Foreign Keys.
To fix this, we selectively perform table scans (only when we are creating/altering the table to resolve foreign keys).
Fixes #6963
Companion PR: #7203