diff options
author | Will Buerger <will.buerger@mongodb.com> | 2023-05-16 20:32:42 +0000 |
---|---|---|
committer | Evergreen Agent <no-reply@evergreen.mongodb.com> | 2023-05-17 00:09:49 +0000 |
commit | d646e44b7801a3e5b3230bbae7dcfe05a5ed8707 (patch) | |
tree | 2eebf945a3f958e3adddc157523a49deb72cc63d /src/mongo/db/commands | |
parent | 8219766da20351edcda5fe21a18da254b382cd35 (diff) | |
download | mongo-d646e44b7801a3e5b3230bbae7dcfe05a5ed8707.tar.gz |
SERVER-76427: Rename $telemetry to $queryStats
Co-authored-by: Ted Tuckman <ted.tuckman@mongodb.com>
Diffstat (limited to 'src/mongo/db/commands')
-rw-r--r-- | src/mongo/db/commands/find_cmd.cpp | 21 | ||||
-rw-r--r-- | src/mongo/db/commands/getmore_cmd.cpp | 2 | ||||
-rw-r--r-- | src/mongo/db/commands/run_aggregate.cpp | 18 |
3 files changed, 21 insertions, 20 deletions
diff --git a/src/mongo/db/commands/find_cmd.cpp b/src/mongo/db/commands/find_cmd.cpp index 90e6fa15ca1..983661fbd15 100644 --- a/src/mongo/db/commands/find_cmd.cpp +++ b/src/mongo/db/commands/find_cmd.cpp @@ -55,7 +55,7 @@ #include "mongo/db/query/find_request_shapifier.h" #include "mongo/db/query/get_executor.h" #include "mongo/db/query/query_knobs_gen.h" -#include "mongo/db/query/telemetry.h" +#include "mongo/db/query/query_stats.h" #include "mongo/db/repl/replication_coordinator.h" #include "mongo/db/s/query_analysis_writer.h" #include "mongo/db/service_context.h" @@ -561,13 +561,14 @@ public: cq->setUseCqfIfEligible(true); if (collection) { - // Collect telemetry. Exclude queries against collections with encrypted fields. + // Collect queryStats. Exclude queries against collections with encrypted fields. if (!collection.get()->getCollectionOptions().encryptedFieldConfig) { - telemetry::registerRequest(std::make_unique<telemetry::FindRequestShapifier>( - cq->getFindCommandRequest(), opCtx), - collection.get()->ns(), - opCtx, - cq->getExpCtx()); + query_stats::registerRequest( + std::make_unique<query_stats::FindRequestShapifier>( + cq->getFindCommandRequest(), opCtx), + collection.get()->ns(), + opCtx, + cq->getExpCtx()); } } @@ -780,9 +781,9 @@ public: processFLEFindD( opCtx, findCommand->getNamespaceOrUUID().nss().value(), findCommand.get()); } - // Set the telemetryStoreKey to none so telemetry isn't collected when we've done a - // FLE rewrite. - CurOp::get(opCtx)->debug().telemetryStoreKeyHash = boost::none; + // Set the queryStatsStoreKey to none so queryStats isn't collected when we've done + // a FLE rewrite. + CurOp::get(opCtx)->debug().queryStatsStoreKeyHash = boost::none; CurOp::get(opCtx)->debug().shouldOmitDiagnosticInformation = true; } diff --git a/src/mongo/db/commands/getmore_cmd.cpp b/src/mongo/db/commands/getmore_cmd.cpp index 8f91862d002..54035c99829 100644 --- a/src/mongo/db/commands/getmore_cmd.cpp +++ b/src/mongo/db/commands/getmore_cmd.cpp @@ -703,7 +703,7 @@ public: metricsCollector.incrementDocUnitsReturned(curOp->getNS(), docUnitsReturned); curOp->debug().additiveMetrics.nBatches = 1; curOp->setEndOfOpMetrics(numResults); - collectTelemetryMongod(opCtx, cursorPin); + collectQueryStatsMongod(opCtx, cursorPin); if (respondWithId) { cursorDeleter.dismiss(); diff --git a/src/mongo/db/commands/run_aggregate.cpp b/src/mongo/db/commands/run_aggregate.cpp index a290ef56713..0bc7d7f6415 100644 --- a/src/mongo/db/commands/run_aggregate.cpp +++ b/src/mongo/db/commands/run_aggregate.cpp @@ -76,7 +76,7 @@ #include "mongo/db/query/query_feature_flags_gen.h" #include "mongo/db/query/query_knobs_gen.h" #include "mongo/db/query/query_planner_common.h" -#include "mongo/db/query/telemetry.h" +#include "mongo/db/query/query_stats.h" #include "mongo/db/read_concern.h" #include "mongo/db/repl/oplog.h" #include "mongo/db/repl/read_concern_args.h" @@ -836,11 +836,11 @@ Status runAggregate(OperationContext* opCtx, }; auto registerTelemetry = [&]() -> void { - // Register telemetry. Exclude queries against collections with encrypted fields. - // We still collect telemetry on collection-less aggregations. + // Register queryStats. Exclude queries against collections with encrypted fields. + // We still collect queryStats on collection-less aggregations. if (!(ctx && ctx->getCollection() && ctx->getCollection()->getCollectionOptions().encryptedFieldConfig)) { - telemetry::registerAggRequest(request, opCtx); + query_stats::registerAggRequest(request, opCtx); } }; @@ -1051,9 +1051,9 @@ Status runAggregate(OperationContext* opCtx, request.getEncryptionInformation()->setCrudProcessed(true); } - // Set the telemetryStoreKey to none so telemetry isn't collected when we've done a FLE - // rewrite. - CurOp::get(opCtx)->debug().telemetryStoreKeyHash = boost::none; + // Set the queryStatsStoreKey to none so queryStats isn't collected when we've done a + // FLE rewrite. + CurOp::get(opCtx)->debug().queryStatsStoreKeyHash = boost::none; } pipeline->optimizePipeline(); @@ -1223,9 +1223,9 @@ Status runAggregate(OperationContext* opCtx, curOp->setEndOfOpMetrics(stats.nReturned); if (keepCursor) { - collectTelemetryMongod(opCtx, pins[0]); + collectQueryStatsMongod(opCtx, pins[0]); } else { - collectTelemetryMongod(opCtx, std::move(curOp->debug().telemetryRequestShapifier)); + collectQueryStatsMongod(opCtx, std::move(curOp->debug().queryStatsRequestShapifier)); } // For an optimized away pipeline, signal the cache that a query operation has completed. |