Skip to content

Conversation

@mschfh
Copy link
Collaborator

@mschfh mschfh commented Dec 3, 2025

Summary

Improve order lookup performance by adding indexes for externalID and radomSubscriptionId metadata fields.

Type of Change

  • Product feature
  • Bug fix
  • Performance improvement
  • Refactor
  • Other

Tested Environments

  • Development
  • Staging
  • Production

Before Requesting Review

  • Does your code build cleanly without any errors or warnings?
  • Have you used auto closing keywords?
  • Have you added tests for new functionality?
  • Have validated query efficiency for new database queries?
  • Have documented new functionality in README or in comments?
  • Have you squashed all intermediate commits?
  • Is there a clear title that explains what the PR does?
  • Have you used intuitive function, variable and other naming?
  • Have you requested security and/or privacy review if needed
  • Have you performed a self review of this PR?

@mschfh mschfh self-assigned this Dec 3, 2025
@mschfh mschfh marked this pull request as ready for review December 3, 2025 08:19
@mschfh mschfh requested a review from clD11 as a code owner December 3, 2025 08:19
@clD11 clD11 self-requested a review December 4, 2025 19:16
@clD11
Copy link
Contributor

clD11 commented Dec 4, 2025

@mschfh you need to update the migration version here to match the newly added migration files or the migration wont run.

Also, I assume you will also run the migrations manually prior to deployment particularly for prod. For existing tables the migration/deployment will fail due to a timeout if it takes a long time to complete. For lower environments this may not be an issue but for prod this will certainly need to be run manually first.

@mschfh mschfh force-pushed the mschfh-order-metadata-idx branch from 790b064 to 41fd4dc Compare December 4, 2025 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants