Use a CopyOnWriteArrayList in Mediator instead of synchronized {}.

[didiergarcia]

Remove synchronize {} from all Mediator functions. Replaced the MutableList with a CopyOnWriteArrayList, that will allow multiple readers of list at once and caller that modifies the list (add,remove, etc) will get a new copy of the list that will automatically synchronized by the JVM in a safe way.

See: https://developer.android.com/reference/kotlin/java/util/concurrent/CopyOnWriteArrayList

Note: It looks to me that this has greatly sped up sending events through the timeline.

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.03 ⚠️

Comparison is base (108a98f) 78.34% compared to head (de63541) 78.32%.

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #141      +/-   ##
============================================
- Coverage     78.34%   78.32%   -0.03%     
- Complexity      492      493       +1     
============================================
  Files            73       73              
  Lines          6036     6030       -6     
  Branches        757      757              
============================================
- Hits           4729     4723       -6     
+ Misses          711      710       -1     
- Partials        596      597       +1     

Impacted Files	Coverage Δ
...segment/analytics/kotlin/core/platform/Mediator.kt	86.20% <100.00%> (-5.23%)	⬇️
...segment/analytics/kotlin/core/platform/Timeline.kt	90.90% <100.00%> (ø)
...core/platform/plugins/DestinationMetadataPlugin.kt	78.78% <0.00%> (-3.04%)	⬇️
...alytics/kotlin/core/utilities/EventsFileManager.kt	95.94% <0.00%> (+2.70%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.