Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dolt 1.43.10 #196171

Merged
merged 2 commits into from
Oct 31, 2024
Merged

dolt 1.43.10 #196171

merged 2 commits into from
Oct 31, 2024

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

release notes
# Merged PRs

dolt

  • 8508: Update more system tables for doltgres
  • 8505: Fix panics on schema changes
    fixes: Crash when refreshing statistics after a PK column change dolthub/dolt#8504
    Use schema hash to detect when database statistics might be incompatible. Drops all statistics for schema update. This could be improved to a subset of statistics if we wanted in the future.
  • 8501: Bug fixes for replication
    • Convert @@server_id to uint32 value when it gets loaded as a string from config.json
    • Send binary logfile name in server heartbeat events, now required in MySQL 8.4
  • 8499: scanner more safe whitespace skip
  • 8497: Update the protobuf version from 22.0 to 28.3
    Also update README to reflect the bazel 6.3.0 version
  • 8493: add bats test for kill query <pid>
  • 8482: Move dolt system tables to dolt schema for doltgres

go-mysql-server

  • 2727: fix dropped error in createView
  • 2726: remove resolveDescribeQuery rule
    This rule just recurses into the analyzer, so we should just do it right at the beginning, so we should do it right away and avoid calling all other rules.
    This would be better/cleaner in the planbuilder, but we would need to tell the analyzer to not analyze this node or skip it somehow.
    It's possible to move this logic into analyzeWithSelector.
    It might be more difficult to move this above Analyze, as we'd have to put this logic in a variety of places.
  • 2723: server/handle.go: doQuery cleanup of some error handling.
  • 2722: internal/sockstate: Restore connection checking on Linux.
    GMS server handler is supposed to cancel running queries if the connection which issued them goes away. It does this by checking the connection state out-of-band anytime the query is running and canceling theh query if the connection goes away. The connection checking code is platform-specific and currently only works on Linux.
    In commit 538696b2c943ac7f3cacf1b67a3a5ff40ae64a00 I introduced a bug where the connection checking code tries to inspect the socket state of an already closed file descriptor. This change fixes the behavior so that the file descriptor is left open until the necessary socket state is extracted.
  • 2720: move wrapTriggerRollback logic
    This PR removes the plan.TriggerRollback and plan.NoopTriggerRollback node and applies the triggerRollbackIter directly.
    When there are multiple nested triggers (aka triggers that call other triggers), we used to have multiple nested trigger rollbacks with different savepoints. It turns out that was unnecessary and a single savepoint at the top of the iter tree is enough; when there's an error, it propagates back to the top, rollingback all changes.
  • 2718: use sqlCtx in handler
    Tests here: add bats test for kill query <pid> dolthub/dolt#8493
  • 2717: push Distinct nodes below Sort nodes
    This PR optimized queries like:
    select distinct i from t order by i;
    When there are many duplicate values in column i, it is much more efficient to eliminate duplicates first, then sort the results.
    There are somewhat unrelated optimizations left as TODOs.
    Optimize query: excessive memory usage during SELECT DISTINCT query dolthub/dolt#8488

vitess

Closed Issues

  • 8504: Crash when refreshing statistics after a PK column change

@github-actions github-actions bot added go Go use is a significant feature of the PR or issue bump-formula-pr PR was created using `brew bump-formula-pr` labels Oct 31, 2024
Copy link
Contributor

🤖 An automated task has requested bottles to be published to this PR.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Oct 31, 2024
@BrewTestBot BrewTestBot enabled auto-merge October 31, 2024 01:40
@BrewTestBot BrewTestBot added this pull request to the merge queue Oct 31, 2024
Merged via the queue into master with commit 7b11e4f Oct 31, 2024
15 checks passed
@BrewTestBot BrewTestBot deleted the bump-dolt-1.43.10 branch October 31, 2024 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. go Go use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants