Skip to content

TSFN callback queue behavior clarification #48356

Closed
Closed
TSFN callback queue behavior clarification#48356
Labels
docIssues and PRs related to Node.js documentation.node-apiIssues and PRs related to Node-API.

Description

@ekraihan
ekraihan
opened on Jun 6, 2023

Affected URL(s)

https://nodejs.org/api/n-api.html#asynchronous-thread-safe-function-calls

Description of the problem

Does a TSFN call only one callback from its queue per iteration of the event loop or does it call all callbacks in the queue in one iteration? Or maybe there is there some other policy?

Perhaps this detail could be clarified in the docs?

Thanks!

Activity

  1. ekraihan
    added
    docIssues and PRs related to Node.js documentation.
    on Jun 6, 2023

  2. mhdawson commented on Jun 6, 2023

    @mhdawson
    mhdawson
    on Jun 6, 2023
    Member

    Taking a look, will add something to the doc.

  3. mhdawson
    added a commit that references this issue on Jun 6, 2023

    doc: add additional info on TSFN dispatch

    08bd4b4

  4. mhdawson commented on Jun 6, 2023

    @mhdawson
    mhdawson
    on Jun 6, 2023
    Member

    @ekraihan added a proposed doc addition in #48367. it shows the code in terms of what we implement today, but the doc addition better captures what applications should depend on.

  5. ekraihan commented on Jun 7, 2023

    @ekraihan
    ekraihan
    on Jun 7, 2023
    Author

    @mhdawson Awesome, thanks for the clarification

  6. legendecas
    added
    node-apiIssues and PRs related to Node-API.
    on Jun 7, 2023
  7. mhdawson
    added a commit that references this issue on Jun 12, 2023

    doc: add additional info on TSFN dispatch

    1da9099

  8. gabrielschulhof commented on Jun 12, 2023

    @gabrielschulhof
    gabrielschulhof
    on Jun 12, 2023
    Contributor

    Closed in #48367.

  9. gabrielschulhof
    closed this as completedon Jun 12, 2023
  10. RafaelGSS
    added a commit that references this issue on Jul 3, 2023

    doc: add additional info on TSFN dispatch

    b9c643e
  11. Ceres6
    added 2 commits that reference this issue on Aug 14, 2023

    doc: add additional info on TSFN dispatch

    ae3243b

    doc: add additional info on TSFN dispatch

    4f242d3
  12. ruyadorno
    added 2 commits that reference this issue on Sep 7, 2023

    doc: add additional info on TSFN dispatch

    a8db331

    doc: add additional info on TSFN dispatch

    2a561ae
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    docIssues and PRs related to Node.js documentation.node-apiIssues and PRs related to Node-API.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions