chibenwa commented on PR #2690: URL: https://github.com/apache/james-project/pull/2690#issuecomment-2765421235
> but could be the case with other actors I believe 1. it is not part of a LTS release 2. this is not an API but inner code 3. there's no known usages So let's not get our code more complicated because of this. > But as it's just a ping for the healthcheck I guess we could just have a pretty simple connection, as we do not do any operations on redis. I prefer write + delete on a sample key, and using a client rahter than PING on top of a connection and getting all the code more complex. -- 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-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
