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

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

    XMLWordPrintableJSON

Details

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major - P3 Major - P3
    • 2.4.0-rc0
    • 2.3.1
    • JavaScript, MapReduce
    • None
    • ALL

    Description

      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.

      Attachments

        Activity

          People

            benjamin.becker Ben Becker
            benjamin.becker Ben Becker
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: