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

Adapt Dalli 3.0 #1042

Merged
merged 1 commit into from
Nov 13, 2022
Merged

Adapt Dalli 3.0 #1042

merged 1 commit into from
Nov 13, 2022

Conversation

Nyoho
Copy link
Member

@Nyoho Nyoho commented Oct 25, 2022

WIP

#984 の対応を調べています。

  • Dalli 3 に上げるには、もう使われない :cache を消して、connection_pool という gem を入れるだけでいいのかもしれない。
  • 手元では動いた。
  • テストの書き方がわからないいので他の環境でもこれで合っているのかわからない。

@tdtds tdtds temporarily deployed to tdiary-adapt-dalli-3-ldtkqetbf October 25, 2022 13:57 Inactive
@Nyoho
Copy link
Member Author

Nyoho commented Oct 29, 2022

tdiary-coreの方針としては、dalli 3対応するときはdalli 2にも対応させて分岐させるコードにした方がいいのか、それとも3だけに対応するのでいいのか、どちらの方針でいけばいいのでしょうか。

@Nyoho
Copy link
Member Author

Nyoho commented Nov 10, 2022

afce67c, 243c734 を見ると3以降に固定して対応する方針でいいのではないかと思ったのでWIPを外してみます。方針を誤読していたらご指摘いただければ幸いです。

@Nyoho Nyoho marked this pull request as ready for review November 10, 2022 09:33
@tdtds
Copy link
Member

tdtds commented Nov 13, 2022

ありがとうございます! とりあえずマージしてみます。

@tdtds tdtds merged commit 671aa6a into master Nov 13, 2022
@tdtds tdtds deleted the adapt-dalli-3 branch November 13, 2022 04:19
@tdtds
Copy link
Member

tdtds commented Nov 29, 2022

これをベースにしてHerokuで動かそうとしてみましたが、まだダメみたいです。次のリリースではHerokuでの動作をいったん外します (freee dynoがなくなってしまったので、ニーズの方がなくなりそうですが)。

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