Cache account ids?

[maltekliemann]

Regarding account_id(), the treasury pallet's documentation says the following about into_sub_account:

This actually does computation. If you need to keep using it, then make sure you cache the value and only call this once. [link]

We use this type of account generation in grand fashion. Should we cache market/pool accounts and trade chain storage for computation time?

[sea212]

Absolutely. Let's keep it a low priority for now because it is an optimizations that is not needed right now, but eventually will be.