スプリント会

[tatsuru]

• 進捗報告
• ふりかえり
• スプリント計画

/remind here おわり @channel in 15 minutes

/remind here qkおわり @channel in 5 minutes

[polamjag]

Sprint 0

• polamjag
  • アプリケーションマニュアル流し読みした
  • Goのソースコードちょっと見た
• pastak
  • アプリケーションマニュアル読んだ #4
  • 外にサービスが一人いるのが見どころ？
• tatsuru
  • CFn 適用して動作確認完了
  • ソースコードpushした
  • スペックはまあ普通そう

ふりかえり

• エンドポイントの数は少なそうだけどシーケンスが複雑
  • 外部サービスがいるため
  • ログインとか、なんか仕込まれてるんじゃないか

スプリント計画

• もうちょっとマニュアル眺めつつソースコードも見る
• Webから普通に触れるようにもなっているのでつついてみる

[polamjag]

Sprint 1

• tatsuru
  • レギュレーション一通り読んだ
  • Webちょっとさわった
  • データの雰囲気見た
• pastak
  • コード2/3くらい流し読みした
• polamjag
  • Webさわりながらコード眺めた

ふりかえり

• なんとなく全体像を理解してきた
  • alp の集計結果をそろそろ見たい

スプリント計画

• tatsuru
  • ログ出してalp見れるように
• pastak/polamjag
  • やれると良さそうなことをバックログに積んでいく
  • prioritizeするのはスプリント会でやるという想定

[polamjag]

Sprint 2

• pastak/polamjag
  • https://github.com/tatsuru/isucon11-qualify/issues/7
• tatsuru
  • https://github.com/tatsuru/isucon11-qualify/issues/6#issuecomment-903036283
  • ベンチ中は CPU でサチってそう

ふりかえり

• alp すぐ出せてよかった
• ロジックがテクいものがちょいちょいあって、いじったあとにベンチ落ちたらめっちゃハマりそう
  • 優先度の考慮が重要そう

スプリント計画

• alp 眺める
  • postIsuCondition
  • 全部受けなくてもスコアは出る
  • 最終的には2台とかに分散するイメージだが……
  • 存在確認に count(*) してる https://github.com/tatsuru/isucon11-qualify/blob/83c172865dd8801d6735034a806d1e7026da5972/webapp/go/main.go#L1189 → これは直す
  • getIsuList
  • N+1 https://github.com/tatsuru/isucon11-qualify/blob/83c172865dd8801d6735034a806d1e7026da5972/webapp/go/main.go#L476-L477
  • トランザクションいるか？ https://github.com/tatsuru/isucon11-qualify/blob/83c172865dd8801d6735034a806d1e7026da5972/webapp/go/main.go#L455
    • JOIN 一発にできるならいらんでしょう
• dropProbability は一旦ステイ https://github.com/tatsuru/isucon11-qualify/issues/7#issuecomment-903035030
• デプロイ出来るように

[polamjag]

Sprint 3

• tatsuru
  • もうちょいでデプロイ設定+MySQLのログ見るのができそう
• pastak/polamjag
  • done #10
  • N+1対処
  • INNER JOIN 先が isu_condition テーブルなので気まずい
    • 行数をうまく絞るにはウィンドウ関数いるんじゃないか
  • 先にインデックス貼って回るのが良いんじゃないか

ふりかえり

• wt: 作業してるけどこのペースでいいのか不安になってきた
• グラフ周りとか明らかに壮大な感じなので不安があるが脳はノッている
  • まだ全く触ってないので見当もついてない

スプリント計画

• pastak/polamjag
  • MySQL インデックス
  • getIsuList N+1 つづき
  • 終わらなかったら次のスプリント会で考える
• tatsuru
  • デプロイ
  • MySQLクエリ分析

[polamjag]

Sprint 4

• wt
  • デプロイできるようになった
  • MySQL は手つかず
• pastak/polamjag
  • 勘でインデックス貼った
  • N+1 脱出どうするか〜っていうの話してたら終わった
  • lastConditionId みたいなの UPDATE していったらいいんじゃないか、一方で condition POST するごとに UPDATE するのは気まずいのでは、というので議論してたら終わった
  • やってみないとわからなそう

ふりかえり

• インデックス追加で？？スコアが 17000 くらいまできたのは良かった
  • まあフルスキャンしてそうだったしね……
• ノッてきたのでペアプロやめてもいい説
  • T: 詰まったら無条件でペアプロ再開する

スプリント計画

• alp https://github.com/tatsuru/isucon11-qualify/issues/6#issuecomment-903043932
  • dropProbability 変えてみる？ https://github.com/tatsuru/isucon11-qualify/issues/7#issuecomment-903035030
  • /api/trend のキャッシュ
  • conditional get
  • スコアが出なくなるだけでキャッシュはめっちゃできる
  • GET /api/condition/:jia_isu_uuid 探検 → pastak
  • GET /api/isu/:jia_isu_uuid 用のインデックス改善 → polamjag
  • 分散の準備 → wtatsuru
  • GET /api/trend 眺めてキャッシュに思いを馳せる → polamjag
  • MySQL クエリ分析 → wtatsuru
  • インデックス取りこぼしとかがないか見れたらよさそう、くらいで

[polamjag]

Sprint 5

• polamjag
  • インデックスやった #13
  • GET /api/trend グローバル変数キャッシュ作戦書き始めていた
• wtatsuru
  • MariaDB のスロークエリログに苦戦
• pastak
  • GET /api/condition/:jia_isu_uuid 探検してた、方針相談したい

ふりかえり

• MariaDB 意外と勝手が違う
• ホワイトボードに文字がかけると便利
• ちゃっちゃかやらないと時間が足りない

スプリント計画

• polamjag
  • GET /api/trend グローバル変数キャッシュ続き
• wtatsuru
  • MariaDB 続き
• pastak
  • GET /api/condition/:jia_isu_uuid 探検もうすこし
  • 暇な人が現れるまで or 今スプリントじゅう

[polamjag]

Sprint 6

• wtatsuru
  • pt-query-digest 成功
  • 分散について考えていた #16
• pastak
  • GET /api/condition/:jia_isu_uuid の出題者の意図を完全に理解して解法も思いついた
• polamjag
  • GET /api/trend グローバル変数キャッシュ多分出来た #14 けどあんま効果なかった
  • 後で使えるとよいかもしれないので置いておく

ふりかえり

• pastak: condition のお膳立てに気づけてよかった
• wtatsuru: 分割わかりやすい気がするけど悩みポイントは多そう
  • POST /api/condition は明らかに分けるべき
  • テーブル単位で分けるのか、とかいろいろありそう
• pol: まあ Go 書けるなという感じになってきた (本当か?)
• dropProbability いじれてないのはまずい
  • 昼休みに回してみる

13:45 まで昼休み

---

スプリント計画

• polamjag, pastak → GET /api/condition/:jia_isu_uuid
• wtatsuru → 分散
• その後
  • 分散後の alp の結果みて考える
  • icon の MySQL 脱出
  • conditional get 整えて回る

[polamjag]

Sprint 7

• pastak, polamjag: 作戦共有して着手し始めた
• wtatsuru: POST だけ 2 に来るようにした https://github.com/tatsuru/isucon11-qualify/issues/6#issuecomment-903059325

ふりかえり

• pastak: ISUCONでしかSQLを書かないのでやばい
• wtatsuru: 分散は一つ一つは大したこと無いけど色々話題がある
• polamjag: 普段やらないことはそんなにできない

スプリント計画

• polamjag, pastak → GET /api/condition/:jia_isu_uuid 引き続き
• wtatsuru → 分散 引き続き

[polamjag]

Sprint 8

• wtatsuru: nginx レイヤーの分散とかDBを移すとかはできた https://github.com/tatsuru/isucon11-qualify/pull/18
• pastak/polamjag: GET /api/condition/:jia_isu_uuid alter table とかはできた https://github.com/tatsuru/isucon11-qualify/pull/17
  • insert するときにこれを使うのと、select 時に考慮するのはまだ

ふりかえり

• wt: DBがとにかくめっちゃくちゃ忙しいけどどうしようもないのでは
  • read replica で読み書きを分離とかはあるが、その前にDBの独立ホスト化をしたほうがよさそう
  • dropprobability 下げるとか
• pasta/pol: 悪くてもあと2スプリントくらいでケリはつきそう
  • アプリケーションが読みやすくて助かる

スプリント計画

• pastak/polamjag: condition 系カラム整理
• wtatsuru: DBの独立ホスト化と dropprobability 調整

[polamjag]

Sprint 9

• wtatsuru: nginx で conditional get 練習
• pastak/polamjag: #17 ハマっていた

ふりかえり

• 手元で書いて本番でデバッグしていてサイクルが長い
  • こんなもんという気はする
• あと3時間しか無い

スプリント計画

• pastak/polamjag: #17
• wtatsuru: conditional get と trend のキャッシュ

[polamjag]

Sprint 10

• wtatsuru: conditional get の準備はできたので試していく #19
• pastak/polamjag: #17 initialize パスするようになった

ふりかえり

• 017fd4416bfae4ddde4a88bfde3caff1747cf975 思い出せてよかった
• 試行錯誤の時間をどう確保していくかが課題そう

スプリント計画

• pastak/polamjag: #17
  • insert 時に新たなカラムを考慮するようにする (簡単)
  • select 時に考慮するようにする (少しかかりそうだがやるだけ)
• wtatsuru: conditional get 投入 + 試行錯誤

[polamjag]

Sprint 11

• wtatsuru:
  • conditional get 意味なかったので戻した
  • 3 で互換性チェックが死ぬようになって困ってる
• pastak/polamjag

  • 17 終わった気がするけど互換性チェックこけていた https://portal.isucon.net/contestant/benchmark_jobs/20362

  • そこまで変なことはしてないので直せるのではないか

ふりかえり

• 3 で互換性チェックが死ぬ理由がわからなくて不気味

• 17 進み具合はまあこんなもんかという感じ

スプリント計画

• wtatsuru: DB の分割をもうちょっと模索する
• pastak/polamjag: #17 完遂、終わったら icon アップロード

[polamjag]

Sprint 12

• pastak/polamjag #17 終わった気がするけど互換性チェックこけている https://portal.isucon.net/contestant/benchmark_jobs/22090
• wtatsuru 分散調整して34000まで

ふりかえり

• pol: 酸素が足りてない気がする
• mismatch: データが正しくありません で互換性チェックコケるのは不気味

スプリント計画

• pastak/polamjag: #17 デバッグ
• wtatsuru: 作戦考える

次qk

[polamjag]

Sprint 13

• polam/pastak #17 終わった気がする https://portal.isucon.net/contestant/benchmark_jobs/24014
• wtatsuru 画像書き出しそんな難しくなさそうなのを見ていた, DBまわり調整

ふりかえり

• 17 長かったけど終わってよかったね

  • CPUの負荷のかかり方が変わった
  • ベンチの傾向が変わってそうな雰囲気

スプリント計画

• さっきのalp https://github.com/tatsuru/isucon11-qualify/issues/6#issuecomment-903073133
• tatsuru: dropProbability調整, conditional get
• polam/pastak: iconファイル書き出し

[polamjag]

Sprint 14

• polam/pastak #20
• wtatsuru conditional get 効いてる気がする
  • /api/trend が一番高いのが悩み

ふりかえり

• 20 普通に考慮漏れっぽいのでやれば終わりそう

  • 対応できたとしてスコアどうなるかって話はある
• 最後作戦むずい
  • あと1時間

スプリント計画

• polam/pastak #20
• wtatsuru 小技試してみる

[polamjag]

Sprint 15

• 20 終わった気がする https://portal.isucon.net/contestant/benchmark_jobs/29197

ふりかえり

N/A

スプリント計画

• 分散調整したら 1.5 倍くらいは狙えそう
• wtatsuru: icon の ETag など
• pastak/polamjag: trend 眺める

[polamjag]

Sprint 16

• polam/pastak trend キャッシュ再興 #21
  • next: icon の etag
• wtatsuru 分散調整したい

[polamjag]

Sprint XX

• icon のキャッシュ成功
• nginx の分散を手で調整して 3 がちょっと仕事するようになった
  • DB ちょっと暇
  • dropProbability を調整したら忙しくできる
  • 1 番の仕事をオフロードしたい、が、ベンチが動かないので試しにくい

スプリント計画

• スコアの計測方法を再確認