fix NPE of cache load

Signed-off-by: Murphy <[email protected]>
murphyatwork · Dec 16, 2024 · 1b209e5 · 1b209e5
1 parent 884981f
commit 1b209e5
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/fe/fe-core/src/main/java/com/starrocks/sql/optimizer/statistics/CachedStatisticStorage.java b/fe/fe-core/src/main/java/com/starrocks/sql/optimizer/statistics/CachedStatisticStorage.java
@@ -627,7 +627,11 @@ private <K, V> Pair<List<Object>, Long> sampleFromCache(AsyncLoadingCache<K, V>
         V value = null;
         try {
             value = next.getValue().getNow(null);
-        } catch (Exception ignored) {
+        } catch (Exception e) {
+            LOG.warn("sample load statistic cache failed", e);
+        }
+        if (value == null) {
+            return Pair.create(List.of(next.getKey()), cache.synchronous().estimatedSize());
         }
         return Pair.create(List.of(next.getKey(), value), cache.synchronous().estimatedSize());
     }