Replace use of synchronized in OkHttpCall with ReentrantLock
Author: bendb-instacartCreated Feb 4, 2025Updated Jul 16, 2026
LabelsPR welcome
We use Retrofit in our backend, and I notice that `retrofit2.OkHttpCall` relies extensively on `synchronized(this)`, which can pin host threads (when using virtual threads). Would you be open to a PR replacing use of `synchronized` here with use of `ReentrantLock`?
Source: lysine-dev/retrofit