rcv icon indicating copy to clipboard operation
rcv copied to clipboard

optimize CVR CDF JSON generation

Open tarheel opened this issue 6 years ago • 1 comments

The main thing is that it's not at all scalable to construct the entire map in memory and then write out the JSON. We should use a streaming JSON writer instead, e.g. https://static.javadoc.io/com.google.code.gson/gson/2.6.2/com/google/gson/stream/JsonWriter.html

We might also consider combining the somewhat redundant data structures in CastVoteRecord: winnerToFractionalValue and cdfSnapshotData.

tarheel avatar Apr 04 '19 17:04 tarheel

Bright Spots call on this. If it is strictly a CDF issue, it can probably wait.

gngilbert avatar Jun 09 '19 16:06 gngilbert