search

Concordium / concordium-node

The main concordium node implementation.
GNU Affero General Public License v3.0
45 stars 22 forks source link

Revise the accountRegIdHistory of the Accounts structure #1096

Open MilkywayPirate opened 9 months ago

MilkywayPirate commented 9 months ago

In the same spirit as https://github.com/Concordium/concordium-node/issues/1040 we should also move the account registry id -> account index mapping to LMDB. (The account map lmdb database is suitable for this)

This will lower the growth rate of the database, entries in the current map are mostly on disk so it is unlikely it will have much effect on memory usage.

abizjak commented 9 months ago

As far as I remember this is not cached, so it is doubtful that removing it will decrease memory usage in normal operation.

MilkywayPirate commented 9 months ago

As far as I remember this is not cached, so it is doubtful that removing it will decrease memory usage in normal operation.

Yes, you are right. Under normal operation we should not expect reduced memory usage by this initiative. I have updated the description.