Skip to content

Commit 294889f

Browse files
authored
Do not flush empty buffer (#262)
1 parent 7227479 commit 294889f

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/Monitoring.cxx

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,10 @@ void Monitoring::flushBuffer()
6161
return;
6262
}
6363
for (auto& [verbosity, buffer] : mStorage) {
64+
if (buffer.empty()) {
65+
MonLogger::Get() << "Not flushing empty buffer" << MonLogger::End();
66+
continue;
67+
}
6468
for (auto& backend : mBackends) {
6569
if (matchVerbosity(backend->getVerbosity(), static_cast<Verbosity>(verbosity))) {
6670
backend->send(std::move(buffer));
@@ -132,9 +136,7 @@ Monitoring::~Monitoring()
132136
mMonitorThread.join();
133137
transmit(mProcessMonitor->makeLastMeasurementAndGetMetrics());
134138
}
135-
if (mBuffering) {
136-
flushBuffer();
137-
}
139+
flushBuffer();
138140
}
139141

140142
void Monitoring::pushLoop()

0 commit comments

Comments
 (0)