reconnect_nodes with an unordered_set

Open roystgnr opened this issue 7 years ago • 2 comments

We don't actually need an ordering here, and count/insert/etc operations in O(1) instead of O(log(N)) might be nice.

@friedmud, try this on the case from #1859 and see if it chops anything off the write_nodesets time?

This might be another pessimization instead but it's an easy enough change to be worth trying.

Oct 10 '18 02:10 roystgnr

Pinging @friedmud - got any time to check this out?

Oct 18 '18 17:10 roystgnr

This PR has been marked "do not merge" since we are no longer accepting PRs into the master branch. All new PRs should be made on the devel branch instead. Once this PR's target branch has been updated to devel, the "do not merge" label will be removed.

Mar 02 '22 20:03 jwpeterson