amplitude / Amplitude-Kotlin

Amplitude Kotlin SDK
MIT License
27 stars 10 forks source link

feat: refactor file storage to fix thread safety issues #144

Closed falconandy closed 6 months ago

falconandy commented 1 year ago

Summary

Refactored file storage to fix thread safety issues.

  1. Added unique part to file names (timestamp, file manager identifier) to eliminate possible collisions between different file manager instances.
  2. Changed internal file format to use json-event-per-line structure instead of single-line-with-json-event-array.
  3. Added a few unit tests to cover file manager logic.

Checklist

justin-fiedler commented 6 months ago

Resolved in #181