diff options
Diffstat (limited to 'src/mongo')
-rw-r--r-- | src/mongo/db/SConscript | 3 | ||||
-rw-r--r-- | src/mongo/db/coll_mod.idl | 125 | ||||
-rw-r--r-- | src/mongo/db/commands/dbcommands.cpp | 11 | ||||
-rw-r--r-- | src/mongo/s/commands/SConscript | 1 | ||||
-rw-r--r-- | src/mongo/s/commands/cluster_collection_mod_cmd.cpp | 14 |
5 files changed, 149 insertions, 5 deletions
diff --git a/src/mongo/db/SConscript b/src/mongo/db/SConscript index e395e915490..ae7496afa42 100644 --- a/src/mongo/db/SConscript +++ b/src/mongo/db/SConscript @@ -572,6 +572,7 @@ env.CppUnitTest( env.Library( target="commands", source=[ + 'coll_mod.idl', 'commands.cpp', 'drop.idl', 'drop_database.idl', @@ -585,6 +586,8 @@ env.Library( ], LIBDEPS_PRIVATE=[ '$BUILD_DIR/mongo/bson/mutable/mutable_bson', + '$BUILD_DIR/mongo/db/catalog/collection_options_idl', + '$BUILD_DIR/mongo/db/commands/create_command', '$BUILD_DIR/mongo/idl/idl_parser', '$BUILD_DIR/mongo/rpc/command_status', '$BUILD_DIR/mongo/rpc/rpc', diff --git a/src/mongo/db/coll_mod.idl b/src/mongo/db/coll_mod.idl new file mode 100644 index 00000000000..65cc46b0e15 --- /dev/null +++ b/src/mongo/db/coll_mod.idl @@ -0,0 +1,125 @@ +# Copyright (C) 2018-present MongoDB, Inc. +# +# This program is free software: you can redistribute it and/or modify +# it under the terms of the Server Side Public License, version 1, +# as published by MongoDB, Inc. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# Server Side Public License for more details. +# +# You should have received a copy of the Server Side Public License +# along with this program. If not, see +# <http://www.mongodb.com/licensing/server-side-public-license>. +# +# As a special exception, the copyright holders give permission to link the +# code of portions of this program with the OpenSSL library under certain +# conditions as described in each individual source file and distribute +# linked combinations including the program with the OpenSSL library. You +# must comply with the Server Side Public License in all respects for +# all of the code used other than as permitted herein. If you modify file(s) +# with this exception, you may extend this exception to your version of the +# file(s), but you are not obligated to do so. If you do not wish to do so, +# delete this exception statement from your version. If you delete this +# exception statement from all source files in the program, then also delete +# it in the license file. +# +global: + cpp_namespace: "mongo" + cpp_includes: + - "mongo/db/commands/create_command_validation.h" + +imports: + - "mongo/idl/basic_types.idl" + - "mongo/db/catalog/collection_options.idl" + +structs: + CollModIndex: + description: "A type representing a spec for an index to be modified by collMod command." + strict: true + fields: + name: + optional: true + type: string + keyPattern: + optional: true + type: object + expireAfterSeconds: + optional: true + type: safeInt + hidden: + optional: true + type: safeBool + + CollModReply: + description: "The collMod command's reply." + strict: true + fields: + ok: + # Used in mongos reply. + optional: true + type: safeDouble + expireAfterSeconds_old: + optional: true + type: safeInt + expireAfterSeconds_new: + optional: true + type: safeInt + hidden_old: + optional: true + type: safeBool + hidden_new: + optional: true + type: safeBool + writeConcernError: + optional: true + type: object +commands: + collMod: + description: "Specify collMod Command." + command_name: collMod + namespace: concatenate_with_db + cpp_name: collMod + strict: true + api_version: "1" + fields: + index: + description: "Index to be modified." + optional: true + type: CollModIndex + validator: + description: "Specify validation rules or expressions for the collection." + type: object + optional: true + validationLevel: + description: "Determines how strictly to apply the validation rules to existing + documents during an update. + Can be one of following values: 'off', 'strict' or 'moderate'." + type: ValidationLevel + optional: true + validationAction: + description: "Determines whether to error on invalid documents or just warn about + the violations but allow invalid documents to be inserted. + Can be either 'warn' or 'error'." + type: ValidationAction + optional: true + viewOn: + description: "The name of the source collection or view that + 'namespace' is based off of." + type: string + optional: true + validator: + callback: create_command_validation::validateViewOnNotEmpty + pipeline: + description: "An array that consists of the aggregation pipeline. Defines the view + by applying the specified pipeline to the 'viewOn' collection or + view." + type: array<object> + optional: true + recordPreImages: + description: "Sets whether updates/deletes should store the pre-image of the + document in the oplog" + optional: true + type: safeBool + reply_type: CollModReply
\ No newline at end of file diff --git a/src/mongo/db/commands/dbcommands.cpp b/src/mongo/db/commands/dbcommands.cpp index d9832267537..20583a4685f 100644 --- a/src/mongo/db/commands/dbcommands.cpp +++ b/src/mongo/db/commands/dbcommands.cpp @@ -53,6 +53,7 @@ #include "mongo/db/catalog/index_key_validate.h" #include "mongo/db/catalog_raii.h" #include "mongo/db/clientcursor.h" +#include "mongo/db/coll_mod_gen.h" #include "mongo/db/commands.h" #include "mongo/db/commands/create_gen.h" #include "mongo/db/commands/profile_common.h" @@ -732,8 +733,14 @@ public: const string& dbname, const BSONObj& jsobj, BSONObjBuilder& result) { - const NamespaceString nss(CommandHelpers::parseNsCollectionRequired(dbname, jsobj)); - uassertStatusOK(collMod(opCtx, nss, jsobj, &result)); + auto cmd = CollMod::parse( + IDLParserErrorContext(CollMod::kCommandName, + APIParameters::get(opCtx).getAPIStrict().value_or(false)), + jsobj); + uassertStatusOK(collMod(opCtx, cmd.getNamespace(), cmd.toBSON(BSONObj()), &result)); + + CollModReply::parse(IDLParserErrorContext("CollModReply"), result.asTempObj()); + return true; } diff --git a/src/mongo/s/commands/SConscript b/src/mongo/s/commands/SConscript index 75a90c02b30..81d5ae92e84 100644 --- a/src/mongo/s/commands/SConscript +++ b/src/mongo/s/commands/SConscript @@ -107,6 +107,7 @@ env.Library( '$BUILD_DIR/mongo/db/auth/auth', '$BUILD_DIR/mongo/db/auth/builtin_roles', '$BUILD_DIR/mongo/db/auth/saslauth', + '$BUILD_DIR/mongo/db/commands', '$BUILD_DIR/mongo/db/commands/core', '$BUILD_DIR/mongo/db/commands/create_command', '$BUILD_DIR/mongo/db/commands/current_op_common', diff --git a/src/mongo/s/commands/cluster_collection_mod_cmd.cpp b/src/mongo/s/commands/cluster_collection_mod_cmd.cpp index 6296c71a8d6..c421dc5ba72 100644 --- a/src/mongo/s/commands/cluster_collection_mod_cmd.cpp +++ b/src/mongo/s/commands/cluster_collection_mod_cmd.cpp @@ -32,6 +32,7 @@ #include "mongo/platform/basic.h" #include "mongo/db/auth/authorization_session.h" +#include "mongo/db/coll_mod_gen.h" #include "mongo/db/commands.h" #include "mongo/logv2/log.h" #include "mongo/s/cluster_commands_helpers.h" @@ -72,7 +73,12 @@ public: const BSONObj& cmdObj, std::string& errmsg, BSONObjBuilder& output) override { - const NamespaceString nss(CommandHelpers::parseNsCollectionRequired(dbName, cmdObj)); + auto cmd = CollMod::parse( + IDLParserErrorContext(CollMod::kCommandName, + APIParameters::get(opCtx).getAPIStrict().value_or(false)), + cmdObj); + + auto nss = cmd.getNamespace(); LOGV2_DEBUG(22748, 1, "collMod: {namespace} cmd: {command}", @@ -84,11 +90,13 @@ public: uassertStatusOK(Grid::get(opCtx)->catalogCache()->getCollectionRoutingInfo(opCtx, nss)); auto shardResponses = scatterGatherVersionedTargetByRoutingTable( opCtx, - nss.db(), + cmd.getDbName(), nss, routingInfo, applyReadWriteConcern( - opCtx, this, CommandHelpers::filterCommandRequestForPassthrough(cmdObj)), + opCtx, + this, + CommandHelpers::filterCommandRequestForPassthrough(cmd.toBSON(BSONObj()))), ReadPreferenceSetting::get(opCtx), Shard::RetryPolicy::kNoRetry, BSONObj() /* query */, |