Madelyn Olson
Madelyn Olson
I am always a fan of free functions not doing null checks.
In cluster mode chained replicas are not considered valid, and should only exist transitively. Looks like you found a case where that invariant isn't being upheld.
I think the cron job seems like an okay mitigation for the time being. @PingXie Do you have any thoughts?
@liyaoxuan do you plan on updating this? This seems important to continue and we can find someone else to help close on it.
@hwware Sure. Should we be propagating the cluster node name to other nodes?
Yes, sorry, I meant node name.
@hwware Hey, now that the hostname extension framework is merged, do you want to follow up on this? I think it should be easy to add a new extension that...
Yeah, I don't think we should merge this document into main redis repository.
Another very real alternative to this could also be that we block and synchronously return if we were able to successfully meet node B. I think that might be a...
100% agree with @zuiderkwast, let's focus on the other PR.