Skip to content

Conversation

@PavelSafronov
Copy link
Contributor

@PavelSafronov PavelSafronov commented Dec 3, 2025

Description

Summary of Changes

Replace process.nextTick() with .queueMicrotask()

What is the motivation for this change?

This helps us reduce our runtime dependencies, as part of https://jira.mongodb.org/browse/NODE-6601

Release Highlight

Replaced Node‑specific timing/scheduling API process.nextTick with standards‑based equivalent

This reduces the number of patches needed to run the driver outside Node.js.

Double check the following

  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Lint is passing (npm run check:lint)
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@PavelSafronov PavelSafronov reopened this Dec 3, 2025
addaleax
addaleax previously approved these changes Dec 3, 2025
@PavelSafronov PavelSafronov marked this pull request as ready for review December 5, 2025 19:59
@PavelSafronov PavelSafronov requested a review from a team as a code owner December 5, 2025 19:59
@tadjik1 tadjik1 added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Dec 8, 2025
@tadjik1 tadjik1 self-assigned this Dec 8, 2025
@tadjik1 tadjik1 requested a review from a team December 8, 2025 10:30
@tadjik1 tadjik1 added the Team Review Needs review from team label Dec 9, 2025
Copy link
Member

@tadjik1 tadjik1 left a comment

Choose a reason for hiding this comment

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

Great work @PavelSafronov, thanks for the investigation around client sessions and queue processing!

@tadjik1 tadjik1 merged commit b1b6e81 into mongodb:main Dec 12, 2025
29 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Primary Review In Review with primary reviewer, not yet ready for team's eyes Team Review Needs review from team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants