natasha/yargy

Lock() в Parser

kuk opened this issue · 4 comments

kuk commented

C самой первой версии yargy в парсере создаётся и используется Lock https://github.com/natasha/yargy/blob/master/yargy/parser.py#L187 . @dveselov , это точно нужно? Я просто не очень шарю в этой теме. В своей практике вроде бы эта фича мне не пригождалась. Может убрать? Если надо, пользователь может сам создать Lock() при использовании парсера

Привет, это делает парсер thread-safe, т.е. делает его работу корректной при использовании из нескольких потоков. Можно убрать лок и добавить предупреждение, как сделано в pymorphy2.

kuk commented

Может просто в доке написать, что типа парсер не thread-safe? ))

Да, я про это и говорю)

Убрали в 1334e6d