Michael Dellanoce
Michael Dellanoce
i updated my PR #1300 with another potential solution
i updated #1300 with a benchmark for the 2nd style of mutation mentioned above
@wfk007 seems a little bit slower than rebuilding the list for that specific case: However, yes, I can see that the other cases were less impacted, and overall it is...
@wfk007 it looks like the first time i ran the benchmark I had this other fix applied: https://github.com/rrweb-io/rrweb/pull/1294, which gives me the 170ish result. If i do the same with...
@wfk007 I converted this PR to "draft" and added 2 commits that I've been working on. 1. On the recording side, I changed the serialization process for added nodes so...
Added another benchmark for node insertion order. This one does not _currently_ perform poorly, but it does perform poorly with the fix from #1302 so this is just meant to...
@wfk007 yes, 2 was my major concern. I agree with @colingm that 4 might be the correct trade-off. I've had the opportunity to see rrweb performing side-by-side with some of...