From b463ca1a2f514248f20e06a3c5e1fda6fbef7d9e Mon Sep 17 00:00:00 2001 From: chrislu Date: Tue, 20 Sep 2022 08:35:10 -0700 Subject: [PATCH] filer replication: compare content changes directly Fix https://github.com/seaweedfs/seaweedfs/issues/3714 The destination chunks may be empty. For example, the file is updated and the volume is vacuumed. In this case, the sync would miss the old chunks. This is fine. However, the entry would have correct metadata but missing chunks. For this case, the simple metadata comparison would be wrongly skipping data changes, and the file will stay empty unless file content md5 is changed. --- weed/replication/sink/filersink/filer_sink.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/weed/replication/sink/filersink/filer_sink.go b/weed/replication/sink/filersink/filer_sink.go index 06c8b1cbc..816d18a02 100644 --- a/weed/replication/sink/filersink/filer_sink.go +++ b/weed/replication/sink/filersink/filer_sink.go @@ -187,10 +187,6 @@ func (fs *FilerSink) UpdateEntry(key string, oldEntry *filer_pb.Entry, newParent // skip if already changed // this usually happens when the messages are not ordered glog.V(2).Infof("late updates %s", key) - } else if filer.ETag(newEntry) == filer.ETag(existingEntry) { - // skip if no change - // this usually happens when retrying the replication - glog.V(3).Infof("already replicated %s", key) } else { // find out what changed deletedChunks, newChunks, err := compareChunks(filer.LookupFn(fs), oldEntry, newEntry)