Skip to content

discuss: remove domain module #45824

Open
@bnoordhuis

Description

@bnoordhuis
  • require('domain') was deprecated 7.5 years ago

  • modern code uses AbortController

  • bug tracker activity suggests it's not in active use (or it works so well people never hit bugs0 - unlikely!)

  • the supporting infrastructure for domain is a kind of technical debt that also has performance implications1; it's not as bad as it used to be but it's still there

Proposal:

  • remove it for good in the next major release

  • leave a stub lib/domain.js that prints a warning, a la the sys module.


0 the prime reason for deprecation was because it was so buggy due to part poor design, part poor implementation

1 at one time it was so bad that if even one module imported domain, whole-program performance tanked

Metadata

Metadata

Assignees

No one assigned

    Labels

    discussIssues opened for discussions and feedbacks.domainIssues and PRs related to the domain subsystem.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions