Merged PRs

dolt

• 8487: Fix server init race Fixes a race between setting the debug log output on DoltDb and running statistics configuration.
• 8465: go/libraries/doltcore/doltdb: root_val.go: Restore ForeignKey TableNames noms serialization as just strings.
• 8453: turn on stats collect by default
• 8451: Stats purge and prune call dolt_stats_prune() should wipe the stats database disk contents, and replace with the currently tracked statistics. This is GC-like behavior. call dolt_stats_purge() should clear the stats database folder and reinitialize it empty. It's equivalent to rm -rf .dolt/stats and then dolt initing a new stats directory. TODO: more testing and docs

go-mysql-server

• 2715: remove exchange node These aren't used anywhere, so it's getting removed. Partition still exists if we ever want to reimplement some version of this.
• 2714: remove plan.QueryProcess and move logic to finalizeIters This PR removes the use of plan.QueryProcess and part of the trackProcess rule that adds/removes this node. Instead the TrackedRowIter is created directly in finalizeIters.
• 2710: Changing selectExprNeedsAlias to consider string literal quotes When fixing a string literal quoting issue in Doltgres (https://github.com/dolthub/doltgresql/pull/868), table functions stopped working, because the selectExprNeedsAlias started returning true to indicate that an alias was needed (even though it wasn't) and the buildTableFunc function would fail. This started happening because the quoted string literals no longer matched InputExpression, since InputExpression always trims off quotes when it is assigned. Another solution could be to expose the trimQuotes function from Vitess and use it to trim expr.String() before matching against InputExpression.
• 2709: fix LOAD DATA 64K buffer limit This PR increase the buffer size for bufio.Scanner to LongTextMax, so hopefully nobody attempts to load a single row larger than 4GB. Other changes:
  • moves ignore lines logic to within the loadDataIter
  • drops extra scanner.Text() logic
  • use byte comparison instead of string cast and string comparison benchmark: https://github.com/dolthub/dolt/pull/8467 fixes: https://github.com/dolthub/dolt/issues/8469
• 2708: removing transaction committing node doltgres fix: https://github.com/dolthub/doltgresql/pull/872
• 2702: fix: make SET system type case-insensitive Resolves #2701
• 2697: expose planbuilder methods for domain support in Doltgres

Closed Issues

• 8469: LOAD DATA LOCAL INFILE fails when row size exceeds 64K (65535 bytes) limit
• 2701: lowercase sql mode values are not supported
• 2671: setting two foreign keys to associate with the same field, deleting data is normal but does not take effect