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

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Done
    • Priority: Major - P3
    • 3.3.2
    • Affects Version/s: 2.6.0-rc1
    • Component/s: Querying
    • Fully Compatible
    • ALL
    • Query F (02/01/16), Query 10 (02/22/16)
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      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

            Assignee:
            Tess Avitabile (Inactive)
            Reporter:
            Mathias Stearn
            Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved: