Drizzle 0.36.0 introduced, among other features, a more concise array syntax for indexes and constraints.
This PR upgrades our dependencies, migrates our schema to that syntax, and also addresses some key issues, namely:
Indexes for GE categories other than Category Ia were incorrectly indexed based on whether the course's is_ge_1a field is true, regardless of what the index was actually named. This PR corrects this issue.
In a similar vein, this PR removes custom index names for all indexes except the course/instructor search GIN indexes, to minimize noise which caused a false sense of correctness.
Updated the dump_db script to also dump the drizzle schema, which Drizzle uses to track migrations. This resolves issues where migrations would fail to apply because some attribute already exists, but Drizzle thought it didn't exist because the drizzle schema did not exist, so it tried to apply that migration.
How Has This Been Tested?
Verified locally that filtering courses by GE categories other than Category Ia exhibits similar performance to filtering by Category Ia.
Types of changes
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
[x] My code involves a change to the database schema.
[ ] My code requires a change to the documentation.
Description
Drizzle 0.36.0 introduced, among other features, a more concise array syntax for indexes and constraints.
This PR upgrades our dependencies, migrates our schema to that syntax, and also addresses some key issues, namely:
is_ge_1a
field is true, regardless of what the index was actually named. This PR corrects this issue.dump_db
script to also dump thedrizzle
schema, which Drizzle uses to track migrations. This resolves issues where migrations would fail to apply because some attribute already exists, but Drizzle thought it didn't exist because thedrizzle
schema did not exist, so it tried to apply that migration.How Has This Been Tested?
Verified locally that filtering courses by GE categories other than Category Ia exhibits similar performance to filtering by Category Ia.
Types of changes
Checklist: