Skip to content

Feat/nm db retry logic#15

Merged
sshenzha merged 4 commits intomainfrom
feat/nm-db-retry-logic
Sep 25, 2025
Merged

Feat/nm db retry logic#15
sshenzha merged 4 commits intomainfrom
feat/nm-db-retry-logic

Conversation

@sshenzha
Copy link
Copy Markdown
Collaborator

@sshenzha sshenzha commented Sep 25, 2025

Key features added (db.py):

  1. Thread-safe connection handling for concurrent access
  2. Retry logic
  3. Connection recovery on failures
  4. Logging for connection attempts and failures

Key features added (main.py):

  1. Logging system
  2. Proper database connection cleanup on shutdown
  3. Better error messages and status reporting

Key features added (sample.env)

  1. Separated database-specific configuration
  2. Provided complete configuration examples for both database types

@sshenzha sshenzha merged commit 864e9b1 into main Sep 25, 2025
1 check passed
@sshenzha sshenzha deleted the feat/nm-db-retry-logic branch September 25, 2025 17:02
Copy link
Copy Markdown
Contributor

@vvcb vvcb left a comment

Choose a reason for hiding this comment

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

@shen1802 this looks good. Can you add a description in the PR about which github issue this solves and how?

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