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

All callers of PlanExecutor::getNext need to deal with error returns

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: 2.6.0-rc1
    • Fix Version/s: 3.3.2
    • Component/s: Querying
    • Labels:
    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Sprint:
      Query F (02/01/16), Query 10 (02/22/16)

      Description

      Some callers merely warn to the log if the Runner reports errors while others don't even check for errors. Both of these will lead to incorrect results being returned to the user rather than reporting an error. Searching the code for PlanExecutor::ADVANCED should show all loops that iterate a runner.

      Incomplete list of things that ignore errors:

      • delete (will report success even though some matching docs won't be deleted)
      • dbhash (will return the incorrect hash)
      • count
      • distinct (will return an incomplete list of distinct values, with no indication of incompleteness)
      • findAndModify (can upsert even if something already matches)
      • geoNear
      • mr
      • validate

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: