@wburns I think you're right, we can't replace the `store` map because some other thread hold a reference to it and write into it after we passed it to the combiner. 

I wonder if it wouldn't be simpler if the size check was done on each intermediate key's list of values, instead of the entire map.