Skip to content

[ブランチを内部プロジェクトから参照しているので削除しない] LRUキャッシュを利用しない変更をいれた#1

Open
alpaca-tc wants to merge 3 commits into
masterfrom
default-no-cache
Open

[ブランチを内部プロジェクトから参照しているので削除しない] LRUキャッシュを利用しない変更をいれた#1
alpaca-tc wants to merge 3 commits into
masterfrom
default-no-cache

Conversation

@alpaca-tc
Copy link
Copy Markdown
Collaborator

やったこと

  • LRUキャッシュを利用するかを決める noCacheTokenize のオプションをデフォルトで undefined から true に変更した
    • 既存の実装だと10000件のLRUキャッシュを利用する実装だったが、これがメモリを喰うしとても遅かった。
  • 他の変更は、ライブラリをhttps指定で利用できるようにする変更
    • .githooksディレクトリを削除した。インストール時にhooksを指定する必要はないため。
    • libディレクトリを追加した。リリース用のファイルとして提供するため。

次やりたい

  • textlint-preset-rule-smarthrで参照しているすべてのライブラリで、kuromojinのバージョンを揃えたい。
    • 揃えられていない場合、トークナイザーを別々のインスタンスに生成するため、ただでさえメモリ使用量の多いインスタンスが大量にできてしまうため。

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.

1 participant