When I implemented the content hashing feature, I thought that committing hashes to Git would be a big performance improvement, but never ran performance testing. After just testing with 100, 1000, and 10000 entities, I think the performance difference is negligible and want to change this before releasing it in the wild. We should just use sha1_file on every export instead, and should probably change the \Drupal\tome_sync\ContentHasherInterface API so that getChangedContent/getDeletedContent/getNewContent all come from one method, to avoid doing this multiple times.

CommentFileSizeAuthor
#2 3054003-2.patch15.39 KBsamuel.mortenson

Comments

samuel.mortenson created an issue. See original summary.

samuel.mortenson’s picture

Status: Active » Needs review
StatusFileSize
new15.39 KB

Patch!

  • samuel.mortenson authored 3c4503e on 8.x-1.x
    Issue #3054003 by samuel.mortenson: Do not store exported content hashes...
samuel.mortenson’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.