PR

【12/2】ログデータを直接ログDBに挿入 #2415

wada opened this at 2022-11-17T12:26:28; closed

Description

opened by wada at 2022-11-17T12:26:28; updated 2022-12-05T15:32:46

概要

重要度低のログデータを直接ログDBに入れる jolly/jr-cs-issue#197
issueの確認事項の内容は意識せず、ログデータへ挿入する箇所のみ実装

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

ログテーブル情報 のsqlシートから、対象テーブルのsqlをjrye_logに対して実行
ゲームをプレイしてログデータがjrye_logの対象テーブルに挿入されているか
パーティションが機能しているか

Comment

#23815 by wada at 2022-11-17T12:26:39; updated 2022-11-17T12:26:39

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

Comment

#24048 by wada at 2022-11-29T10:27:56; updated 2022-11-29T10:28:48

@kouho
12/6,8あたりに本番環境へ反映したいので、今週中(12/2)にレビューしてもらえると助かります。

Comment

#24049 by kouho at 2022-11-29T10:29:06; updated 2022-11-29T10:29:06

かしこまりました。

wada change_title at 2022-11-30T10:29:29: 【11/25】ログデータを直接ログDBに挿入 【12/2】ログデータを直接ログDBに挿入

Comment

#24110 by wada at 2022-11-30T11:40:27; updated 2022-11-30T11:40:27

@kouho
交易のduplicate entryエラーについて、原因がわかれば教えてください。
レビュー期日までに原因が判明しない場合は、交易系の更新はrevertしてマージします。

Comment

#24111 by kouho at 2022-11-30T11:42:26; updated 2022-11-30T11:42:26

@wada
> 交易のduplicate entryエラーについて、原因がわかれば教えてください。
レビュー期日までに原因が判明しない場合は、交易系の更新はrevertしてマージします。

かしこまりました。現状別のアカウントでも発生しており原因は確認中です。

Comment

#24165 by kouho at 2022-12-02T10:01:30; updated 2022-12-02T10:01:30

@wada
交易のduprirate keyエラーについて再現しなくなりました。その他は確認OKです。
原因として考えられるのが環境の問題ですが、開発にマージしてデバッグしていただけたらと思います。

kouho add_label at 2022-12-02T10:01:33: 皇甫 LGTM
wada merge at 2022-12-02T13:38:34: 【12/2】ログデータを直接ログDBに挿入
wada close at 2022-12-02T13:38:34: Close
wada refer at 2022-12-05T15:32:46: 2449:【12/7】ログデータを直接ログDBに挿入

Commit range

basee622278c08424316a4a69587339785831dc6884a
heada3b1b57fb5ba01bf6a7fcf62e62880e6dbf67329
base branchjolly/jr-web:develop
head branchjolly/jr-web:change_to_log_db

View commits

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