Radish alpha
h
rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5
Radicle Heartwood Protocol & Stack
Radicle
Git
node: Use refs cache to decide on fetching
Merged did:key:z6MksFqX...wzpT opened 2 years ago

When deciding whether or not to fetch a repo based on a refs announcement, we need to know whether our signed refs are stale or not, compared to the ones in the announcement. This can be quite expensive as it requires multiple reads from Git’s ODB, which often results in packfile loading.

To remedy this, we introduce a refs cache in our node database that is consulted when an announcement is received.

We also remove the logic that generates info messages, since they are rarely needed and introduced more complexity.

Additionally, we revert commit 3ad2b4431f4406fbb96c30d1edc77c5d969f17cb, which introduced logic to minimze wants and haves with additional logic, since it potentially also increases the load on the ODB.

did:key:z6MksFqX...wzpT opened with revision 78bbf988 on base 345fa57b +455 -578 2 years ago

When deciding whether or not to fetch a repo based on a refs announcement, we need to know whether our signed refs are stale or not, compared to the ones in the announcement. This can be quite expensive as it requires multiple reads from Git’s ODB, which often results in packfile loading.

To remedy this, we introduce a refs cache in our node database that is consulted when an announcement is received.

We also remove the logic that generates info messages, since they are rarely needed and introduced more complexity.

Additionally, we revert commit 3ad2b4431f4406fbb96c30d1edc77c5d969f17cb, which introduced logic to minimze wants and haves with additional logic, since it potentially also increases the load on the ODB.

did:key:z6MksFqX...wzpT pushed revision 2 7baaba00 on base 345fa57b +463 -579 2 years ago

Only cache sigrefs. So other minor changes.

did:key:z6MksFqX...wzpT pushed revision 3 c17b1c36 on base 345fa57b +472 -582 2 years ago

Don’t cache refs twice.

did:key:z6MksFqX...wzpT pushed revision 4 2da69608 on base 345fa57b +472 -582 2 years ago

Use existing local time.

did:key:z6MksFqX...wzpT merged revision 2da69608 at 256c620d 2 years ago