SERVER-78803 Reject concurrent query settings updates in sharded clusters Until now, a new value of a cluster parameter was just replacing the old one. For the query settings project it is important to read / modify / write cluster parameter values. In this scenario, concurrent calls of setQuerySettings command may lead to seemingly successful, but incorrect results. This commit introduces an additional 'previousTime' parameter to ensure that the cluster parameter value that was read has not changed at the time of writing, otherwise the operation must fail. SERVER-78802 has added the 'previousTime' parameter and the related check for replica set deployments. This change is adding the check to sharded cluster deployments. As of now, the optional 'previousTime' parameter is only being used by the query settings command. The design consideration for this new parameter and the related check are described in the design document attached to the parent ticket SERVER-78801. GitOrigin-RevId: f5f35648c676f7dd132ba69d34bbd5251b71a2e1
184 lines
7.7 KiB
C++
184 lines
7.7 KiB
C++
/**
|
|
* Copyright (C) 2023-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.
|
|
*/
|
|
|
|
|
|
#include <boost/move/utility_core.hpp>
|
|
#include <boost/none.hpp>
|
|
#include <memory>
|
|
#include <string>
|
|
#include <utility>
|
|
|
|
#include "mongo/db/commands/set_cluster_parameter_command_impl.h"
|
|
|
|
#include "mongo/base/error_codes.h"
|
|
#include "mongo/base/shim.h"
|
|
#include "mongo/base/string_data.h"
|
|
#include "mongo/bson/timestamp.h"
|
|
#include "mongo/db/auth/action_type.h"
|
|
#include "mongo/db/auth/authorization_session.h"
|
|
#include "mongo/db/auth/privilege.h"
|
|
#include "mongo/db/auth/resource_pattern.h"
|
|
#include "mongo/db/bson/dotted_path_support.h"
|
|
#include "mongo/db/cluster_role.h"
|
|
#include "mongo/db/commands.h"
|
|
#include "mongo/db/commands/cluster_server_parameter_cmds_gen.h"
|
|
#include "mongo/db/commands/feature_compatibility_version.h"
|
|
#include "mongo/db/commands/set_cluster_parameter_invocation.h"
|
|
#include "mongo/db/database_name.h"
|
|
#include "mongo/db/dbdirectclient.h"
|
|
#include "mongo/db/feature_flag.h"
|
|
#include "mongo/db/namespace_string.h"
|
|
#include "mongo/db/operation_context.h"
|
|
#include "mongo/db/repl/replication_coordinator.h"
|
|
#include "mongo/db/server_feature_flags_gen.h"
|
|
#include "mongo/db/server_options.h"
|
|
#include "mongo/db/service_context.h"
|
|
#include "mongo/db/write_concern_options.h"
|
|
#include "mongo/rpc/op_msg.h"
|
|
#include "mongo/s/grid.h"
|
|
#include "mongo/s/request_types/sharded_ddl_commands_gen.h"
|
|
#include "mongo/util/assert_util.h"
|
|
#include "mongo/util/str.h"
|
|
|
|
#define MONGO_LOGV2_DEFAULT_COMPONENT ::mongo::logv2::LogComponent::kCommand
|
|
|
|
|
|
namespace mongo {
|
|
namespace {
|
|
MONGO_FAIL_POINT_DEFINE(hangInSetClusterParameter);
|
|
const auto setClusterParameterImplDecoration =
|
|
Service::declareDecoration<SetClusterParameterImplFn>();
|
|
|
|
const WriteConcernOptions kMajorityWriteConcern{WriteConcernOptions::kMajority,
|
|
WriteConcernOptions::SyncMode::UNSET,
|
|
WriteConcernOptions::kNoTimeout};
|
|
|
|
void hangInSetClusterParameterFailPointCheck(const SetClusterParameter& request) {
|
|
// The failpoint will block the thread unless the 'data' parameter contains a pattern
|
|
// that does not match the 'cmdObject'.
|
|
if (MONGO_unlikely(hangInSetClusterParameter.shouldFail(
|
|
[cmdObject = request.getCommandParameter()](const BSONObj& data) {
|
|
if (data.isEmpty()) {
|
|
return true;
|
|
}
|
|
BSONElementMultiSet bSet;
|
|
dotted_path_support::extractAllElementsAlongPath(
|
|
cmdObject, data.firstElementFieldNameStringData(), bSet, false);
|
|
return std::any_of(bSet.begin(), bSet.end(), [data](const BSONElement& elem) {
|
|
return elem.Obj().woCompare(data.firstElement().Obj()) == 0;
|
|
});
|
|
}))) {
|
|
hangInSetClusterParameter.pauseWhileSet();
|
|
}
|
|
}
|
|
|
|
void setClusterParameterImplShard(OperationContext* opCtx,
|
|
const SetClusterParameter& request,
|
|
boost::optional<Timestamp> clusterParameterTime,
|
|
boost::optional<LogicalTime> previousTime) {
|
|
uassert(ErrorCodes::ErrorCodes::NotImplemented,
|
|
"setClusterParameter can only run on mongos in sharded clusters",
|
|
(serverGlobalParams.clusterRole.has(ClusterRole::None)));
|
|
|
|
if (!feature_flags::gFeatureFlagAuditConfigClusterParameter.isEnabled(
|
|
serverGlobalParams.featureCompatibility.acquireFCVSnapshot())) {
|
|
uassert(ErrorCodes::IllegalOperation,
|
|
str::stream() << SetClusterParameter::kCommandName
|
|
<< " cannot be run on standalones",
|
|
repl::ReplicationCoordinator::get(opCtx)->getSettings().isReplSet());
|
|
}
|
|
|
|
hangInSetClusterParameterFailPointCheck(request);
|
|
|
|
// setClusterParameter is serialized against setFeatureCompatibilityVersion.
|
|
FixedFCVRegion fcvRegion(opCtx);
|
|
|
|
std::unique_ptr<ServerParameterService> parameterService =
|
|
std::make_unique<ClusterParameterService>();
|
|
|
|
DBDirectClient dbClient(opCtx);
|
|
ClusterParameterDBClientService dbService(dbClient);
|
|
|
|
SetClusterParameterInvocation invocation{std::move(parameterService), dbService};
|
|
|
|
invocation.invoke(opCtx, request, clusterParameterTime, previousTime, kMajorityWriteConcern);
|
|
}
|
|
|
|
void setClusterParameterImplRouter(OperationContext* opCtx,
|
|
const SetClusterParameter& request,
|
|
boost::optional<Timestamp>,
|
|
boost::optional<LogicalTime> previousTime) {
|
|
|
|
hangInSetClusterParameterFailPointCheck(request);
|
|
ConfigsvrSetClusterParameter configsvrSetClusterParameter(request.getCommandParameter());
|
|
configsvrSetClusterParameter.setDbName(request.getDbName());
|
|
configsvrSetClusterParameter.setPreviousTime(previousTime);
|
|
|
|
const auto configShard = Grid::get(opCtx)->shardRegistry()->getConfigShard();
|
|
|
|
const auto cmdResponse = uassertStatusOK(configShard->runCommandWithFixedRetryAttempts(
|
|
opCtx,
|
|
ReadPreferenceSetting(ReadPreference::PrimaryOnly),
|
|
DatabaseName::kAdmin,
|
|
configsvrSetClusterParameter.toBSON({}),
|
|
Shard::RetryPolicy::kIdempotent));
|
|
|
|
uassertStatusOK(Shard::CommandResponse::getEffectiveStatus(std::move(cmdResponse)));
|
|
}
|
|
} // namespace
|
|
|
|
SetClusterParameterImplFn getSetClusterParameterImpl(Service* service) {
|
|
auto fn = setClusterParameterImplDecoration(service);
|
|
invariant(fn);
|
|
return fn;
|
|
}
|
|
|
|
SetClusterParameterImplFn getSetClusterParameterImpl(OperationContext* ctx) {
|
|
return getSetClusterParameterImpl(ctx->getService());
|
|
}
|
|
|
|
namespace {
|
|
ServiceContext::ConstructorActionRegisterer setParameterImplRegisterer(
|
|
"setClusterParameterImpl-registerer",
|
|
{},
|
|
[](ServiceContext* serviceContext) {
|
|
invariant(serviceContext);
|
|
auto routerService = serviceContext->getService(ClusterRole::RouterServer);
|
|
if (routerService) {
|
|
setClusterParameterImplDecoration(routerService) = &setClusterParameterImplRouter;
|
|
}
|
|
auto shardService = serviceContext->getService(ClusterRole::ShardServer);
|
|
if (shardService) {
|
|
setClusterParameterImplDecoration(shardService) = &setClusterParameterImplShard;
|
|
}
|
|
},
|
|
{});
|
|
} // namespace
|
|
} // namespace mongo
|