[cta-statistics-save] The CTA statistics save tool will return ALL VOs...

[cta-statistics-save] The CTA statistics save tool will return ALL VOs statistics instead of only the ones that have files stored

statistics/DatabaseStatisticsService.cpp

@@ -188,18 +188,18 @@ std::unique_ptr<cta::statistics::Statistics> DatabaseStatisticsService::getStati
   const char * const sql = 
   "SELECT "
     "VIRTUAL_ORGANIZATION_NAME AS VO,"
-    "SUM(NB_MASTER_FILES) AS TOTAL_MASTER_FILES_VO,"
-    "SUM(MASTER_DATA_IN_BYTES) AS TOTAL_MASTER_DATA_BYTES_VO,"
-    "SUM(NB_COPY_NB_1) AS TOTAL_NB_COPY_1_VO,"
-    "SUM(COPY_NB_1_IN_BYTES) AS TOTAL_NB_COPY_1_BYTES_VO,"
-    "SUM(NB_COPY_NB_GT_1) AS TOTAL_NB_COPY_NB_GT_1_VO,"
-    "SUM(COPY_NB_GT_1_IN_BYTES) AS TOTAL_COPY_NB_GT_1_IN_BYTES_VO "
+    "COALESCE(SUM(TAPE.NB_MASTER_FILES),0) AS TOTAL_MASTER_FILES_VO,"
+    "COALESCE(SUM(TAPE.MASTER_DATA_IN_BYTES),0) AS TOTAL_MASTER_DATA_BYTES_VO,"
+    "COALESCE(SUM(TAPE.NB_COPY_NB_1),0) AS TOTAL_NB_COPY_1_VO,"
+    "COALESCE(SUM(TAPE.COPY_NB_1_IN_BYTES),0) AS TOTAL_NB_COPY_1_BYTES_VO,"
+    "COALESCE(SUM(TAPE.NB_COPY_NB_GT_1),0) AS TOTAL_NB_COPY_NB_GT_1_VO,"
+    "COALESCE(SUM(TAPE.COPY_NB_GT_1_IN_BYTES),0) AS TOTAL_COPY_NB_GT_1_IN_BYTES_VO "
   "FROM "
-    "TAPE "
-  "INNER JOIN "
-    "TAPE_POOL ON TAPE_POOL.TAPE_POOL_ID = TAPE.TAPE_POOL_ID "
-  "INNER JOIN "
-    "VIRTUAL_ORGANIZATION ON TAPE_POOL.VIRTUAL_ORGANIZATION_ID = VIRTUAL_ORGANIZATION.VIRTUAL_ORGANIZATION_ID "
+    "VIRTUAL_ORGANIZATION "
+  "LEFT OUTER JOIN "
+    "TAPE_POOL ON VIRTUAL_ORGANIZATION.VIRTUAL_ORGANIZATION_ID = TAPE_POOL.VIRTUAL_ORGANIZATION_ID "
+  "LEFT OUTER JOIN "
+    "TAPE ON TAPE.TAPE_POOL_ID =  TAPE_POOL.TAPE_POOL_ID "
   "GROUP BY VIRTUAL_ORGANIZATION_NAME";
   try {
     auto stmt = m_conn.createStmt(sql);