fix: Lazy-import botocore exception to avoid unnecessary dependency for using vectorize module (#457)

vishal-bala · web-flow · commit 9fd5643c0ac3 · 2025-12-19T10:47:37.000+02:00
PR #452 imports an exception from botocore outside of the function where it is needed, which unintentionally adds a dependency on botocore for the package. This PR fixes this issue to avoid importing from botocore outside of the function that needs the exception.
diff --git a/redisvl/utils/vectorize/bedrock.py b/redisvl/utils/vectorize/bedrock.py
@@ -5,7 +5,6 @@
 from pathlib import Path
 from typing import TYPE_CHECKING, Any, Dict, List, Literal, Optional, Union
 
-from botocore.exceptions import ValidationError
 from pydantic import ConfigDict
 from tenacity import retry, stop_after_attempt, wait_random_exponential
 from tenacity.retry import retry_if_not_exception_type
@@ -221,6 +220,8 @@ def _embed(self, content: Any, **kwargs) -> List[float]:
             ValueError: If attempting to embed an image with a text model
             ValueError: If embedding fails
         """
+        from botocore.exceptions import ValidationError
+
         body = self._serialize_request_body(content)
 
         try:
@@ -261,6 +262,8 @@ def _embed_many(
             ValueError: If attempting to embed an image with a text model
             ValueError: If embedding fails
         """
+        from botocore.exceptions import ValidationError
+
         if not isinstance(contents, list):
             raise TypeError("`contents` must be a list")
 
