search

fire-eggs / YAGP

"Yet Another GEDCOM Parser" - newer/faster/complete GEDCOM parser in C#
Apache License 2.0
9 stars 3 forks source link

Memory: surnames #39

Open fire-eggs opened 6 years ago

fire-eggs commented 6 years ago

dotMemory tracing found repetition of surnames. E.g. for (TomsTreePriv.ged?) the surname "Skillman" was repeated 12,996 times

Possible candidate for lookup table? See issue #38

fire-eggs commented 6 years ago

Preliminary testing suggests the benefit of caching (e.g.) surnames is outweighed by the cost.

There is a memory overhead from the cache proper, especially from the default Dictionary expansion allocation. There is the performance cost of the cache lookup, aggravated by parallelism support.

I'm going to leave this open for now, pending further measurement and experiments.