finish the job on module variable shuffling

Author: abrookins

redisvl/index/index.py

@@ -418,11 +418,7 @@ def set_client(self, redis_client: redis.Redis, **kwargs):
             index.set_client(client)
 
         """
-        # TODO: Including required modules to match existing behavior, but we
-        # need to review.
-        RedisConnectionFactory.validate_sync_redis(
-            redis_client, required_modules=self.required_modules
-        )
+        RedisConnectionFactory.validate_sync_redis(redis_client)
         self.__redis_client = redis_client
         return self
 
@@ -931,8 +927,6 @@ async def connect(self, redis_url: Optional[str] = None, **kwargs):
             "connect() is deprecated; pass connection parameters in __init__",
             DeprecationWarning,
         )
-        # TODO: Intentionally not including required modules to match existing
-        # behavior, but we need to review.
         client = await RedisConnectionFactory._get_aredis_connection(
             redis_url=redis_url, **kwargs
         )
@@ -960,8 +954,6 @@ async def _get_client(self) -> aredis.Redis:
                     if self._redis_url:
                         kwargs["url"] = self._redis_url
                     self._redis_client = (
-                        # TODO: Intentionally not including required modules to match existing
-                        # behavior, but we need to review.
                         await RedisConnectionFactory._get_aredis_connection(**kwargs)
                     )
             await RedisConnectionFactory.validate_async_redis(