Netflix / dynomite

A generic dynamo implementation for different k-v storage engines
Apache License 2.0
4.19k stars 531 forks source link

Add ownership information for dnode_peer/server errors and fix cross-… #731

Closed smukil closed 4 years ago

smukil commented 4 years ago

…DC checksum logic

When we hit an error trying to connect to a dnode peer or a server, we create an appropriate error message but don't link the respective connection to the error response 'msg'. This is necessary at least for processing DC_EACH_SAFE_QUORUM responses.

Also, checksums for cross-DC messages were not done properly since msg_payload_crc32() only accounted for intra-DC messages. More explained in code comments.