diff options
author | Maria van Keulen <maria@mongodb.com> | 2018-01-03 16:49:38 -0500 |
---|---|---|
committer | Maria van Keulen <maria@mongodb.com> | 2018-01-04 18:28:55 -0500 |
commit | 489d177dbd0f0420a8ca04d39fd78d0a2c539420 (patch) | |
tree | 4e23f554a69e1c233cfd12340a05a13972f57c17 /src/mongo/db/repl/replication_coordinator_external_state_impl.cpp | |
parent | 2e346aaf5452b86a2106af645ddf65a51a9590fa (diff) | |
download | mongo-489d177dbd0f0420a8ca04d39fd78d0a2c539420.tar.gz |
SERVER-32255 Ensure UUIDs exist for local collections on ShardServersr3.6.2-rc0r3.6.2
(cherry picked from commit 2158ea8d6c84c9a68b50f76c9dd23b8a193b65f8)
Conflicts:
buildscripts/resmokeconfig/suites/sharding_last_stable_mongos_and_mixed_shards.yml
jstests/multiVersion/do_upgrade_downgrade.js
src/mongo/db/commands/validate.cpp
Diffstat (limited to 'src/mongo/db/repl/replication_coordinator_external_state_impl.cpp')
-rw-r--r-- | src/mongo/db/repl/replication_coordinator_external_state_impl.cpp | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/src/mongo/db/repl/replication_coordinator_external_state_impl.cpp b/src/mongo/db/repl/replication_coordinator_external_state_impl.cpp index fb675304cfe..f2a2ff633ad 100644 --- a/src/mongo/db/repl/replication_coordinator_external_state_impl.cpp +++ b/src/mongo/db/repl/replication_coordinator_external_state_impl.cpp @@ -404,12 +404,18 @@ Status ReplicationCoordinatorExternalStateImpl::initializeReplSetStorage(Operati // Set UUIDs for all non-replicated collections. This is necessary for independent replica // sets and config server replica sets started with no data files because collections in // local are created prior to the featureCompatibilityVersion being set to 3.6, so the - // collections are not created with UUIDs. This is not an issue for shard servers because - // the config server sends a setFeatureCompatibilityVersion command with the - // featureCompatibilityVersion equal to the cluster's featureCompatibilityVersion during - // addShard, which will add UUIDs to all collections that do not already have them. Here, - // we add UUIDs to the non-replicated collections on the primary. We add them on the - // secondaries during InitialSync. + // collections are not created with UUIDs. We exclude ShardServers when adding UUIDs to + // non-replicated collections on the primary because ShardServers are started up by default + // with featureCompatibilityVersion 3.4, so we don't want to assign UUIDs to them until the + // cluster's featureCompatibilityVersion is explicitly set to 3.6 by the config server. The + // below UUID addition for non-replicated collections only occurs on the primary; UUIDs are + // added to non-replicated collections on secondaries during InitialSync. When the config + // server sets the featureCompatibilityVersion to 3.6, the shard primary will add UUIDs to + // all the collections that need them. One special case here is if a shard is already in + // featureCompatibilityVersion 3.6 and a new node is started up with --shardsvr and added to + // that shard, the new node will still start up with featureCompatibilityVersion 3.4 and + // need to have UUIDs added to each collection. These UUIDs are added during InitialSync, + // because the new node is a secondary. if (serverGlobalParams.clusterRole != ClusterRole::ShardServer && FeatureCompatibilityVersion::isCleanStartUp()) { auto schemaStatus = updateUUIDSchemaVersionNonReplicated(opCtx, true); |