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.29.6 #157667

Merged
merged 2 commits into from
Dec 18, 2023
Merged

dolt 1.29.6 #157667

merged 2 commits into from
Dec 18, 2023

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

release notes
# Merged PRs

dolt

  • 7179: go: sqle: DoltDatabaseProvider: If we encounter an error while creating a database, try to clean up after ourselves so we do not leave partially initialized database state around.
  • 7178: go: sqle: cluster: Create missing remotes for cluster replication on startup if they do not already exist.
    Previously dolt sql-server would error if there were existing databases and they were missing any of the remotes listed as cluster replication remotes in the config.yaml. This changes it so that those remotes are automatically created at startup instead.
  • 7175: /go/store/{blobstore,datas,nbs}: working, needs cleanup
  • 7173: go: sqle: cluster: Fix MySQLDb and dolt_branch_control replication for the cases where you have more than one standby server in the cluster.
    Previously these changes would only get replicated to the first database in the cluster.
  • 7171: Db/oc persister
  • 7169: More detailed warning message for import schema not matching
    Updates the warning message for dolt table import to print when the import schema does not match the destination table's schema, not just when there are fewer columns in the import file. Also updates the warning message to list the column names that differ between the import file and the destination table's schema.
    Resolves: Warning during csv import dolthub/dolt#6113
  • 7161: Return a different error message for tx commit conflicts depending on @autocommit
    When a transaction commit conflict occurs, the next steps to take are slightly different depending on whether @autocommit is enabled or not. This PR changes the error message we return to describe those steps in more detail when @autocommit is enabled.
  • 7153: proto/third_party/flatbuffers,go/gen/fb/serial: Bump flatc to pick up the version that only has Try accessors.
  • 7137: Adding tests for column name metadata through MySQL C++ and Rust Connector
    Added assertions that result set column name metadata is populated for the C++ and Rust MySQL Connector libraries.
  • 7115: serialize on update expressions
    This PR add serialization for ON UPDATE expressions on columns.
    This change requires changes to the flatbuffer schema as we have to store an additonal field.
    Older versions are able to read and modify databases in the newer version without any error, but any ON UPDATE functionality is missing (which makes sense). Tables with ON UPDATE ... defined on them won't be displayed and performing an UPDATE won't trigger any of the defined expressions to run. So, this should be a Feature Verison bump.
    companion pr: support on update current_timestamp dolthub/go-mysql-server#2188
    fixes ON UPDATE CURRENT_TIMESTAMP in column definition does not update TIMESTAMP dolthub/dolt#6030
  • 7114: .github: Iterate on getting bats tests running with lambdabats.

go-mysql-server

  • 2203: guard ctx.Span with nil check
    We have places where we call expression.Eval(nil, nil).
    The null context causes a panic when we call Span on it.
    This PR just adds a nil check inside the *context.Span() receiver
    fixes Panic when Using BETWEEN and CASE WHEN dolthub/dolt#7154
  • 2200: Replace carder with stats
    I wanted to separate 1) moving cardinality into stats objects, and 2) changing costing to start at join relations, but I kept finding small bugs and hard a time separating the two. So i did the move, fixed bugs along the way, and then regenerated plans with the changes.
    Our current join plans are less useful after we start doing costed join reordering, which makes it kind of annoying that these refactors are dominated by mostly irrelevant plan changes. Still I think the current refactors were necessary, because we need the costing changes and users will still use stats-less tables. Soon I will start adding more unit-esque plan tests that are equally comprehensive but test fewer things, like whether certain plans' cardinalities are accurate, and how closely cost estimates reflect execution runtime.
    Dolt side is passing: [no-release-notes] bump for carder refactor dolthub/dolt#7156
  • 2199: remove delimiter before caching the prepared stmt
    When using prepared statement, some create statements were being stored in the cache with delimiter at the end of it, which does not trimmed when executing the prepared statement. This causes consistency issue where in cases of storing a create statement, it will have the delimiter whereas running the same query without it being prepared statement trims before storing.
  • 2197: prevent RangeJoinHeap plans over Filter nodes
    Currently, we don't have proper handling of nodes that aren't RangeHeaps in the RangeJoinIter.
    This PR makes it so that we don't create plans that would have these nodes.
    Additionally, we error instead of panicking when we see an invalid node in this area.
    fixes panic: interface conversion: sql.Node is *plan.Filter, not *plan.RangeHeap dolthub/dolt#7139
  • 2194: fix decimal result scale difference on server
    This PR fixes:
    • converting the decimal type values to given decimal column type before inserting.
    • getting precision and scale of final decimal type correctly.
  • 2191: Move index costing into join planning phase
    Put index costing inside join planning, so that in the future join planning will have better cardinalities (statistics) for join ordering. Most of the changes will look like refactoring the way we expression index lookups in the memo. I attempted to do this in a way that makes as few changes as possible to join planning; the goal here is to set me up for rewriting cardinality checks with stats objects. It didn't go as cleanly as I wanted, I ended up shifting a lot of join plans back to lookup plans because HASH_JOIN was beating LOOKUP_JOIN in several key places.
    One downside of the current PR is that it converts a sysbench MERGE_JOIN into a LOOKUP_JOIN. I would prefer fixing this in the next PR when I do a bigger costing overhaul.
    Variety of fixes for join hinting, correctness, etc.
    At some point we appeared to fix this:
    Functional Dependency anaylsis on lookup joins doesn't handle when key expression is an Arithmetic expression dolthub/go-mysql-server#1893
  • 2188: support on update current_timestamp
    This PR adds support for handling and executing ON UPDATE column expressions, which are triggered when an update is applied to any column in the row.
    TODO:

Closed Issues

  • 7182: Previous panic concerning CASE WHEN
  • 6030: ON UPDATE CURRENT_TIMESTAMP in column definition does not update TIMESTAMP
  • 6113: Warning during csv import
  • 7154: Panic when Using BETWEEN and CASE WHEN
  • 7148: UPDATE counters SET value = value + 1 WHERE key = 'foo'; should be concurrency-safe
  • 7164: How to connect a dolt sql-server from a remote server

@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 Dec 18, 2023
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 Dec 18, 2023
@BrewTestBot BrewTestBot added this pull request to the merge queue Dec 18, 2023
Merged via the queue into Homebrew:master with commit 4b75b1a Dec 18, 2023
12 checks passed
@BrewTestBot BrewTestBot deleted the bump-dolt-1.29.6 branch December 18, 2023 22:26
@github-actions github-actions bot added the outdated PR was locked due to age label Jan 18, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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 outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants