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

Author: Zaczero

stac_fastapi/elasticsearch/stac_fastapi/elasticsearch/database_logic.py

@@ -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