Merged PRs
dolt
- 11028: drop remote db cache, Close() after each use
SSH subprocesses created bysql-serverinstances were failing to clean up because theClose()method was never being called. This was the result of some caching done by GetRemoteDB in the sqle package. Most of this change is to removesqle.DoltDatabaseProvider.GetRemoteDB()caching, allowing for push/pull operations to safely Close() remote databases after use. Thus, preventing the PID leak in the associated issue (#10897)
Git remote database singletons are a special case. Previously thepushoperation only would cache these instances - which are expensive to create initially since they need to perform git operations to get in sync with the remote. There is no reason go Close() them though, so we keep a singleton for the duration of thesql-serverprocess.
Fixes: #10897 - 10990: replace all
decimal.Decimalwith*apd.Decimal
Depends on dolthub/go-mysql-server#3532 - 10953: fix autogc load scheduler to be independent on number of CPUs
Before, we had a threshold that was incorrectly calculating the threshold based off the number of CPUs.
The original intention is to prevent AutoGC if a single CPU core exceeds 50% usage.
We've determined that CPU Load itself is not a great metric to schedule AutoGC, but this is a fix for now.
addresses: #10944
go-mysql-server
- 3539: Bug fix for dropping sort nodes
Also fixed a couple under-specified goup concat tests.
The latter were relying on a particular row storage order which is not guaranteed and broke when Dolt changed some encoding parameters. - 3532: replace all
decimal.Decimalwith*apd.Decimal