Introduce IceTipRepositoryStatusModel to avoid multiple commit lookups#1918
Introduce IceTipRepositoryStatusModel to avoid multiple commit lookups#1918jecisc merged 2 commits intopharo-vcs:Pharo14from
Conversation
|
@guillep what should we do? I can integrate this fix. |
|
Some tests are failing but I think they are just failing in P13 branch and got fixed in P14. Should we integrate and port forward to P14? |
|
Yes let's merge. |
|
I have the impression that this broke a few things in Iceberg but I don't understand what is happening for now. I'll probably check this week |
|
I found the error but I need to discuss with Esteban and Guille because I think this is a Spec bug but we need to decide what is the right behavior |
|
I will revert this change for now because even with a fix in Spec, this is breaking the coloration in the repositories list |
|
Reverted in: 688b39d |
…okups" This reverts commit 688b39d.
Opening iceberg does a lot of commit lookups to show info on the browsers.
This in turn stresses a lot the GC and the ephemeron queue because all those commits are set as autorelease.
This change adds a description object where values can be cached per table.
This allows to go from 1111 to ~33 lookups in my testing environment.