Skip to content

#72979 - Adds retry logic to database connection attempts - Tested With Rohan#147

Open
tushar-singh-spraxa wants to merge 3 commits intomainfrom
dev/tushar/connection_retry_logic
Open

#72979 - Adds retry logic to database connection attempts - Tested With Rohan#147
tushar-singh-spraxa wants to merge 3 commits intomainfrom
dev/tushar/connection_retry_logic

Conversation

@tushar-singh-spraxa
Copy link
Collaborator

Improves connection reliability by introducing configurable retry mechanism with logging for connection failures. Helps handle intermittent network issues and reduces startup failures.

…th Rohan

Improves connection reliability by introducing configurable retry
mechanism with logging for connection failures. Helps handle
intermittent network issues and reduces startup failures.
Increments the package version to prepare for a new release or to reflect recent updates. No functional or dependency changes included.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds configurable retry logic for MSSQL pool connection initialization to reduce transient connection failures during startup, and bumps the package version.

Changes:

  • Add retry loop with warn/error logging around mssql.ConnectionPool(...).connect().
  • Make retry behavior configurable via maxRetries and retryDelayMs.
  • Bump package version to 1.0.65.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
package.json Version bump for the release containing connection retry behavior.
lib/sql.js Implements MSSQL connection retry logic with configurable retry count/delay and logging.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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.

2 participants