PR

【8/25】島イメージがない場合の対応 #3026

kouho opened this at 2023-08-18T11:24:35; closed

Description

opened by kouho at 2023-08-18T11:24:35; updated 2023-09-05T15:32:36

概要

DB削減対応の一環でuser_island_imageにデータがない場合に島データを生成する対応

TODO

  • [x] 2週間ログインしていないユーザーのuser_island_imageを削除するスクリプトの作成とcron追加
  • [x] 1回目の対応まとめ
  • [ ] 島画像の出力をそもそも変更することを検討(現在JPEG 品質90 ⇒ WebP 品質90)など 対応する場合は別PRを想定

1回目の対応まとめ

|総レコード|削除対象|
|--|--|
| 47209 | 40240 |

手順

1. webにuser_island_imageがない場合の更新を反映(このPull Request)
1. 削除スクリプトを手動実行。LIMIT1⇒10⇒20⇒30などタイムアウトしない範囲で増やす。LIMIT句はスクリプトの引数でもたせる。
1. 小刻みな実行が必要ならcronで夜0時~2時ぐらいの時間に実行するように何分かおきに設定して少しずつレコードを減らす
1. 削除し終えたら手順3のcronは削除もしくは書き換えて毎日の決まった時間に実行するように変更

島画像の生成確認メモ

ローカルで以下のアドレスにアクセスすれば削除されている場合にも島画像が生成
u=以下にuser_noが入っているため書き換えることで島画像の生成の確認が可能

http://127.0.0.1:20080/island.php?u=1&n=base&t=
kouho change_title at 2023-08-18T13:04:04: [WIP]島イメージがない場合の対応 島イメージがない場合の対応

Comment

#29764 by kouho at 2023-08-18T13:04:44; updated 2023-08-18T13:04:44

@wada
レビューお願い致します

Comment

#29765 by kouho at 2023-08-18T13:44:11; updated 2023-08-18T14:48:38

以下のTODOを追加

  • [ ] 島画像の出力をそもそも変更することを検討(現在JPEG 品質90 ⇒ WebP 品質90)など 対応する場合は別PRを想定

もしも島画像をWebPにした場合、ローカルで実験。1レコード427KB ⇒ 128KB  大幅に削減を確認
imagejpeg(〜 ⇒ imagewebp(〜 に書き換えることで確認可能

kouho change_title at 2023-08-21T10:08:44: 島イメージがない場合の対応 【8/25】島イメージがない場合の対応

Comment

#29816 by wada at 2023-08-22T09:52:07; updated 2023-08-22T09:52:07

@kouho
確認okです。

wada add_label at 2023-08-22T09:52:10: 和田 LGTM
kouho merge at 2023-08-25T18:34:25: 【8/25】島イメージがない場合の対応
kouho close at 2023-08-25T18:34:25: Close
kouho refer at 2023-09-01T18:49:39: 3057:島イメージがない場合の対応のリリース
kouho refer at 2023-09-05T15:32:36: 3063:【】島イメージ削除スクリプト対応修正

Commit range

basec86aa1f55fd853112e74197c9353bc4b17e82cb6
headafaa2ec1aafc66790caec27a2383cef9cab27bdc
base branchjolly/jr-web:develop
head branchjolly/jr-web:island_img_check

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed3 filesView patch
statusfile
addedlib/exec/del_user_island_image_no_login.inc
addedlib/exec/del_user_island_image_no_login.sh
modifiedpublic_html/island.php
Patch body is stored as a separate file to keep this page lightweight.