Skip to content

Commit 1932f01

Browse files
streams1MongoDB Bot
authored andcommitted
SERVER-91128 Remove function OpDebug::incrementExecutionTime (#31875)
GitOrigin-RevId: 33c1b9e
1 parent e097164 commit 1932f01

File tree

3 files changed

+0
-15
lines changed

3 files changed

+0
-15
lines changed

src/mongo/db/curop.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2412,13 +2412,6 @@ void OpDebug::AdditiveMetrics::incrementNUpserted(long long n) {
24122412
*nUpserted += n;
24132413
}
24142414

2415-
void OpDebug::AdditiveMetrics::incrementExecutionTime(Microseconds n) {
2416-
if (!executionTime) {
2417-
executionTime = Microseconds{0};
2418-
}
2419-
*executionTime += n;
2420-
}
2421-
24222415
std::string OpDebug::AdditiveMetrics::report() const {
24232416
StringBuilder s;
24242417

src/mongo/db/curop.h

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -194,11 +194,6 @@ class OpDebug {
194194
*/
195195
void incrementNUpserted(long long n);
196196

197-
/**
198-
* Increments executionTime by n.
199-
*/
200-
void incrementExecutionTime(Microseconds n);
201-
202197
/**
203198
* Generates a string showing all non-empty fields. For every non-empty field field1,
204199
* field2, ..., with corresponding values value1, value2, ..., we will output a string in

src/mongo/db/curop_test.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,6 @@ TEST(CurOpTest, AdditiveMetricsFieldsShouldIncrementByN) {
199199
additiveMetrics.writeConflicts.store(1);
200200
additiveMetrics.keysInserted = 2;
201201
additiveMetrics.nreturned = 3;
202-
additiveMetrics.executionTime = Microseconds{160};
203202

204203
// Increment the fields.
205204
additiveMetrics.incrementWriteConflicts(1);
@@ -209,7 +208,6 @@ TEST(CurOpTest, AdditiveMetricsFieldsShouldIncrementByN) {
209208
additiveMetrics.incrementNUpserted(6);
210209
additiveMetrics.incrementNreturned(2);
211210
additiveMetrics.incrementNBatches();
212-
additiveMetrics.incrementExecutionTime(Microseconds{120});
213211

214212
ASSERT_EQ(additiveMetrics.writeConflicts.load(), 2);
215213
ASSERT_EQ(*additiveMetrics.keysInserted, 7);
@@ -218,7 +216,6 @@ TEST(CurOpTest, AdditiveMetricsFieldsShouldIncrementByN) {
218216
ASSERT_EQ(*additiveMetrics.nUpserted, 6);
219217
ASSERT_EQ(*additiveMetrics.nreturned, 5);
220218
ASSERT_EQ(*additiveMetrics.nBatches, 1);
221-
ASSERT_EQ(*additiveMetrics.executionTime, Microseconds{280});
222219
}
223220

224221
TEST(CurOpTest, AdditiveMetricsShouldAggregateCursorMetrics) {

0 commit comments

Comments
 (0)