The problem is that event contains references to snapshots and other large custom objects which consumes memory.

Also we need to get the extra info such as origin before we put data into buffer because in theory it might change (such as possibly when doing restore in the future).

Yet another problem is that we should not buffer events we do not want to store.

hmm, and the patch actually reduces code duplication too...