[PR] WIP Redis client factory refactoring [james-project]

["Arsnael (via GitHub)" <gi...@apache.org>]

Arsnael opened a new pull request, #2668:
URL: https://github.com/apache/james-project/pull/2668

   Still ongoing... mainly refactoring around backend redis tests, too much boiler plate and redundant code
   
   Also after discussing with @chibenwa , for testing ssl we can just find acceptable to test it once with standalone redis, testing with every topology is just time consuming and make the code more complex for not much gains.
   
   The `RedisSentinelHealthCheckTest` is breaking, I must have messed up some changes.
   
   Also might not be really clean enough, this needs more work (just not urgent atm)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org

Re: [PR] WIP Backend Redis test suite refactoring [james-project]

["Arsnael (via GitHub)" <gi...@apache.org>]

Arsnael closed pull request #2668: WIP Backend Redis test suite refactoring
URL: https://github.com/apache/james-project/pull/2668


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org