PR

【】島イメージ削除スクリプト対応修正 #3063

kouho opened this at 2023-09-05T15:32:36; closed

Description

opened by kouho at 2023-09-05T15:32:36; updated 2023-09-06T18:55:50

概要

【8/25】島イメージがない場合の対応 #3026
を本番に反映させた後スクリプト実行の際にサーバー負荷が増加したため対応を修正

JOINとLIMITが同時に使えないため、別スクリプトで対象のデータを別テーブルに保存して島イメージを削除する形に修正

保存用のテーブルのCREATE文

CREATE TABLE `no_login_user` (
  `user_no` int(11) DEFAULT NULL COMMENT 'ユーザーNO',
  `reg_date` int(11) DEFAULT NULL COMMENT 'テーブル挿入時間',
  UNIQUE KEY `user_no` (`user_no`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

Comment

#30148 by kouho at 2023-09-05T15:34:33; updated 2023-09-05T15:34:33

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

Comment

#30164 by wada at 2023-09-06T09:19:20; updated 2023-09-06T09:19:20

@kouho
ユーザ取得とデータ削除のスクリプトを分けた理由はありますか?

Comment

#30184 by kouho at 2023-09-06T18:32:42; updated 2023-09-06T18:33:05

@wada
ユーザ取得とデータ削除を分ける理由がないので対応修正しました

Comment

#30185 by wada at 2023-09-06T18:48:12; updated 2023-09-06T18:48:12

@kouho
ありがとうございます。確認OKです。

wada add_label at 2023-09-06T18:48:14: 和田 LGTM
kouho merge at 2023-09-06T18:55:50: 【】島イメージ削除スクリプト対応修正
kouho close at 2023-09-06T18:55:50: Close

Commit range

baseca411db96334e6b25dde015e82e3148d01c5e0ec
headdb6b7282e538edec5a4e602274f15d58a18886ec
base branchjolly/jr-web:master
head branchjolly/jr-web:island_img_check_fix

View commits

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