Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change tokenize cache to prevent memory leak caused by inifinite cache #19

Merged
merged 1 commit into from
Dec 10, 2021

Conversation

k-tahiro
Copy link
Contributor

@k-tahiro k-tahiro commented Dec 10, 2021

初めまして。

_tokensCacheMapが無限にキャッシュしていくため、
APIのような形式でプロセスを立ち上げたままにしておくと、どこかでheapが足りなくなると思います。

getTokenizerではなくtokenizeを使用すれば、LRUMapによる上限付きのキャッシュになるため、そちらに変更するPRです。

何か間違いがあればご指摘ください。
ご確認よろしくお願いします。

@azu
Copy link
Member

azu commented Dec 10, 2021

getTokenizerではなくtokenizeを使用すれば、LRUMapによる上限付きのキャッシュになるため、そちらに変更するPRです。

あー、確かにそうですね。
今はkuromojinの方にLRU実装したので、このライブラリレベルでは不要そうですね。
ありがとうございます。

@azu azu merged commit e363b7e into textlint-ja:master Dec 10, 2021
@azu
Copy link
Member

azu commented Dec 10, 2021

https://github.com/textlint-ja/analyze-desumasu-dearu/releases/tag/v5.0.1
でリリースしました。
patchバージョンなので、各ルールはlockファイルをアップデートすれば更新できそうです。

@k-tahiro
Copy link
Contributor Author

素早い対応ありがとうございます。

@k-tahiro k-tahiro deleted the use-tokenize-function branch December 10, 2021 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants