MODFQMMGR-1097 Remove the instance classification number index#1334
Closed
mweaver-ebsco wants to merge 5 commits intomasterfrom
Closed
MODFQMMGR-1097 Remove the instance classification number index#1334mweaver-ebsco wants to merge 5 commits intomasterfrom
mweaver-ebsco wants to merge 5 commits intomasterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR removes four recently added (but unreleased) PostgreSQL index-creation snippets for the instance table from the DB schema script registry, eliminating index maintenance cost that reportedly doesn’t provide sufficient benefit.
Changes:
- Removed four
instance/*Index.sqlsnippet references fromschema.json. - Deleted the four corresponding index-creation SQL snippet files.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
mod-inventory-storage-server/src/main/resources/templates/db_scripts/schema.json |
Removes the four index snippet entries so they are no longer applied during schema setup/migration. |
mod-inventory-storage-server/src/main/resources/templates/db_scripts/instance/createInstanceNotesStaffOnlyIndex.sql |
Deletes the index creation snippet for notes staffOnly. |
mod-inventory-storage-server/src/main/resources/templates/db_scripts/instance/createInstanceNotesNoteIndex.sql |
Deletes the index creation snippet for notes note. |
mod-inventory-storage-server/src/main/resources/templates/db_scripts/instance/createInstanceClassificationNumberIndex.sql |
Deletes the index creation snippet for classifications.classificationNumber. |
mod-inventory-storage-server/src/main/resources/templates/db_scripts/instance/createInstanceAdministrativeNotesIndex.sql |
Deletes the index creation snippet for administrativeNotes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mod-inventory-storage-server/src/main/resources/templates/db_scripts/schema.json
Show resolved
Hide resolved
ncovercash
approved these changes
Mar 31, 2026
|
psmagin
approved these changes
Mar 31, 2026
Contributor
Author
|
This was merged (via the GitHub UI, even). I'm not sure why GH is claiming it wasn't... |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Purpose
This PR is to remove 4 indexes that don't provide sufficient benefit for their cost. These were recently added and have never been included in any release
Approach
The approach here is to simply remove the creation scripts. Since the indexes were never in any released versions, I did not bother adding scripts to remove the indexes.
Changes Checklist
Related Issues
MODFQMMGR-1097