From 1d2ad909db70df78e6a923c4364e8e2f1fa704cf Mon Sep 17 00:00:00 2001
From: Steven Murray <steven.murray@cern.ch>
Date: Mon, 9 Jan 2017 14:24:56 +0100
Subject: [PATCH] Scheduler::getPendingArchiveJobs() checks if tape pool exists
 in catalogue

---
 scheduler/Scheduler.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/scheduler/Scheduler.cpp b/scheduler/Scheduler.cpp
index 7145f4b10c..33e4e5dcaf 100644
--- a/scheduler/Scheduler.cpp
+++ b/scheduler/Scheduler.cpp
@@ -309,6 +309,9 @@ std::map<std::string, std::list<common::dataStructures::ArchiveJob> > Scheduler:
 // getPendingArchiveJobs
 //------------------------------------------------------------------------------
 std::list<common::dataStructures::ArchiveJob> Scheduler::getPendingArchiveJobs(const std::string &tapePoolName) const {
+  if(!m_catalogue.tapePoolExists(tapePoolName)) {
+    throw exception::Exception(std::string("Tape pool ") + tapePoolName + " does not exist");
+  }
   return m_db.getArchiveJobs(tapePoolName);
 }
 
@@ -322,7 +325,6 @@ std::map<std::string, std::list<common::dataStructures::RetrieveJob> > Scheduler
 //------------------------------------------------------------------------------
 // getPendingRetrieveJobs
 //------------------------------------------------------------------------------
-
 std::list<common::dataStructures::RetrieveJob> Scheduler::getPendingRetrieveJobs(const std::string& vid) const {
   throw exception::Exception(std::string("Not implemented: ") + __PRETTY_FUNCTION__);
 }
-- 
GitLab