dragonflydb / dragonfly

A modern replacement for Redis and Memcached
https://www.dragonflydb.io/
Other
25.96k stars 954 forks source link

fix: migration ACK response processing #4140

Closed BorysTheDev closed 4 days ago

BorysTheDev commented 4 days ago

problem: the migration process can get stuck because of the pause between migration finalization attempts for https://github.com/dragonflydb/dragonfly/pull/4081

fix: try reading the correct ACK response several times until timeout happens

BorysTheDev commented 4 days ago

@chakaz When get time out we don't know if the data is lost or delayed we try to resend ack but can get the previous response in this case we try to read one more time. In the main connection can't be other data, All other data is sent via flow connections