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

doltgres 0.11.1 #182051

Merged
merged 2 commits into from
Aug 22, 2024
Merged

doltgres 0.11.1 #182051

merged 2 commits into from
Aug 22, 2024

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

release notes
# Merged PRs

doltgresql

  • 640: current_schema function return non "$user" schema
  • 636: add Postgres default access methods in pg_am table
    This allows displaying index information on TablePlus
  • 635: More bug fixes for subqueries
    Fixes column resolution problems when using subqueries in joins.
    Partially addresses Column alias from subquery cannot be referenced in join dolthub/doltgresql#554
  • 632: Support for IN with a subquery
    Tuples won't work until we have a doltgres tuple type, but should work for everything else.
    New expression type is a combination of InTuple from Doltgres and InSubquery from GMS. GMS code couldn't really be reused usefully.
  • 631: Unskip some subquery tests, add more workbench tests, allow cross-database references for regclass
  • 624: Bug fix for subqueries
    The context finalizer node was implemented incorrectly in terms of the semantics required for subquery logic to function -- the top-level child was never being considered in the row-prepend process.
    IN clauses with subqueries still do not work, that's next.
  • 621: add test for non default schema qualified view
    depends on handle view resolution when the schema is not defined in SELECT stmt dolthub/go-mysql-server#2633
  • 620: Allow varchar columns as primary keys
    The problem: the varchar type in pgtypes was declared as Text, not Varchar. This triggered validation checks in both GMS and Dolt when such a column was used in a primary key. I originally tried adding additional index / length info in the CREATE TABLE transformation, but with the varchar type being interpreted as Text, there's no combination of params that can satisfy all the validation checks.
    The error before this fix is e.g.
    "ERROR: blob/text column 't' used in key specification without a key length"
    
    Given that we have a lot of validation logic with key prefix lengths related to BLOB / TEXT types, it seems like this type really can't be declared as TEXT, but open to other ideas.
    Collation / charset support is still fake and will need to be fixed, but no worse than it was before.
  • 617: use unknown type for string literal
    This PR also adds some build in functions:
    • pg_get_indexdef - parses
    • pg_indexes_size - parses
    • pg_tables_size - parses
    • pg_total_relation_size - parses
  • 614: Implement information_schema.schemata
    GMS PR: [no-release-notes] Support information_schema.schemata hook for doltgres dolthub/go-mysql-server#2632

Closed Issues

  • 41: TablePlus can Connect and run queries but does not show Table Metadata
  • 14: TablePlus connection kind of works
  • 16: show create table and show tables should not work
  • 64: CREATE SCHEMA not yet supported
  • 171: current_schema() not yet supported
  • 599: querying a schema qualified view from different schema fails

@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 Aug 22, 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 Aug 22, 2024
@BrewTestBot BrewTestBot enabled auto-merge August 22, 2024 19:03
@BrewTestBot BrewTestBot added this pull request to the merge queue Aug 22, 2024
Merged via the queue into master with commit 6293cf4 Aug 22, 2024
15 checks passed
@BrewTestBot BrewTestBot deleted the bump-doltgres-0.11.1 branch August 22, 2024 19:10
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