Max Leske
Max Leske
This is very interesting. I found the issue to be time spent in `FLLargeIdentityDictionary`. Replacing one particular use of `FLLargeIdentityDictionary` with `IdentityDictionary` cut time down to ~40 seconds. `FLLargeIdentityDictionary` was...
Still investigating but interesting find: in Pharo 11 the string hashes appear to be badly distributed.
I'm not sure the hash distribution is bad per se, it's just that the hashes have only uneven numbers when using modulo 4096. It looks like `FLLargeIdentityDictionary` has been slower...
Brilliant, thanks!
Just the standard "tricks": - count number of instances per class - check collection sizes With v5 I had introduced "graph culling", so you could have configured the depth of...
Well, it looks like it's time to switch to the standard collections :) I quickly checked and all tests pass using the standard collections instead of the Fuel custom ones....
> i can propose a pull request on Pharo 12 branch That would be nice. > What was the purpose of the custom collections vs. standard ones? Unfortunately, the original...
This test is still commented in 4.0.0. Either fix it or remove it.