]> gerrit.simantics Code Review - simantics/platform.git/commit
Avoid useless reallocation of empty TreeMaps and map iterators 80/4280/1
authorTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Thu, 4 Jun 2020 22:33:16 +0000 (01:33 +0300)
committerTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Thu, 4 Jun 2020 23:20:41 +0000 (23:20 +0000)
commite0dc6bf7e22eac19c4764a2210eb83a8a66388ca
treeee6e9b5c97825d1fc686c74a55801dc3b589c32d
parentd162802f90f0d913d53c0244505a023b6c0f08cd
Avoid useless reallocation of empty TreeMaps and map iterators

The problem was that when the database is doing work, MainProgram.run
runs in pretty much busy loop mode and previously it was always
allocating a new TreeMap on each round and also calling Map.entrySet()
to get construct iterators for empty TreeMaps which eventually
accumulates up to somewhat signinificant amount of memory allocated.

Possibly an even more efficient way would be to have the Map be a closed
hashing hashmap instead and then sorting the data for iteration
separately.

gitlab #548

Change-Id: Ib2208dc35b270c9d682362d45f24f1fe01bb8969
(cherry picked from commit d3f7c0196ecf58b78574d8b80e1844ecb4e223d6)
bundles/org.simantics.acorn/src/org/simantics/acorn/MainProgram.java