Merge pull request #1186 from msvinaykumar/listRecommendationfix

minor change
kruize · Apr 24, 2024 · 41a536d · 41a536d
2 parents d0c8571 + 886915d
commit 41a536d
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/main/java/com/autotune/analyzer/recommendations/engine/RecommendationEngine.java b/src/main/java/com/autotune/analyzer/recommendations/engine/RecommendationEngine.java
@@ -1407,7 +1407,7 @@ private String getResults(Map<String, KruizeObject> mainKruizeExperimentMAP, Kru
      * @param interval_start_time The start time of the interval for fetching metrics.
      * @param dataSourceInfo      The datasource object to fetch metrics from.
      * @throws Exception if an error occurs during the fetching process.
-     *                   TODO: Need to add right abstractions for this
+     *                                                                                                                               TODO: Need to add right abstractions for this
      */
     public void fetchMetricsBasedOnDatasource(KruizeObject kruizeObject, Timestamp interval_end_time, Timestamp interval_start_time, DataSourceInfo dataSourceInfo) throws Exception {
         try {
@@ -1570,7 +1570,8 @@ public void fetchMetricsBasedOnDatasource(KruizeObject kruizeObject, Timestamp i
                         }
                     }
                     containerData.setResults(containerDataResults);
-                    setInterval_end_time(Collections.max(containerDataResults.keySet()));    //TODO Temp fix invalide date is set if experiment having two container with different last seen date
+                    if (containerDataResults.size() > 0)
+                        setInterval_end_time(Collections.max(containerDataResults.keySet()));    //TODO Temp fix invalide date is set if experiment having two container with different last seen date
                 }
             }
         } catch (Exception e) {