Change release_memory jstests to use try/finally so memory knobs always get reset

ExportXMLWordPrintableJSON

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Execution
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      The following tests change values for server parameters, but don't wrap their assertions in try/finally. This means if the test fails, other tests using the same fixture could run with the knobs changed. We should use try/finally to make sure the values are reset at the end of the test

      jstests/core/query/release_memory/group.js
      jstests/core/query/release_memory/hash_lookup_unwind.js
      jstests/core/query/release_memory/hash_lookup.js

            Assignee:
            Matt Boros
            Reporter:
            Matt Boros
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: