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.44.0 #200358

Merged
merged 2 commits into from
Dec 7, 2024
Merged

dolt 1.44.0 #200358

merged 2 commits into from
Dec 7, 2024

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

release notes
# Merged PRs

dolt

  • 8646: Allow importing parquet files with arbitrary root column names.
    Parquet represents schemas as trees with a root node, and columns as paths from the root to a leaf.
    parquet-go, the library Dolt uses to export tables as parquet files, uses parquet_go_root as the default name of the root node, but other names are allowed.
    Prior to this PR, Dolt was assuming that the parquet file always had a root node named parquet_go_root, and would fail if any other name was used. This PR changes the behavior to have Dolt read the name from the file instead.
    I also added a test that imports a simple parquet file that was not created with parquet-go.
  • 8645: Removed persitence behavior from config
  • 8639: Fix for updated issue 8623
    This is an actual fix for the following issue. Additional information was added, which actually exposes the root issue at hand. It's possible for the db/revision combo to make it to branch control, which expects just the branch name (since the revision is given under "branch"). This adds the proper split, so that we always get the database name.
  • 8638: go: doltcore: diff: table_deltas.go: Fix perf regression in dolt_add, dolt_commit.
  • 8636: Schema fixes for dolt_status, dolt_merge_status, and dolt_constraint_violations_* tables for doltgres

go-mysql-server

  • 2784: implement EXPLAIN and EXPLAIN PLAN
    Moving our current implementation of EXPLAIN to EXPLAIN PLAN, and replace EXPLAIN with a dummy implementation of MySQL's EXPLAIN
    Looks like this now:
    tmp2/main> explain select * from t;
    +----+-------------+-------+------------+------+---------------+------+---------+------+------+----------+-------+
    | id | select_type | table | partitions | type | possible_keys | key  | key_len | ref  | rows | filtered | Extra |
    +----+-------------+-------+------------+------+---------------+------+---------+------+------+----------+-------+
    | 1  | SELECT      | NULL  | NULL       | NULL | NULL          | NULL | NULL    | NULL | NULL | NULL     |       |
    +----+-------------+-------+------------+------+---------------+------+---------+------+------+----------+-------+
    1 row in set (0.00 sec)
    tmp2/main> explain plan select * from t;
    +------------------+
    | plan             |
    +------------------+
    | Table            |
    |  ├─ name: t      |
    |  └─ columns: [i] |
    +------------------+
    3 rows in set (0.00 sec)
    related: Make Dolt work with mydumper dolthub/dolt#8592
  • 2782: avoid fmt.Sprintf and string alloc for time.Sql
    perf here [no-release-notes] bump for time bytes proto dolthub/dolt#8640
  • 2781: return ok result for select into statements
    Our SELECT ... INTO ... statements return empty result set, which produces strange output in the dolt sql shell.
    MySQL just returns ok results, so we should too.
    discovered in: fix AS OF clause panic for certain expressions dolthub/go-mysql-server#2779
  • 2779: fix AS OF clause panic for certain expressions
    We attempt to parse eval AS OF expressions in the builder (because we assume it is going to be a literal), but Subqueries cannot be evaluated until after they have gone through the analyzer.
    partially addresses: "AS OF" doesn't work with partial commit hashes, trying to improvise this crashes Dolt dolthub/dolt#8635
  • 2776: bump mysql version
    Certain tools expect a higher version of MySQL.
    Currently, the latest stable version of MySQL is 8.4.4, but 8.0.23 is the minimum needed to satisfy mydumper.
    Additionally, this alters the version() method to select directly from the @@version system variable.
    related: Make Dolt work with mydumper dolthub/dolt#8592

vitess

Closed Issues

  • 8623: Branch-qualified DSN gets different branch permissions

@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 7, 2024
Copy link
Contributor

github-actions bot commented Dec 7, 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 Dec 7, 2024
@BrewTestBot BrewTestBot enabled auto-merge December 7, 2024 02:40
@BrewTestBot BrewTestBot added this pull request to the merge queue Dec 7, 2024
Merged via the queue into master with commit 81c35d8 Dec 7, 2024
15 checks passed
@BrewTestBot BrewTestBot deleted the bump-dolt-1.44.0 branch December 7, 2024 02:51
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