more cleaning

SConstruct

@@ -352,7 +352,7 @@ if has_option( "asio" ):
     coreServerFiles += [ "util/message_server_asio.cpp" ]
 
 # mongod files - also files used in tools. present in dbtests, but not in mongos and not in client libs.
-serverOnlyFiles = Split( "db/key.cpp db/btreebuilder.cpp util/logfile.cpp util/alignedbuilder.cpp db/mongommf.cpp db/dur.cpp db/durop.cpp db/dur_writetodatafiles.cpp db/dur_preplogbuffer.cpp db/dur_commitjob.cpp db/dur_recover.cpp db/dur_journal.cpp db/query.cpp db/introspect.cpp db/btree.cpp db/clientcursor.cpp db/tests.cpp db/repl.cpp db/repl/rs.cpp db/repl/consensus.cpp db/repl/rs_initiate.cpp db/repl/replset_commands.cpp db/repl/manager.cpp db/repl/health.cpp db/repl/heartbeat.cpp db/repl/rs_config.cpp db/repl/rs_rollback.cpp db/repl/rs_sync.cpp db/repl/rs_initialsync.cpp db/oplog.cpp db/repl_block.cpp db/btreecursor.cpp db/cloner.cpp db/namespace.cpp db/cap.cpp db/matcher_covered.cpp db/dbeval.cpp db/restapi.cpp db/dbhelpers.cpp db/instance.cpp db/client.cpp db/database.cpp db/pdfile.cpp db/record.cpp db/cursor.cpp db/security_commands.cpp db/security.cpp db/queryoptimizer.cpp db/queryoptimizercursor.cpp db/extsort.cpp db/cmdline.cpp" )
+serverOnlyFiles = Split( "db/key.cpp db/btreebuilder.cpp util/logfile.cpp util/alignedbuilder.cpp db/mongommf.cpp db/dur.cpp db/durop.cpp db/dur_writetodatafiles.cpp db/dur_preplogbuffer.cpp db/dur_commitjob.cpp db/dur_recover.cpp db/dur_journal.cpp db/introspect.cpp db/btree.cpp db/clientcursor.cpp db/tests.cpp db/repl.cpp db/repl/rs.cpp db/repl/consensus.cpp db/repl/rs_initiate.cpp db/repl/replset_commands.cpp db/repl/manager.cpp db/repl/health.cpp db/repl/heartbeat.cpp db/repl/rs_config.cpp db/repl/rs_rollback.cpp db/repl/rs_sync.cpp db/repl/rs_initialsync.cpp db/oplog.cpp db/repl_block.cpp db/btreecursor.cpp db/cloner.cpp db/namespace.cpp db/cap.cpp db/matcher_covered.cpp db/dbeval.cpp db/restapi.cpp db/dbhelpers.cpp db/instance.cpp db/client.cpp db/database.cpp db/pdfile.cpp db/record.cpp db/cursor.cpp db/security_commands.cpp db/security.cpp db/queryoptimizer.cpp db/queryoptimizercursor.cpp db/extsort.cpp db/cmdline.cpp" )
 
 serverOnlyFiles += [ "db/index.cpp" ] + Glob( "db/geo/*.cpp" ) + Glob( "db/ops/*.cpp" )
 

db/cap.cpp

@@ -26,8 +26,8 @@
 #include "btree.h"
 #include <algorithm>
 #include <list>
-#include "query.h"
 #include "json.h"
+#include "clientcursor.h"
 
 /*
  capped collection layout

db/clientcursor.cpp

@@ -23,7 +23,7 @@
 */
 
 #include "pch.h"
-#include "query.h"
+#include "clientcursor.h"
 #include "introspect.h"
 #include <time.h>
 #include "db.h"

db/cloner.cpp

@@ -22,7 +22,7 @@
 #include "../client/dbclient.h"
 #include "../bson/util/builder.h"
 #include "jsobj.h"
-#include "query.h"
+#include "ops/query.h"
 #include "commands.h"
 #include "db.h"
 #include "instance.h"

db/db.cpp

@@ -18,7 +18,6 @@
 
 #include "pch.h"
 #include "db.h"
-#include "query.h"
 #include "introspect.h"
 #include "repl.h"
 #include "../util/unittest.h"

db/dbcommands.cpp

@@ -21,7 +21,7 @@
 */
 
 #include "pch.h"
-#include "query.h"
+#include "ops/query.h"
 #include "pdfile.h"
 #include "jsobj.h"
 #include "../bson/util/builder.h"

db/dbcommands_generic.cpp

@@ -20,7 +20,7 @@
  */
 
 #include "pch.h"
-#include "query.h"
+#include "ops/query.h"
 #include "pdfile.h"
 #include "jsobj.h"
 #include "../bson/util/builder.h"

db/dbeval.cpp

@@ -18,7 +18,7 @@
 */
 
 #include "pch.h"
-#include "query.h"
+#include "ops/query.h"
 #include "pdfile.h"
 #include "jsobj.h"
 #include "../bson/util/builder.h"

db/dbhelpers.cpp

@@ -19,7 +19,6 @@
 #include "pch.h"
 #include "db.h"
 #include "dbhelpers.h"
-#include "query.h"
 #include "json.h"
 #include "queryoptimizer.h"
 #include "btree.h"

db/indexkey.cpp

@@ -20,7 +20,7 @@
 #include "namespace-inl.h"
 #include "index.h"
 #include "btree.h"
-#include "query.h"
+#include "ops/query.h"
 #include "background.h"
 
 namespace mongo {

db/instance.cpp

@@ -19,7 +19,6 @@
 
 #include "pch.h"
 #include "db.h"
-#include "query.h"
 #include "introspect.h"
 #include "repl.h"
 #include "dbmessage.h"
@@ -41,6 +40,7 @@
 #include "dur_recover.h"
 #include "ops/update.h"
 #include "ops/delete.h"
+#include "ops/query.h"
 
 namespace mongo {
 

db/namespace.cpp

@@ -25,10 +25,11 @@
 #include "btree.h"
 #include <algorithm>
 #include <list>
-#include "query.h"
 #include "queryutil.h"
 #include "json.h"
 #include "ops/delete.h"
+#include "ops/query.h"
+
 
 namespace mongo {
 

db/oplog.cpp

@@ -28,6 +28,7 @@
 #include "queryoptimizer.h"
 #include "ops/update.h"
 #include "ops/delete.h"
+#include "ops/query.h"
 
 namespace mongo {
 

db/query.cpp → db/ops/query.cpp

@@ -16,26 +16,26 @@
  *    along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "pch.h"
+#include "../../pch.h"
 #include "query.h"
-#include "pdfile.h"
-#include "jsobjmanipulator.h"
-#include "../bson/util/builder.h"
+#include "../pdfile.h"
+#include "../jsobjmanipulator.h"
+#include "../../bson/util/builder.h"
 #include <time.h>
-#include "introspect.h"
-#include "btree.h"
-#include "../util/lruishmap.h"
-#include "json.h"
-#include "repl.h"
-#include "replutil.h"
-#include "scanandorder.h"
-#include "security.h"
-#include "curop-inl.h"
-#include "commands.h"
-#include "queryoptimizer.h"
-#include "lasterror.h"
-#include "../s/d_logic.h"
-#include "repl_block.h"
+#include "../introspect.h"
+#include "../btree.h"
+#include "../../util/lruishmap.h"
+#include "../json.h"
+#include "../repl.h"
+#include "../replutil.h"
+#include "../scanandorder.h"
+#include "../security.h"
+#include "../curop-inl.h"
+#include "../commands.h"
+#include "../queryoptimizer.h"
+#include "../lasterror.h"
+#include "../../s/d_logic.h"
+#include "../repl_block.h"
 
 namespace mongo {
 

db/query.h → db/ops/query.h

@@ -18,15 +18,15 @@
 
 #pragma once
 
-#include "../pch.h"
-#include "../util/message.h"
-#include "dbmessage.h"
-#include "jsobj.h"
-#include "diskloc.h"
-#include "projection.h"
+#include "../../pch.h"
+#include "../../util/message.h"
+#include "../dbmessage.h"
+#include "../jsobj.h"
+#include "../diskloc.h"
+#include "../projection.h"
 
 // struct QueryOptions, QueryResult, QueryResultFlags in:
-#include "../client/dbclient.h"
+#include "../../client/dbclient.h"
 
 namespace mongo {
 
@@ -250,4 +250,4 @@ namespace mongo {
 
 } // namespace mongo
 
-#include "clientcursor.h"
+

db/ops/update.cpp

@@ -17,7 +17,7 @@
  */
 
 #include "pch.h"
-#include "../query.h"
+#include "query.h"
 #include "../pdfile.h"
 #include "../jsobjmanipulator.h"
 #include "../queryoptimizer.h"

db/ops/update.h

@@ -44,6 +44,7 @@ namespace mongo {
     };
 
 
+    class RemoveSaver;
 
     /* returns true if an existing object was updated, false if no existing object was found.
        multi - update multiple objects - mostly useful with things like $set

db/pdfile.cpp

@@ -44,6 +44,7 @@ _ disallow system* manipulations from the database.
 #include "background.h"
 #include "compact.h"
 #include "ops/delete.h"
+#include "instance.h"
 
 namespace mongo {
 

db/repl.cpp

@@ -40,7 +40,7 @@
 #include "../client/dbclient.h"
 #include "../client/connpool.h"
 #include "pdfile.h"
-#include "query.h"
+#include "ops/query.h"
 #include "db.h"
 #include "commands.h"
 #include "security.h"

db/repl.h

@@ -30,7 +30,6 @@
 #include "pdfile.h"
 #include "db.h"
 #include "dbhelpers.h"
-#include "query.h"
 #include "../client/dbclient.h"
 #include "../util/optime.h"
 #include "oplog.h"

db/repl/rs_rollback.cpp

@@ -20,7 +20,7 @@
 #include "../../client/dbclient.h"
 #include "rs.h"
 #include "../repl.h"
-#include "../query.h"
+#include "../ops/query.h"
 #include "../cloner.h"
 #include "../ops/update.h"
 #include "../ops/delete.h"

db/replutil.h

@@ -23,6 +23,7 @@
 #include "repl.h"
 #include "cmdline.h"
 #include "repl/rs.h"
+#include "ops/query.h"
 
 namespace mongo {
 

dbtests/directclienttests.cpp

@@ -18,7 +18,7 @@
  */
 
 #include "pch.h"
-#include "../db/query.h"
+#include "../db/ops/query.h"
 #include "../db/db.h"
 #include "../db/instance.h"
 #include "../db/json.h"

dbtests/perf/perftest.cpp

@@ -21,7 +21,7 @@
 
 #include "../../client/dbclient.h"
 #include "../../db/instance.h"
-#include "../../db/query.h"
+#include "../../db/ops/query.h"
 #include "../../db/queryoptimizer.h"
 #include "../../util/file_allocator.h"
 

dbtests/perftests.cpp

@@ -23,7 +23,7 @@
  */
 
 #include "pch.h"
-#include "../db/query.h"
+#include "../db/ops/query.h"
 #include "../db/db.h"
 #include "../db/instance.h"
 #include "../db/json.h"

dbtests/queryoptimizertests.cpp

@@ -21,7 +21,7 @@
 #include "../db/queryoptimizer.h"
 #include "../db/querypattern.h"
 #include "../db/instance.h"
-#include "../db/query.h"
+#include "../db/ops/query.h"
 #include "../db/ops/delete.h"
 #include "dbtests.h"
 

dbtests/querytests.cpp

@@ -18,7 +18,9 @@
  */
 
 #include "pch.h"
-#include "../db/query.h"
+#include "../db/ops/query.h"
+#include "../db/dbhelpers.h"
+#include "../db/clientcursor.h"
 
 #include "../db/instance.h"
 #include "../db/json.h"

dbtests/updatetests.cpp

@@ -18,7 +18,7 @@
  */
 
 #include "pch.h"
-#include "../db/query.h"
+#include "../db/ops/query.h"
 
 #include "../db/db.h"
 #include "../db/instance.h"

s/d_logic.cpp

@@ -29,7 +29,7 @@
 #include "../db/commands.h"
 #include "../db/jsobj.h"
 #include "../db/dbmessage.h"
-#include "../db/query.h"
+#include "../db/ops/query.h"
 
 #include "../client/connpool.h"
 

shell/mongo_vstudio.cpp

@@ -676,7 +676,9 @@ const StringData _jscode_raw_utils =
 "\"jstests/killop.js\",\n" 
 "\"jstests/run_program1.js\",\n" 
 "\"jstests/notablescan.js\",\n" 
-"\"jstests/drop2.js\"] );\n" 
+"\"jstests/drop2.js\",\n" 
+"\"jstests/dropdb_race.js\",\n" 
+"\"jstests/bench_test1.js\"] );\n" 
 "\n" 
 "// some tests can't be run in parallel with each other\n" 
 "var serialTestsArr = [ \"jstests/fsync.js\",\n"