mongodb/mongo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

SERVER-122957 Fix the feature flag checks as it conflicted with the bulk rename and it used a new process (#54155)

Browse Source 
GitOrigin-RevId: 8586e253494e6ea4fb613bccc56b989cfed5b76a
This commit is contained in:
Jordi Olivares Provencio 2026-05-21 10:26:54 +02:00 committed by MongoDB Bot
parent 1ba8dd0ddf
commit 0adedd4f48
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/mongo/db/global_catalog/ddl/rename_collection_coordinator.cpp
View File

@ -817,9 +817,9 @@ ExecutorFuture<void> RenameCollectionCoordinator::_runImpl(
// for updating the shard catalog with current information. This flag is
// evaluated at insertion time because on secondaries, metadata is cleared
// during the onDelete of the critical section document.
if (feature_flags::gAuthoritativeShardsDDL.isEnabled(
VersionContext::getDecoration(opCtx),
serverGlobalParams.featureCompatibility.acquireFCVSnapshot())) {
bool isDDLAuthoritative = _doc.getAuthoritativeMetadataAccessLevel() >=
AuthoritativeMetadataAccessLevelEnum::kWritesAllowed;
if (isDDLAuthoritative) {
blockCRUDOperationsRequest.setClearCollMetadata(false);
}
@ -945,9 +945,9 @@ ExecutorFuture<void> RenameCollectionCoordinator::_runImpl(
// for updating the shard catalog with current information. This flag is
// evaluated at insertion time because on secondaries, metadata is cleared
// during the onDelete of the critical section document.
if (feature_flags::gAuthoritativeShardsDDL.isEnabled(
VersionContext::getDecoration(opCtx),
serverGlobalParams.featureCompatibility.acquireFCVSnapshot())) {
bool isDDLAuthoritative = _doc.getAuthoritativeMetadataAccessLevel() >=
AuthoritativeMetadataAccessLevelEnum::kWritesAllowed;
if (isDDLAuthoritative) {
renameCollParticipantRequest.setClearCollMetadata(false);
}
@ -1049,9 +1049,9 @@ ExecutorFuture<void> RenameCollectionCoordinator::_runImpl(
session);
}
if (feature_flags::gAuthoritativeShardsDDL.isEnabled(
VersionContext::getDecoration(opCtx),
serverGlobalParams.featureCompatibility.acquireFCVSnapshot())) {
bool isDDLAuthoritative = _doc.getAuthoritativeMetadataAccessLevel() >=
AuthoritativeMetadataAccessLevelEnum::kWritesAllowed;
if (isDDLAuthoritative) {
const auto session = getNewSession(opCtx);
sharding_ddl_util::commitRenameCollectionMetadataToShardCatalog(
opCtx,