Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-2590

Reimplement AsyncIterator for Cursor optionally using a provided promise implementation

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0.0
    • Component/s: None
    • Labels:
      None

      Description

      A recent change removed our custom implementation for AsyncIterator on the cursor type. While this fixes a bug by bringing our cursor in line with the Readable async iterator implementation, it also drops support for using a custom promise library passed into the driver. We should reimplement this, and use the solution Thomas is developing in his "global promise" feature.

        Attachments

          Issue Links

          depends on

          Task - A task that needs to be done. NODE-2820 Create a new CursorStream type, remove Readable base class from Cursor

          • Major - P3 - Major loss of function.
          • Closed
          is duplicated by

          Task - A task that needs to be done. NODE-2321 Rewrite lib/async/async_iterator.js as an async generator

          • Major - P3 - Major loss of function.
          • Closed

            Activity

              People

              Assignee:
              thomas.reggi Thomas Reggi (Inactive)
              Reporter:
              matt.broadstone Matt Broadstone
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: