Logs improvements. (Including removal of per memory block logs).

castor/legacymsg/VmgrProxyTcpIp.cpp

@@ -157,7 +157,7 @@ void castor::legacymsg::VmgrProxyTcpIp::tapeMountedForRead(const std::string &vi
   } catch(castor::exception::Exception &ne) {
     castor::exception::Exception ex;
     ex.getMessage() << "Failed to notify the VMGR that the tape " << vid <<
-      " was mounted: " << ne.getMessage().str();
+      " was mounted for READ by jid " << jid << " : " << ne.getMessage().str();
     throw ex;
   }
 }
@@ -179,7 +179,7 @@ void castor::legacymsg::VmgrProxyTcpIp::tapeMountedForWrite(const std::string &v
   } catch(castor::exception::Exception &ne) {
     castor::exception::Exception ex;
     ex.getMessage() << "Failed to notify the VMGR that the tape " << vid <<
-      " was mounted: " << ne.getMessage().str();
+      " was mounted for WRITE by jid " << jid << " : " << ne.getMessage().str();
     throw ex;
   }
 }

castor/tape/tapeserver/daemon/RecallMemoryManager.cpp

@@ -38,9 +38,13 @@ RecallMemoryManager::RecallMemoryManager(const size_t numberOfBlocks, const size
   for (size_t i = 0; i < numberOfBlocks; i++) {
     m_freeBlocks.push(new MemBlock(i, blockSize));
 
-    m_lc.pushOrReplace(log::Param("blockId", i));
-    m_lc.log(LOG_DEBUG, "RecallMemoryManager created a block");
+    //m_lc.pushOrReplace(log::Param("blockId", i));
+    //m_lc.log(LOG_DEBUG, "RecallMemoryManager created a block");
   }
+  log::ScopedParamContainer params(m_lc);
+  params.add("blockCount", numberOfBlocks)
+        .add("blockSize", blockSize)
+        .add("totalSize", numberOfBlocks*blockSize);
   m_lc.log(LOG_INFO, "RecallMemoryManager: all blocks have been created");
 }