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

Fix: async fs operations #676

Merged
merged 2 commits into from
Feb 4, 2025
Merged

Conversation

zone117x
Copy link
Member

Replaces the sync node:fs operations with async node:fs/promises async operations. This fixes potential issues with the main program loop completely blocking while waiting on disk operations. E.g. sync disk operations causing lots of small pauses/delays while serving http requests.

@zone117x zone117x requested a review from rafaelcr October 31, 2024 15:50
@zone117x zone117x changed the base branch from stacks-signers to feat/reregister-callback October 31, 2024 15:50
Copy link
Collaborator

@rafaelcr rafaelcr left a comment

Choose a reason for hiding this comment

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

Great!

@zone117x zone117x had a problem deploying to Development-mainnet October 31, 2024 16:32 — with GitHub Actions Failure
@zone117x zone117x had a problem deploying to Development-testnet October 31, 2024 16:32 — with GitHub Actions Failure
@rafaelcr rafaelcr merged commit 7de9ea0 into feat/reregister-callback Feb 4, 2025
13 of 15 checks passed
@rafaelcr rafaelcr deleted the fix/async-fs-ops branch February 4, 2025 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants