Replace process.nextTick() with .queueMicrotask()

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Unresolved
    • Priority: Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • 5
    • Not Needed
    • None
    • Hide

      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.

      Show
      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.
    • Not Needed
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      As a... bundling user of the Node driver
      I want... node-specific APIs (process.nextTick) to be replaced with non-node specific ones (queueMicrotask)
      So that... bundling is easier

      User Experience

      • process.nextTick isn't used in the driver

      Dependencies

      • N/A

      Risks/Unknowns

      • We have to be very careful with timing - for each instance of usage, consider whether moving the execution in the event loop will make a difference

      Acceptance Criteria

      Implementation Requirements

      • Review Node.js event loop fundamentals (specifically relative to nextTick and queueMicrotask)
      • Replace process.nextTick() with queueMicrotask() (or another alternative)
      • Ask Anna for review

      Testing Requirements

      • Existing tests pass

      Documentation Requirements

      • Release note

      Follow Up Requirements

      • Regression testing is deferred to a consolidated follow up ticket for M0

            Assignee:
            Pavel Safronov
            Reporter:
            Sergey Zelenov
            None
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: