Log "dbexit: really exiting now" after all locks needed to shutdown are acquired

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Done
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Internal Code
    • None
    • Storage Execution
    • ALL
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

          void exitCleanly( ExitCode code ) {
      ...
              log() << "dbexit: really exiting now";
              if ( c ) c->shutdown();
              ::_exit(rc);
          }
      

      c->shutdown requires locking a mutex. It is possible the process is not really exiting now after logging "dbexit: really exiting now". This line should only be printed when exit is guaranteed to be imminent. I suggest moving to after the if statement.

            Assignee:
            [DO NOT USE] Backlog - Storage Execution Team
            Reporter:
            Charlie Page (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: