PR

交易系ログの挿入クエリを調整 #2457

wada opened this at 2022-12-08T15:40:15; closed

Description

opened by wada at 2022-12-08T15:40:15; updated 2023-05-29T17:40:13

概要

重要度低のログデータを直接ログDBに入れる jolly/jr-cs-issue#197
メインDBとログDBを横断するクエリを実行するとduplicate entryエラーが発生したため、
・selectとinsertを別々のクエリに分割
・insert時にreplaceするように変更

対象テーブル

  • log_commerce_data
  • log_commerce_deck
  • log_commerce_friend_data

テスト方法、確認した内容

jrye_logのテーブル構造を最新にして、各ログデータがjrye_logのテーブルに挿入されているか
duplicate entryエラーが発生しないか

wada change_title at 2022-12-08T16:29:35: [WIP]交易系ログの挿入クエリを調整 交易系ログの挿入クエリを調整
wada add_label at 2022-12-08T16:29:37: レビュー待ち

Comment

#24318 by wada at 2022-12-08T16:30:17; updated 2022-12-08T16:30:17

@kouho
レビューお願いします。

Comment

#24332 by kouho at 2022-12-09T15:40:06; updated 2022-12-09T15:40:06

@wada
修正後dupricate key entryエラーが出ないことを確認しました。
また、dupricate key entryのエラーを再現させて問題がないことを確認したかったですが、
再現させられず、以下に確認した内容を記載しております。

修正前に交易を出して修正後に結果を見ることが何か関係するかと思いましたが関係ありませんでした。
https://docs.google.com/spreadsheets/d/1DhjKzkVjrXCdQ_e52Qayw4TPdA8uHhRxejJJ9oTUJoc/edit#gid=1512324057

kouho add_label at 2022-12-09T15:40:11: 皇甫 LGTM
wada close_comment at 2023-05-29T17:40:13: 実装から時間が経ったため一度クローズ

Commit range

base64f9caee24bf715253ef48c9c998a5136f3ba082
headb892f9724fdbc31c7fc91b044fbded97af437446
base branchjolly/jr-web:develop
head branchjolly/jr-web:update_log_commerce_query

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/SubFunc.inc
Patch body is stored as a separate file to keep this page lightweight.