Skip to content

Commit

Permalink
Merge pull request #59 from Kazark/rm-unused-param
Browse files Browse the repository at this point in the history
refactor(core): remove unused parameter in private method
  • Loading branch information
ChristopherDavenport authored Mar 29, 2023
2 parents fcd3720 + 28db7f2 commit 8e19d7f
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,8 @@ object RedisConnection{

// Guarantees With Socket That Each Call Receives a Response
// Chunk must be non-empty but to do so incurs a penalty
private[rediculous] def explicitPipelineRequest[F[_]: Concurrent](socket: Socket[F], calls: Chunk[Resp], maxBytes: Int = 16 * 1024 * 1024, timeout: Option[FiniteDuration] = 5.seconds.some): F[Chunk[Resp]] = {
val out = calls.flatMap(resp =>
private[rediculous] def explicitPipelineRequest[F[_]: Concurrent](socket: Socket[F], calls: Chunk[Resp], maxBytes: Int = 16 * 1024 * 1024): F[Chunk[Resp]] = {
val out = calls.flatMap(resp =>
Resp.CodecUtils.codec.encode(resp).toEither.traverse(bits => Chunk.byteVector(bits.bytes))
).sequence.leftMap(err => new Throwable(s"Failed To Encode Response $err")).liftTo[F]
out.flatMap(socket.write) >>
Expand Down

0 comments on commit 8e19d7f

Please sign in to comment.