Skip to content

dolt 1.30.6#159035

Merged
BrewTestBot merged 2 commits intoHomebrew:masterfrom
BrewTestBot:bump-dolt-1.30.6
Jan 5, 2024
Merged

dolt 1.30.6#159035
BrewTestBot merged 2 commits intoHomebrew:masterfrom
BrewTestBot:bump-dolt-1.30.6

Conversation

@BrewTestBot
Copy link
Copy Markdown
Contributor

Created by brew bump


Created with brew bump-formula-pr.

Details

release notes
# Merged PRs

dolt

  • 7257: Update MySQL Connector/J integration tests to cover table-qualified field lookups
    When using the MySQL Connector/J library to pull out a field value from a row, you can specify the column name, or you can specify the table-qualified column name (e.g. myTable.col1). Add source table and database to field metadata in wire responses dolthub/go-mysql-server#2235 fixed the missing metadata, and this PR adds an integration test for the MySQL Connector/J library to ensure we don't regress.
    Related to Problem with selecting fully qualified column names dolthub/dolt#7247
  • 7252: dolt_pull performance improvement
    dolt_pull currently performs unnecessary FastForwards for remote branches. There is a more optimal mechanism to fetch all remotes, actions.FetchRefSpecs, and by using that we greatly increase the speed of dolt_pull when there are lot of remote branches.
  • 7249: fix minver
  • 7248: omit empty on new yaml items

go-mysql-server

  • 2238: [memo] RequiredIndex interface
    Some table nodes need to be executed as IndexScans with mandatory filters. The new interface makes this transparent to join planning.
    re: [no-release-notes] bump and test for required commit_diff index predicates dolthub/dolt#7256
  • 2236: trim floats when hashing
    fmt.Sprintf("%v", x) writes floats with a x.0 which causes it to never equal hashed strings.
    Initially, I wanted to always convert the LHS to the type in the RHS, but this is difficult when there are multiple types in the RHS
    fixes Unexpected Result when Using -'' in IN dolthub/dolt#7246
  • 2235: Add source table and database to field metadata in wire responses
    The field metadata messages we were sending back for a result set did not include the source table and source database for fields. This caused a behavior difference from MySQL where the table-qualified column name (e.g. table1.ID) would not work with Dolt when using the MySQL Java Connector library. See Problem with selecting fully qualified column names dolthub/dolt#7247 for more details.
    Updates to unit tests for the schemaToFields code are in this PR, and I'll follow up with a PR in the Dolt repo that updates our MySQL Connector library integration tests to add coverage for the table-qualified column name.
  • 2234: use cached decimal type for casts
    Fixes a variety (hopefully 49) of SQLLogicTests involving CASTS(... AS DECIMAL)
  • 2233: Add logging for COM_RESET_CONNECTION command
    Adding debug logging to see when clients are sending this command, even though we don't implement it yet. For example, ORMs may send this command when returning a connection to a connection pool, so it may be helpful to implement this command and clear out session state in that case.
  • 2230: guard ctx and session with nil
    fixes Panic when Using INNER JOIN dolthub/dolt#7235

vitess

  • 298: parse empty begin end block in trigger
    14 SQL correctness tests are failing because we throw syntax errors on queries like this:
    CREATE TRIGGER t1r1 AFTER UPDATE ON t1 FOR EACH ROW BEGIN END;
  • 297: Fixing the version keyword to not require identifier quotes
    The version keyword still required identifier quoting in some usages, such as SELECT * FROM base.version;. See Unable to query tables named version in another database without escaping dolthub/dolt#7237 for more details.
    This change moves the version keyword into the main list of non-reserved keywords. There was one conflict from use of the version keyword in the function_call_keyword rule, but it turns out that use of version there is not required. We have an existing test for using the version() function, so I didn't add a new one.

Closed Issues

  • 7246: Unexpected Result when Using -'' in IN
  • 3467: Support dolt rebase
  • 7237: Unable to query tables named version in another database without escaping

@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 5, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 5, 2024

🤖 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 5, 2024
@BrewTestBot BrewTestBot enabled auto-merge January 5, 2024 04:03
@BrewTestBot BrewTestBot added this pull request to the merge queue Jan 5, 2024
Merged via the queue into Homebrew:master with commit 072cef0 Jan 5, 2024
@BrewTestBot BrewTestBot deleted the bump-dolt-1.30.6 branch January 5, 2024 04:14
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