Uploaded image for project: 'Compass '
  1. Compass
  2. COMPASS-7023

Investigate changes in NODE-5186: Remove duplicate BulkWriteResult accessors

    • Type: Icon: Investigation Investigation
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • No version
    • Affects Version/s: None
    • Component/s: None
    • Labels:
    • Needed
    • Hide

      The documentation in https://www.mongodb.com/docs/manual/reference/method/Bulk.execute/ needs to account for the fact that the BulkWriteResult returned from mongosh does not match the actual result:

      • upserted is not present on bulk write results in mongosh (neither 1.x nor 2.x)
      • upsertedIds used to be an array in 1.x, is now a map from document index to ObjectId in mongosh 2.x
      • insertedIds follows the same format as upsertedIds now does (in both 1.x and 2.x)
      Show
      The documentation in https://www.mongodb.com/docs/manual/reference/method/Bulk.execute/ needs to account for the fact that the BulkWriteResult returned from mongosh does not match the actual result: upserted is not present on bulk write results in mongosh (neither 1.x nor 2.x) upsertedIds used to be an array in 1.x, is now a map from document index to ObjectId in mongosh 2.x insertedIds follows the same format as upsertedIds now does (in both 1.x and 2.x)
    • Iteration Iguanodon

      This ticket is a follow up for NODE-4791 to remove BulkWriteResult getters that were deprecated in NODE V5/V6

      AC

      • Remove BulkWriteResult#nInserted
      • Remove BulkWriteResult#nUpserted
      • Remove BulkWriteResult#nMatched
      • Remove BulkWriteResult#nModified
      • Remove BulkWriteResult#nRemoved
      • Remove BulkWriteResult#getInsertedIds
      • Remove BulkWriteResult#getUpsertedIds
      • Add defaultValue documentation to BulkOptions

            Assignee:
            anna.henningsen@mongodb.com Anna Henningsen
            Reporter:
            dbeng-pm-bot PM Bot
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: