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

libuv-async #2608

Closed
wants to merge 1 commit into from
Closed

libuv-async #2608

wants to merge 1 commit into from

Conversation

AEsir777
Copy link
Contributor

@AEsir777 AEsir777 commented Dec 22, 2023

  • possible improvement: thread pool size
  • buffer for request: manage life cycle (think its bugged in this draft PR)
  • uv_run use NO_WAIT but need to make sure the number of returned requests is exactly corresponding to one node group

test failed due to:
nodejs/node#50717

@AEsir777 AEsir777 changed the title libuv-sync libuv-async Dec 22, 2023
@AEsir777 AEsir777 force-pushed the libuv-async branch 4 times, most recently from 02e9b2e to 32cd844 Compare December 22, 2023 21:35
@ray6080
Copy link
Contributor

ray6080 commented Jan 29, 2024

Close for now. See #2729

@ray6080 ray6080 closed this Jan 29, 2024
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