Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-13865

Shell returns incorrect WriteResult for compatibility-mode upsert with non-OID equality pred on _id field

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major - P3
    • Resolution: Fixed
    • 2.6.1, 2.7.0
    • 2.6.2, 2.7.1
    • Shell
    • None
    • ALL

    Description

      Issue Status as of Jun 9, 2014

      ISSUE SUMMARY
      When using a 2.6 shell against a 2.4 server, or during the upgrade period where a 2.6 mongos talks to a 2.4 mongod server, the shell returns an incorrect WriteResult for upsert operations where the query predicate includes an equality match on _id with a non-ObjectID type. For example:

      db.foo.update({_id:0},{$set:{a:1}},{upsert:true})

      "When using a 2.6 shell against a 2.4 server, or during the upgrade period where a 2.6 mongos talks to a 2.4 mongod server..."
      Note that non-ObjectID upsert _ids were not reported in v2.4, the issue is the incorrect upsert statistics.

      Resolution Details:

      The WriteResult returned is:

      WriteResult({ "nMatched" : 1, "nUpserted" : 0 })

      The correct WriteResult should be one of the following:

        
      WriteResult({ "nMatched" : 0, "nUpserted" : 1 })
      WriteResult({ "nMatched" : 0, "nUpserted" : 1, "_id" : <value> })

      USER IMPACT
      The upsert statistics in WriteResult messages returned by the shell are incorrect.

      WORKAROUNDS
      getLastError should report the correct upsert statistics.

      AFFECTED VERSIONS
      MongoDB 2.6.0 and 2.6.1 are affected by this issue.

      FIX VERSION
      The fix is included in the 2.6.2 production release.

      RESOLUTION DETAILS
      Do not rely on the presence of the upserted field in the getLastError response to populate write results.

      Original description

      In compatibility mode, the shell returns an incorrect WriteResult for upserts where the query predicate includes an equality match on _id with a non-OID type.

      The WriteResult returned is:

      WriteResult({ "nMatched" : 1, "nUpserted" : 0 })

      The correct WriteResult should be one of the following:

      WriteResult({ "nMatched" : 0, "nUpserted" : 1 })
      WriteResult({ "nMatched" : 0, "nUpserted" : 1, "_id" : <value> })

      Reproduce with:

      MongoDB shell version: 2.6.1
      connecting to: test
      > db.version()
      2.4.10
      > db.foo.remove({})
      Cannot use commands write mode, degrading to compatability mode
      WriteResult({ "nRemoved" : 0 })
      > db.foo.update({_id:ObjectId("ffffffffffffffffffffffff")},{$set:{a:1}},{upsert:true})
      // OID type, correct result
      WriteResult({
      	"nMatched" : 0,
      	"nUpserted" : 1,
      	"_id" : ObjectId("ffffffffffffffffffffffff")
      })
      > db.foo.update({_id:0},{$set:{a:1}},{upsert:true})
      // non-OID type, incorrect result
      WriteResult({ "nMatched" : 1, "nUpserted" : 0 })

      Attachments

        Issue Links

          Activity

            People

              greg_10gen Greg Studer
              rassi J Rassi
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: