В коде вызывается semaphore.acquire(), затем работа с ресурсом, затем semaphore.release() в конце метода. Что здесь опасно?
release не в finally: исключение навсегда съедает разрешение. С каждым сбоем параллелизм падает на единицу, пока система не встанет без единой записи в логе.
- acquire — до try, release — в finally
- Если acquire прерван, release вызывать нельзя — разрешение не было получено
- Тот же принцип, что lock/unlock
А ещё Метрика доступных разрешений помогает поймать утечку
Подробно
Утечка разрешений — коварная ошибка: система работает, но с каждым исключением параллелизм падает на единицу, и в какой-то момент всё встаёт намертво без единой записи в логе о причине. Тот же принцип, что и с блокировками: захват — перед try, освобождение — в finally. Дополнительно стоит помнить про acquire с таймаутом (tryAcquire), чтобы не ждать разрешения бесконечно, и про то, что acquire бросает InterruptedException, а acquireUninterruptibly — нет.
Что хотят услышать
- Утечка разрешений и последствия.
- Правильная идиома.
Частые ошибки
- Ставить acquire внутрь try — тогда finally освободит незанятое разрешение.
- Ждать разрешение без таймаута там, где важна отзывчивость.
Проверьте себя
Один верный вариант. Подумайте, потом откройте ответ.
Показать варианты и ответ
- ✗acquire нужно вызывать только из главного потока приложения
- ✗Semaphore нельзя использовать без внешнего ReentrantLock на ресурсе
- ✓release не в finally: при исключении разрешение не вернётся
- ✗Ничего опасного: семафор сам возвращает разрешение при выходе из метода
Что спросят дальше
- Где именно должен стоять acquire относительно try?
- Чем tryAcquire удобнее acquire?
Разбор темы
- Координация: семафоры, защёлки и барьеры
Semaphore, CountDownLatch, CyclicBarrier, Phaser и Exchanger — что каждый делает и когда выбирать.
Связанные вопросы
Разбор собеседования с sotto
После собеседования sotto разбирает разговор: оценка, каждый вопрос — хорошо, частично или слабо, чего не хватило в ответе и над чем поработать к следующему разу. Первые 30 минут бесплатно.