diff options
author | Tor Brede Vekterli <vekterli@yahooinc.com> | 2021-11-10 15:11:58 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-11-10 15:11:58 +0100 |
commit | dbb2914c4ae54e0d80dfa151ee7768a42c600868 (patch) | |
tree | fe0805d338b503394000081f17cced867e5580c4 | |
parent | 6393fbb164ee9bee75159fc8b1a625452e87265a (diff) | |
parent | 34c5a4d2f3c66e1ebbc5d4d1415a812a15933119 (diff) |
Merge pull request #19952 from vespa-engine/vekterli/remove-writing-to-tmp-during-health-ping
Remove implicit write to tmp as part of cluster controller health ping
-rw-r--r-- | storage/src/vespa/storage/storageserver/statemanager.cpp | 24 |
1 files changed, 0 insertions, 24 deletions
diff --git a/storage/src/vespa/storage/storageserver/statemanager.cpp b/storage/src/vespa/storage/storageserver/statemanager.cpp index 7596bdf1b84..7d08f738abe 100644 --- a/storage/src/vespa/storage/storageserver/statemanager.cpp +++ b/storage/src/vespa/storage/storageserver/statemanager.cpp @@ -577,21 +577,6 @@ StateManager::sendGetNodeStateReplies(framework::MilliSecTime olderThanTime, uin return true; } -namespace { - std::string getHostInfoFilename(bool advanceCount) { - static uint32_t fileCounter = 0; - static pid_t pid = getpid(); - if (advanceCount) { - ++fileCounter; - } - uint32_t fileIndex = fileCounter % 8; - std::ostringstream fileName; - fileName << vespa::Defaults::underVespaHome("tmp/hostinfo") - << "." << pid << "." << fileIndex << ".report"; - return fileName.str(); - } -} - std::string StateManager::getNodeInfo() const { @@ -631,15 +616,6 @@ StateManager::getNodeInfo() const stream << End(); stream.finalize(); - // Dump report to new report file. - std::string oldFile(getHostInfoFilename(false)); - std::string newFile(getHostInfoFilename(true)); - std::ofstream of(newFile.c_str()); - of << json.str(); - of.close(); - // If dumping went ok, delete old report file - vespalib::unlink(oldFile); - // Return report return json.str(); } |