Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Use default storage for
TemporaryRenderEntity
(#16462)
# Objective `TemporaryRenderEntity` currently uses `SparseSet` storage, but doesn't seem to fit the criteria for a component that would benefit from this. Typical usage of `TemporaryRenderEntity` (and all current usages of it in engine as far as I can tell) would be to spawn an entity with it once and then iterate over it once to despawn that entity. `SparseSet` is said to be useful for insert/removal perf at the cost of iteration perf. ## Solution Use the default table storage ## Testing Possibly this could show up in stress tests like `many_buttons`. I didn't do any benchmarking.
- Loading branch information