Skip to content

Conversation

epipav
Copy link
Collaborator

@epipav epipav commented Oct 25, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 740fc9e

Modified the database initialization and model creation functions to allow a custom query timeout value. This enables more flexibility and control over the database queries and avoids errors or interruptions for long-running queries.

🤖 Generated by Copilot at 740fc9e

databaseInit
Customize query timeout
Autumn leaves falling

Why

How

🤖 Generated by Copilot at 740fc9e

  • Add queryTimeoutMilliseconds parameter to databaseInit and models functions to customize query timeout value (link, link, link)
  • Use queryTimeoutMilliseconds parameter to configure query_timeout option of pg library in models function (link)
  • Modify backend/src/database/databaseConnection.ts and backend/src/database/models/index.ts files to implement and pass the new parameter (link, link, link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@epipav epipav marked this pull request as ready for review October 25, 2023 13:06
@epipav epipav requested a review from sausage-todd October 25, 2023 13:06
@epipav epipav self-assigned this Oct 25, 2023
@epipav epipav added the Improvement Created by Linear-GitHub Sync label Oct 25, 2023
@epipav epipav merged commit ca866cf into main Oct 25, 2023
@epipav epipav deleted the improvement/increase-sequelize-query-timeout branch October 25, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants