timeoutMs was never implemented in V8Scope's invoke() or exec()

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Done
    • Priority: Major - P3
    • 2.4.0-rc0
    • Affects Version/s: 2.3.1
    • Component/s: JavaScript, MapReduce
    • None
    • ALL
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      Noticed after changing the following line to an massert():

      https://github.com/mongodb/mongo/blob/master/src/mongo/scripting/engine_v8.cpp#L890

      Needed for:

      • group() specifies a 100ms timeout when executing a trivial 'assign to global' script. I think this timeout can be safely removed. See group.cpp, lines 83, 84, 154 and 169.
      • $where specifies a 60 second timeout per document. See matcher.cpp L115 and L1173.
      • dbEval() also uses cmdLine.quota to determine if it should limit db.eval() times to 10 minutes. See dbeval.cpp L86.
      • benchRun() uses the timeout in a few places. See bench.cpp, L381, L401, L448, L480, L509, L542.

            Assignee:
            Ben Becker (Inactive)
            Reporter:
            Ben Becker (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: