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

DMP-4478: (Dev Only) Add Additional index's to the data base #2428

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Ben-Edwards-cgi
Copy link
Contributor

Links

Jira

Change description

Summary of Git Diff

This diff introduces a new SQL migration file that contains several index creation statements for the PostgreSQL database and updates to the user_account table, specifically modifying the user_name based on specific user_email_address values.

Highlights

  • New SQL File Created: V1_424_additional_indxes.sql

  • Index Drops and Creations:

    • Drops the index event_event_id_is_current_idx concurrently.
    • Creates multiple indices concurrently on various tables:
      • request_status_mer_id_idx on media_request
      • created_ts_med_id_idx on media
      • event_status_idx on event
      • hearing_date_ctr_id_idx on hearing
      • cas_id_mlc_id_idx on media_linked_case
      • med_id_idx on media_linked_case
      • source_idx on media_linked_case
      • court_case_created_ts_idx on court_case
  • User Account Updates:

    • Updates user_name for user_email_address systemDETSCleanupArmResponseFilesAutomatedTask@hmcts.net to system_DETSCleanupArmResponseFiles.
    • Updates user_name for user_email_address systemArmMissingResponseReplayAutomatedTask@hmcts.net to system_ArmMissingResponseReplay.

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[X] No

@jackmaloney
Copy link
Contributor

Need to wait for migration team to give the go ahead before merging this one.

Copy link
Contributor

@chrisbellingham-hmcts chrisbellingham-hmcts left a comment

Choose a reason for hiding this comment

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

You'll need an additional underscore in the filename V1_424__, otherwise the migration will be ignored, and there's a small typo in indxes. Also you'll need to bump the version number upon rebase as some scripts have been added since this PR was raised.

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