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

SERVER-127128 Use fmt::format to make _id field helper more readable for persistent samples (#54073)

Browse Source 
GitOrigin-RevId: dd50cf3c8e446c8154fa2474c6fde000c592fa5a
This commit is contained in:
natalie-hill 2026-05-20 11:28:31 -04:00 committed by MongoDB Bot
parent ebfca3c2a0
commit 00fbf91ff5
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/mongo/db/query/compiler/ce/sampling/persistent_sample_loader.cpp
View File

@ -38,26 +38,32 @@
#include "mongo/util/assert_util.h"
#include "mongo/util/str.h"
#include <string>
#include <fmt/format.h>
namespace mongo::ce {
std::string buildPersistentSampleId(const UUID& collectionUuid,
SamplingCEMethodEnum method,
size_t sampleSize,
boost::optional<int> numChunks) {
str::stream key;
key << collectionUuid.toString() << "_";
std::string methodStr;
if (method == SamplingCEMethodEnum::kChunk) {
tassert(
12432800, "Chunk-based persistent sample ID requires numChunks", numChunks.has_value());
key << "chunk" << *numChunks;
methodStr = fmt::format("chunk{}", *numChunks);
} else {
tassert(12432801,
"numChunks must only be set for chunk-technique persistent samples",
!numChunks.has_value());
key << idlSerialize(method);
methodStr = std::string(idlSerialize(method));
}
key << "_" << sampleSize << "_v" << kPersistentSampleSchemaVersion;
return key;
return fmt::format("{}_{}_{}_v{}",
collectionUuid.toString(),
methodStr,
sampleSize,
kPersistentSampleSchemaVersion);
}
StatusWith<PersistentSampleDoc> parsePersistentSample(const BSONObj& doc) {