Python: словарь с надежными словарями с короткими ключами, это правильно?

Question

Предполагает, что do_thing() много раз используется для многих потоков с номерами идентификаторов, которые могут быть или не быть одинаковыми (скажем, 4 раза с идентификатором 3 и один раз с разными идентификаторами), является ли этот поток безопасным? Будет ли one_time_init() когда-либо запускаться более одного раза для определенного идентификатора задания одновременно? (PS: one_time_init сохраняет это состояние, которое было запустить один раз для каждого идентификатора, поэтому вызов не является не-оп, если он уже бежать к завершению)

---

Обновленный код (спасибо THC4k):

import threading 
import weakref 
_mainlock = threading.RLock() 
_job_locks = weakref.WeakValueDictionary() 

def do_thing(job_id): 
    with _mainlock: 
     jl = _job_locks.setdefault(job_id, threading.RLock()) 
    with jl: 
     one_time_init(job_id) 

Answer 1

Похоже, очень безопасный. Зачем вам нужен _job_locks, если one_time_init еще раз проверяет, был ли он запущен? Вы можете добавить замок. Почему RLock вместо Lock (функция, кажется, никогда не вводится повторно)?

В любом случае, with заявление выглядит способ лучше:

import threading 
import weakref 
_mainlock = threading.RLock() 
_job_locks = weakref.WeakValueDictionary() 

def do_thing(job_id): 
    with _mainlock: 
     _job_locks.setdefault(job_id, threading.RLock()) 
    with _job_locks[job_id]: 
     one_time_init(job_id)