fix: Eager initialization of Lettuce resources outside Resource (#424)

avast · Jan 14, 2021 · 0403831 · 0403831
1 parent 950c59e
commit 0403831
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lettuce/src/main/scala/com/avast/sst/lettuce/LettuceModule.scala b/lettuce/src/main/scala/com/avast/sst/lettuce/LettuceModule.scala
@@ -14,7 +14,7 @@ object LettuceModule {
 
   /** Makes [[io.lettuce.core.RedisClient]] initialized with the given config and optionally [[io.lettuce.core.resource.ClientResources]]. */
   def makeClient[F[_]: Sync](config: LettuceConfig, clientResources: Option[ClientResources] = None): Resource[F, RedisClient] = {
-    val create = clientResources match {
+    lazy val create = clientResources match {
       case Some(resources) => RedisClient.create(resources)
       case None            => RedisClient.create()
     }

diff --git a/site/docs/subprojects/lettuce.md b/site/docs/subprojects/lettuce.md
@@ -21,7 +21,7 @@ implicit val runtime = zio.Runtime.default // this is just needed in example
 implicit val lettuceCodec: RedisCodec[String, String] = StringCodec.UTF8
 
 for {
-connection <- LettuceModule.makeConnection[Task, String, String](LettuceConfig("redis://localhost"))
-value <- Resource.liftF(Task.effect(connection.sync().get("key")))
+  connection <- LettuceModule.makeConnection[Task, String, String](LettuceConfig("redis://localhost"))
+  value <- Resource.liftF(Task.effect(connection.sync().get("key")))
 } yield value
 ```