PR

【9/17】ヤマダapiについてログをとる #1145

wada opened this at 2021-09-16T16:29:46; closed

Description

opened by wada at 2021-09-16T16:29:46; updated 2025-07-30T10:42:12

概要

ヤマダAPIの遅延について調査する

APIの送信回数を記録

log_yamada_api_countにapiの送信回数、遅延回数を記録
送信回数と遅延回数の相関があるか調査
回数は10分区切りでカウント(00,10,20,30,40,50)

遅延内容の記録

log/yamada_api_slow_time.txtに、一定時間かかったapi送信の内容を記録
遅延が発生しやすい箇所を調査

テーブル追加

CREATE TABLE `log_yamada_api_count` (
  `date` bigint(20) unsigned NOT NULL COMMENT 'ymdH00',
  `type` tinyint(4) unsigned NOT NULL COMMENT '1:get,2:post',
  `count` int(11) unsigned NOT NULL,
  PRIMARY KEY (`date`,`type`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

TODO

その他、残した方が良いデータがあれば対応
遅延を1秒にしているが変更した方がよければ変更

wada change_title at 2021-09-16T16:30:03: 【9/16】ヤマダapiについてログをとる 【9/17】ヤマダapiについてログをとる

Comment

#13750 by wada at 2021-09-16T16:32:05; updated 2021-09-16T16:32:05

@iwamoto @kouho
レビューお願いします。
他に対応した方が良いことなどあれば教えてください。

9/17休みなので、急ぎでなければ週明けマージします。

Comment

#13759 by kouho at 2021-09-16T17:23:53; updated 2021-09-16T17:23:53

@wada
yamada_api_slow_time.txtに他にできれば欲しい情報です。

  • user_no or sap_key
  • 日時

その他

  • ログ出力のフラグ管理があれば何かの時にすぐに対応しやすいと思います。
  • 遅延を1秒にしているのを変更するかどうかは入れてみて判断で大丈夫と思います。

Comment

#13761 by wada at 2021-09-16T18:15:18; updated 2021-09-16T18:15:18

@kouho
諸々了解です。
時間と出来る範囲でsapkeyもログ出力しました。

Comment

#13762 by wada at 2021-09-16T18:19:09; updated 2021-09-16T18:19:09

開発環境の時点でgetBalance, appPointが遅い

Comment

#13763 by kouho at 2021-09-16T18:38:41; updated 2021-09-16T18:38:41

@wada
対応ありがとうございます。確認しました。OKです。

kouho add_label at 2021-09-16T18:38:47: 皇甫 LGTM

Comment

#13765 by iwamoto at 2021-09-16T18:58:11; updated 2021-09-16T18:58:11

@wada
確認しました。こちらで大丈夫と思います。

iwamoto add_label at 2021-09-16T18:58:16: 岩本 LGTM
wada merge at 2021-09-21T10:21:05: 【9/17】ヤマダapiについてログをとる
wada close at 2021-09-21T10:21:05: Close
wada refer at 2021-09-21T16:09:48: 1169:【9/22】リリース
wada refer at 2021-09-22T09:53:50: 1173:【9/22】リリース

Commit range

base4f61843fbdaf519d636871b92dee1d2fdce23738
head73e4234f6739db275e16df500e0e9122b971e0e6
base branchjolly/jr-web:develop
head branchjolly/jr-web:measure_api_time

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/class/platform/yamada/YamadaAPI.class.php
Patch body is stored as a separate file to keep this page lightweight.