Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-5932 BulkWriteResult show incorrect insertedIds on certain errors when using bulk insert (insertMany)
  3. NODE-5965

6.x: Fix BulkWriteResult.insertedIds to reflect actual inserted docs in insertMany

XMLWordPrintableJSON

    • Type: Icon: Sub-task Sub-task
    • Resolution: Unresolved
    • Priority: Icon: Unknown Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • 0
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      6.x branch fix 

      Unknowns

      • How do we know which ids were inserted or not if the error is not specific to an index? (Top-Level error), does that indicate all inserts failed?

      Acceptance Criteria

      Implementation Requirements

      • Make BulkWriteResult.insertedIds reflect actual inserted docs in insertMany, even when no items are inserted / invalid objects are passed in

      Testing Requirements

      • Ensure user test passes for insertMany and bulkWrite
      • When top level error (in resultHandler function) occurs for bulkWrite, handle insertedIds appropriately 

            Assignee:
            Unassigned Unassigned
            Reporter:
            aditi.khare@mongodb.com Aditi Khare
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: