We're preparing the upgrade to GitLab 10! Preview the new UI: http://cern.ch/go/8stB

Commit 42466d1f by Steven Murray

CASTOR-5487 New Recommended Access Order (RAO) logs

Done.
parent a7b9f971
Pipeline #208793 passed with stages
in 12 minutes 29 seconds
......@@ -152,7 +152,7 @@ void RecallTaskInjector::injectBulkRecalls() {
for (uint32_t i = 0; i < njobs; i++) {
tapegateway::RAOFileToRecallStruct *job = m_jobs.at(i);
if (! job->hasFileSize()) {
m_lc.log(LOG_INFO, "Skipping Recommended Access Order because the jobs do not provide file size");
m_lc.log(LOG_INFO, "RAO supported by the drive but not by the stager");
m_useRAO = false;
break;
}
......@@ -194,13 +194,19 @@ void RecallTaskInjector::injectBulkRecalls() {
files.clear();
}
std::string queryOrderLog = "Query fseq order:";
std::ostringstream recallOrderLog;
if(m_useRAO) {
recallOrderLog << "Recall order of FSEQs using RAO:";
} else {
recallOrderLog << "Recall order of FSEQs:";
}
for (uint32_t i = 0; i < njobs; i++) {
uint32_t index = m_useRAO ? raoOrder.at(i) : i;
tapegateway::RAOFileToRecallStruct *job = m_jobs.at(index);
queryOrderLog += std::to_string(job->fseq()) + " ";
recallOrderLog << " " << job->fseq();
// Workaround for bug CASTOR-4829: tapegateway: should request positioning by blockid for recalls instead of fseq
// When the client is a tape gateway, we should *always* position by block id.
......@@ -238,7 +244,7 @@ void RecallTaskInjector::injectBulkRecalls() {
delete job;
}
m_jobs.clear();
m_lc.log(LOG_INFO, queryOrderLog);
m_lc.log(LOG_INFO, recallOrderLog.str());
LogContext::ScopedParam sp03(m_lc, Param("nbFile", m_jobs.size()));
m_lc.log(LOG_INFO, "Finished processing batch of recall tasks from client");
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment