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

@gpean is the need to schedule work on a separate thread for an existing module or something new ? #1597

Closed
damnboy opened this issue Nov 6, 2018 · 5 comments

Comments

@damnboy
Copy link

damnboy commented Nov 6, 2018

Originally posted by @mhdawson in nodejs/node#13512 (comment)

@damnboy
Copy link
Author

damnboy commented Nov 6, 2018

I am trying to move massca into nodejs env(as an addon),
in the main function of this project, it creates serveral threads waiting for the packet coming back which uses libcap). These threads never get close until the scan task is done,so these threads seems not fit into the Worker Poll.
As the guide book on the nodejs office site, you should never block the Event Loop and the Work Pool.
But the implementation of napi_create_async_work only corporate with the Worker Pool, not create a new thread.
Is there any easy way more than use the libuv APIs directly for thread creating?
Thanks.

@Trott
Copy link
Member

Trott commented Nov 6, 2018

@nodejs/workers @nodejs/n-api

(Should this be moved to the help repo?)

@Trott
Copy link
Member

Trott commented Nov 8, 2018

I'm going to move this to the help repo in the hopes that it attracts attention there that it is not getting here. If that's not the right repo, feel free to comment (or if you're an org owner, move it back yourself if you'd like!).

@Trott Trott transferred this issue from nodejs/node Nov 8, 2018
@gireeshpunathil
Copy link
Member

@damnboy - is this still pending?

@gireeshpunathil
Copy link
Member

inactive, closing

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

No branches or pull requests

3 participants