Skip to content

Conversation

@tbantle22
Copy link
Contributor

@tbantle22 tbantle22 commented Nov 12, 2024

GMS PR: dolthub/go-mysql-server#2747

  • Support for dolt schema for the schemas and procedures dolt system tables
  • Support for adding a schema_name column to the dolt.schemas table for doltgres
  • Improve logic for system table functions to account for doltgres system tables in the dolt schema

@coffeegoddd
Copy link
Contributor

@tbantle22 DOLT

comparing_percentages
100.000000 to 100.000000
version result total
c4a3524 ok 5937457
version total_tests
c4a3524 5937457
correctness_percentage
100.0

Comment on lines +2318 to +2327
if resolve.UseSearchPath && schemaName == "" {
root, err := db.GetRoot(ctx)
if err != nil {
return err
}
schemaName, err = resolve.FirstExistingSchemaOnSearchPath(ctx, root)
if err != nil {
return err
}
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's kinda gross IMO that I need to set db.schemaName to dolt above to get the dolt.schemas table, but then set it to the first existing schema here to get the view to drop. Option to suggestions

@coffeegoddd
Copy link
Contributor

@tbantle22 DOLT

comparing_percentages
100.000000 to 100.000000
version result total
f57331c ok 5937457
version total_tests
f57331c 5937457
correctness_percentage
100.0

@tbantle22 tbantle22 closed this Nov 14, 2024
@tbantle22 tbantle22 deleted the taylor/proc-sch branch May 28, 2025 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants