wordpress-mobile / wordpress-mobile/WordPress-Android
Move LogFileWriter.write to a bg thread
Nobody has claimed this yet.
- Dominant language
- Kotlin
- Stars
- 3.2k
- Forks
- 1.4k
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 69
Description
@jkmassel and I will be looking at what is the best way to achieve this. Since the code is in the utils library, the solution may not be very straightforward. Furthermore, depending on how FileWriter works we might need to setup a queue since calling write function multiple times in a row might introduce issues if we depend on its internal state.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in the utils library by locating LogFileWriter.write and inspect how FileWriter handles repeated writes and internal state. Determine how background execution and any required queuing should preserve safe ordering. Done means writes no longer block the calling thread and consecutive writes remain reliable.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, kotlin
- Domain
- mobile, performance
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100