Use logger.warning instead of ValueError for too many unique values in enum

Author: Zaczero

stac_fastapi/elasticsearch/stac_fastapi/elasticsearch/database_logic.py

@@ -8,11 +8,11 @@
 from typing import Any, Dict, Iterable, List, Optional, Tuple, Type, Union
 
 import attr
+from starlette.requests import Request
+
 import elasticsearch.helpers as helpers
 from elasticsearch.dsl import Q, Search
 from elasticsearch.exceptions import NotFoundError as ESNotFoundError
-from starlette.requests import Request
-
 from stac_fastapi.core.base_database_logic import BaseDatabaseLogic
 from stac_fastapi.core.serializers import CollectionSerializer, ItemSerializer
 from stac_fastapi.core.utilities import MAX_LIMIT, bbox2polygon
@@ -916,7 +916,13 @@ async def get_items_unique_values(
         result: Dict[str, List[str]] = {}
         for field, agg in query["aggregations"].items():
             if len(agg["buckets"]) > limit:
-                raise ValueError(f"Field {field} has more than {limit} unique values.")
+                logger.warning(
+                    "Skipping enum field %s: exceeds limit of %d unique values. "
+                    "Consider excluding this field from enumeration or increase the limit.",
+                    field,
+                    limit,
+                )
+                continue
             result[field] = [bucket["key"] for bucket in agg["buckets"]]
         return result
 

stac_fastapi/opensearch/stac_fastapi/opensearch/database_logic.py

@@ -925,7 +925,13 @@ async def get_items_unique_values(
         result: Dict[str, List[str]] = {}
         for field, agg in query["aggregations"].items():
             if len(agg["buckets"]) > limit:
-                raise ValueError(f"Field {field} has more than {limit} unique values.")
+                logger.warning(
+                    "Skipping enum field %s: exceeds limit of %d unique values. "
+                    "Consider excluding this field from enumeration or increase the limit.",
+                    field,
+                    limit,
+                )
+                continue
             result[field] = [bucket["key"] for bucket in agg["buckets"]]
         return result