SERVER-4328 wrong lock for commitNow was messing up sharding with the new dblocking

Author: dwight

src/mongo/s/d_migrate.cpp

@@ -1571,7 +1571,7 @@ namespace mongo {
             log() << "migrate commit succeeded flushing to secondaries for '" << ns << "' " << min << " -> " << max << migrateLog;
 
             {
-                readlock lk(ns);  // commitNow() currently requires it
+                Lock::GlobalRead lk;
 
                 // if durability is on, force a write to journal
                 if ( getDur().commitNow() ) {

src/mongo/util/assert_util.cpp

@@ -179,7 +179,7 @@ namespace mongo {
         assertionCount.condrollover( ++assertionCount.warning );
         tlog() << "Assertion: " << msgid << ":" << msg << endl;
         raiseError(msgid,msg && *msg ? msg : "massert failure");
-        breakpoint();
+        //breakpoint();
         printStackTrace();
         throw MsgAssertionException(msgid, msg);
     }