[Draft] [Android] Add BackgroundExecutor #205
Conversation
Force init on background thread
| /** | ||
| * A single threaded [ExecutorService] that can be used to run an specific block of code | ||
| */ | ||
| object BackgroundExecutor { |
There was a problem hiding this comment.
I like the idea of having executors definition on a separate place, I think this could be a good opportunity to organize them all, maybe something similar to what I was playing with here https://github.com/bitdriftlabs/capture-sdk/pull/128/files#diff-118bdebccae8c1bde50314ef9d78f897d272e1f222d57a5d8a44d4f0f348b5d4R13
There was a problem hiding this comment.
another example https://github.com/measure-sh/measure/blob/0721674bbba592e32b3b66d2e5a390e6c594cf89/android/measure/src/main/java/sh/measure/android/executors/ExecutorServiceRegistry.kt
There was a problem hiding this comment.
totally agree, will update
There was a problem hiding this comment.
i'm not sure if making them lazy would yield any significant improvements for start, but worth trying as well
Explorations for not blocking caller thread doing initial setup of sdk via Capture.Logger.start()
Will link doc with more context soon