Skip to content

dolt 1.32.2#160784

Merged
BrewTestBot merged 2 commits intoHomebrew:masterfrom
BrewTestBot:bump-dolt-1.32.2
Jan 24, 2024
Merged

dolt 1.32.2#160784
BrewTestBot merged 2 commits intoHomebrew:masterfrom
BrewTestBot:bump-dolt-1.32.2

Conversation

@BrewTestBot
Copy link
Contributor

Created by brew bump


Created with brew bump-formula-pr.

Details

release notes
# Merged PRs

dolt

go-mysql-server

  • 2286: Earlier LOAD DATA and insert validation
    re: Move validation for InsertInto nodes into the analyzer's validation phase dolthub/go-mysql-server#2283
    fixes: sql server panic when uploading table dolthub/dolt#7313
  • 2207: Fix partial left join update when columns have default values or join table is in where clause
    I ran into an issue where I was getting a panic when in update queries using left joins with at least some rows being null. The cases where I could confirm this happens is when the query planner wraps the JoinNode in Project or Filter or both. The logic that is supposed to skip null rows for update fails to skip when the node is not directly a JoinNode. I don't know if there are other node types that should be handled the same way in the toJoinNode function I added. This fix resolved the issues I was having.
    Side note:
    Another related issue I ran into while working on the tests for this was the same type of error happening when the join table is backed by IndexedTableAccess. It seems to set the root join filter to true, with a nested node that has the actual filter. The true filter causes the update skip logic to allow updates on every attempted join, even when null. This was happening for a bit but it stopped while I was messing with the test data so I couldn't keep diagnosing it.

vitess

Closed Issues

  • 7313: sql server panic when uploading table

@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 Jan 24, 2024
@github-actions
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 Jan 24, 2024
@BrewTestBot BrewTestBot enabled auto-merge January 24, 2024 03:03
@BrewTestBot BrewTestBot added this pull request to the merge queue Jan 24, 2024
Merged via the queue into Homebrew:master with commit 617b99e Jan 24, 2024
@BrewTestBot BrewTestBot deleted the bump-dolt-1.32.2 branch January 24, 2024 03:16
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