ServiceContext::unsetKillAllOperations should be marked as _forTest or removed

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Server Programmability
    • Programmability 2025-04-14, Programmability 2025-04-28, Programmability 2025-05-12, Programmability 2025-05-26, Programmability 2025-06-09, Programmability 2025-06-23
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      We assume that the global kill switch is a one-way toggle in production code. If it can be flipped back off, that unleashes a bunch of new race conditions that we shouldn't need to think about. Luckily it is only used from test code. At the very least we should mark it as test only to ensure it doesn't get used in production code. It may also be worth checking if the tests can easily be modified to not require that function at all, but it probably isn't worth putting too much time into that if it proves tricky.

              Assignee:
              Richy Gerard (Inactive)
              Reporter:
              Mathias Stearn
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated: