Incorporate file sizes in dvc file

[dmpetrov]

If we add file sizes in DVC-files (when we calculate checksum - so, no extra reads) it will help us to show this info in dvc diff/dvc list and other commands with no I/O or computational overhead.

Related to #2982

[shcheklein]

I like the idea! On the other hand it might complicate the merge, diffs will become bigger? Also, are there any other fields that potentially could be useful (names? modes? type?).

Just a though to consider - does it makes sense to revisit the discussion around taking md5 files hashes out? Potentially all this meta information can go into the same place.

Just a though to consider - does it makes sense to revisit the discussion around taking md5 files hashes out? Potentially all this meta information can go into the same place.

Yes 🙂

[dmpetrov]

does it makes sense to revisit the discussion around taking md5 files hashes out? Potentially all this meta information can go into the same place.

💯

[efiop]

For the record, might be solved in #1871 .

[pmrowla]

From #2982

we should store file size in addition to hashes. It will give us an ability to show file sizes in dvc diff.

• store file size together with checksums (it might be a separate issue)
• display file size in diff (like metrics diff). It might be disabled by default.