What does this add to processing time in say a high event scenario? (like 10,000 events or even higher) ... obviously this hangs up a master thread for longer if slow so I think we should double check this and make sure we know how bad and/or acceptable the hit is.

If it does add too much time ... can we version check this and only apply it optionally perhaps in > 3.3.x? Can we do the filter earlier inside the inject loop perhaps avoiding the need for the resource_status_to_skipped_event_hash and the concat perhaps to shave time? Is it worth doing these kind of things anyway even if the time is small, for the sake of incremental efficiency?