refactor: use ULID instead of UUID4

Author: bsbodden

redisvl/index/storage.py

@@ -1,11 +1,11 @@
 import asyncio
-import uuid
 from typing import Any, Callable, Dict, Iterable, List, Optional
 
 from pydantic.v1 import BaseModel
 from redis import Redis
 from redis.asyncio import Redis as AsyncRedis
 from redis.commands.search.indexDefinition import IndexType
+from ulid import ULID
 
 from redisvl.redis.utils import convert_bytes
 
@@ -64,7 +64,7 @@ def _create_key(self, obj: Dict[str, Any], id_field: Optional[str] = None) -> st
             ValueError: If the id_field is not found in the object.
         """
         if id_field is None:
-            key_value = uuid.uuid4().hex
+            key_value = str(ULID())
         else:
             try:
                 key_value = obj[id_field]  # type: ignore

redisvl/utils/utils.py

@@ -3,15 +3,15 @@
 from functools import wraps
 from time import time
 from typing import Any, Callable, Dict, Optional
-from uuid import uuid4
 from warnings import warn
 
 from pydantic.v1 import BaseModel
+from ulid import ULID
 
 
 def create_uuid() -> str:
     """Generate a unique indentifier to group related Redis documents."""
-    return str(uuid4())
+    return str(ULID())
 
 
 def current_timestamp() -> float:

tests/unit/test_session_schema.py

@@ -1,5 +1,3 @@
-from uuid import uuid4
-
 import pytest
 from pydantic.v1 import ValidationError