SERVER-17817 Make ServiceContext create and manage Client objects.

Also, deduplicate Client::* method implementations, guard the identity of the
current CurOp of a Client with the Client's _mutex instead of the mutex guarding
the list of all clients.

Makes the currentClient object private to client.cpp, and all access to the
thread-bound client is now done with haveClient() and cc() free functions in the
mongo namespace.

Removes the vesitgal Client::shutdown() methods.

1 files changed, 0 insertions, 7 deletions

diff --git a/src/mongo/dbtests/threadedtests.cpp b/src/mongo/dbtests/threadedtests.cpp
index 4d959ace021..ae94c32fb1d 100644
--- a/src/mongo/dbtests/threadedtests.cpp
+++ b/src/mongo/dbtests/threadedtests.cpp
@@ -246,7 +246,6 @@ namespace ThreadedTests {
                 }
                 pm.hit();
             }
-            cc().shutdown();
         }
 
         virtual void validate() {
@@ -568,8 +567,6 @@ namespace ThreadedTests {
             default:
                 ASSERT(false);
             }
-
-            cc().shutdown();
         }
     };
 
@@ -736,7 +733,6 @@ namespace ThreadedTests {
                 LOG(Z) << t.millis() << endl;
                 ASSERT( t.millis() > 50 );
             }
-            cc().shutdown();
         }
     };
 
@@ -800,9 +796,6 @@ namespace ThreadedTests {
                     mongo::unittest::log() << "checked in " << i << " times..." << endl;
 
             }
-
-            cc().shutdown();
-
         }
 
         virtual void validate() {