Merged PRs

dolt

• 5160: change --out-and-err to append to existing file
• 5157: Fix call DOLT_VERIFY_CONSTRAINTS() false-positive
  Fix for bug reported by Fritter in Discord.
  Here's what happened:
  • Parent table has a simple pk
  • Child table has a compound pk of length 3
  • The foreign key is a single int column. It is the first column in the child table's pk, and the only column of parent's pk.
  • No additional indexes are defined because we can reuse the pk index.
    If we reuse the pk index, we have some special case code that builds a fake index definition. The issue is that we happened to pass sorted tags of the pks, instead the column-order tags. We then use the index definition to prefix iter secondary indexes.
    This issue would not appear if you got lucky with the tags of your columns. Specifically, the sorted tag order had to match the tag order. For tables with one pk, this is always the case, and for the test we have for two pks, we also happen to get lucky. Unfortunately, Fritter has three column compound pk and he did not get lucky.
• 5152: go/libraries/doltcore/sqle/read_replica_database.go: Optimize the sql-server read replica case when syncing a lot of new branches.
  We already fetch all the new commits we need at once. Some lingering code had us refetching the commit values themselves from the upstream instead of directly setting the new branch to the commit hash which we just fetched.

go-mysql-server

• 1541: truncate really long queries
• 1539: if privilegeSet is nil, return empty row for info schema tables
  • privilegeSet was nil for ld repository tests as there were no privileges granted and was trying to test on information_schema.views table.
  • the now exported method is used in ld to grant all privileges for the test user
    fixes test in https://github.com/dolthub/ld/pull/12642

Closed Issues

Latency