Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

スプリント会 会場 #4

Open
polamjag opened this issue Nov 25, 2023 · 15 comments
Open

スプリント会 会場 #4

polamjag opened this issue Nov 25, 2023 · 15 comments

Comments

@polamjag
Copy link
Collaborator

  • 進捗報告
  • ふりかえり
  • スプリント計画
/remind here おわり @channel in 15 minutes

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

- polamjag
  -
- gurrium
  - 
- wtatsuru
  - 
- ふりかえり
  - 
- 次やること
  -
@Gurrium
Copy link
Collaborator

Gurrium commented Nov 25, 2023

  • polamjag
    • 触っていた
      • いろいろある
    • ユーザーごとに/etc/hostsにエントリを追加する必要がありそう
  • gurrium
    • マニュアル等を読んだ
    • 触っていた
  • wtatsuru
    • サーバー立ち上げた
    • nginx, go, MySQL っぽい
  • ふりかえり
    • 順調！！！
  • 次やること
    • 実装を読む
    • もっと触ってみる
    • リポジトリに入れる
    • 全体像の整理、作戦会議(11:00~)

@Gurrium
Copy link
Collaborator

Gurrium commented Nov 25, 2023

  • polamjag
    • 実装を眺めていた
      • 読みやすい
      • rangeで検索したらN+1が8箇所くらいありそう
      • モデレーション機能
  • gurrium
    • 実装を眺めていた
      • fillLivestreamResponse が結構呼ばれていたので直す候補になりそう？

  • wtatsuru

  • ふりかえり
    • 順調…？詰まってることはなさそう
  • 次やること
    • インデックスの追加
      • 対象を洗い出す
    • アクセスログ取る
    • fillLivestreamResponseの方針見る

@Gurrium
Copy link
Collaborator

Gurrium commented Nov 25, 2023

  • polamjag
    • インデックスの追加
      • 今必要そうなのには全部つけた
  • gurrium
    • 全部ざっと見た
    • fillLivestreamResponse
      • 中でさらにN+1になっていた
  • wtatsuru
    • alp出した
  • ふりかえり
    • 詰まってはなさそう
    • 作戦会議できそうなので順調！！🎉
  • 次やること
    • 作戦会議

@Gurrium
Copy link
Collaborator

Gurrium commented Nov 25, 2023

  • polamjag
    • N+1解消はもう1スプリントで終わったらいいな
  • gurrium
    • アイコンのハッシュを事前に計算するやつ
    • 次のスプリントで直せたら
  • wtatsuru
    • MySQLのスロークエリログ
    • デプロイ
    • nginxのファイルディスクリプタの数？
  • ふりかえり
    • まだ元気
      • 一方で疲れてないから休まないはアンチパターン
    • 次終わったら
  • 次やること
    • 引き続き

@Gurrium
Copy link
Collaborator

Gurrium commented Nov 25, 2023

  • polamjag
    • 次のスプリントで終わりそう
  • gurrium
    • 次のスプリントで終わりそう
  • wtatsuru
    • 画像配信を書き始めた
      • タスクを分けましょう
  • ふりかえり
    • 順調ですか？
      • ペアプロチャンス
  • 次やること
    • poloamjagドライバーでgurriumがモブ
    • 画像配信の作戦を考える
    • 一旦5分休憩

@polamjag
Copy link
Collaborator Author

  • polamjag
    • N+1 どっかで死んでることはわかったので、どこで死んでるかエラーで出せるように
  • gurrium
    • アイコンのハッシュ事前計算成功
  • wtatsuru
    • アイコンの画像nginxから配信 レビューほしい
  • ふりかえり
    • ハッシュ事前計算動いてよかった
  • 次やること
    • N+1
    • 画像の書き出し

@Gurrium
Copy link
Collaborator

Gurrium commented Nov 25, 2023

  • polamjag
    • N+1の原因を潰した。次試したい
    • これで駄目ならペアプロタイム
  • gurrium
    • 条件付きGETリクエストに対応していた
      • 304が返ってないのでなにかおかしそう
  • wtatsuru
    • nginxでのアイコン配信
      • あまりやりたくない感じ
    • 全体の作戦を眺めていた
  • ふりかえり
    • 順調？→はい
    • Copilotたまに間違える
  • 次やること
    • 昼ごはん 30m

@Gurrium
Copy link
Collaborator

Gurrium commented Nov 25, 2023

  • polamjag
    • コメントのNGワードのマッチをaho-corasickでやる
      • もうちょっと詰められそう
  • gurrium
    • moderateを見ていたむずい
  • wtatsuru
    • power dns見た
    • サーバーを分けて負荷を分散する
  • ふりかえり
    • ベンチマーカーが動かないのがつらいがどうしようもない→動いた
  • 次やること
    • ベンチマーカーを動かす
    • NGワードの更新頻度が少ないので全部メモリに乗せてもよさそう
    • statistics/moderate

@polamjag
Copy link
Collaborator Author

  • polamjag
    • N+1 そろそろ動くはず…
  • gurrium
    • なにも
  • wtatsuru
    • DBを3番で動かすように
    • DNS水責めがMySQLのCPUを食い尽くしている
      • MySQL剥がせるかも 効果高いかも
  • ふりかえり
    • wt) DNS理解した
    • pol) 一昨日練習しておいてよかった感
  • 次やること
    • モデレーション
    • unbound

@tatsuru
Copy link
Owner

tatsuru commented Nov 25, 2023

  • polamjag
    • N+1 困ってる→理解した→動いた
  • gurrium
    • ユーザー登録時に zone ファイル登録するやつ
  • wtatsuru
    • pdns 設定方法調べた
  • ふりかえり
    • statistics Redis 入れたらやれそう。クライアントからの N+1 っぽさ
    • pdns の動作確認は2人でやるとよさそう
    • 動いてよかった
  • 次やること

@tatsuru
Copy link
Owner

tatsuru commented Nov 25, 2023 •
edited

  • polamjag
    • N-gram 速くなったけど動きがまずそう
  • gurrium
    • powerdns レビュー出して修正中
  • wtatsuru
    • アプリを2で動かすと、DNSが動いているサーバーとズレるのでどうしようかな
    • proxy ごと 2で受けるのがいいかなと思うけどアクセスログ集計がうまく動かなくなりそう。困る
  • ふりかえり
    • moderate
      • ngword をメモリに載せる方法はあるんじゃないか
    • 元気
  • 次やること

@tatsuru
Copy link
Owner

tatsuru commented Nov 25, 2023

  • polamjag
    • キャッシュ実装を開始。あと2スプリントくらい
  • gurrium, wtatsuru
    • DNS実装してた
    • Go で処理するところは終わった？
    • pdns 処理周りを次にやる
  • ふりかえり
    • ngram, ngword は戻すことになった
      • collation が設定されてない状態の動作を go で再現するのがだるい
  • 次やること
    • statistics キャッシュ
    • livecomments みる
    • pdns と格闘する

@tatsuru
Copy link
Owner

tatsuru commented Nov 25, 2023

  • polamjag
    • キャッシュ実装できた。初期データの値をどうしようか => initialize で計算する
  • gurrium
    • reaction, comment 見てる。N+1 ときほぐそうとしている
    • いけそう。時間結構かかりそうだが、外から改善していけばいけるんじゃないか
      • 二重になってる
  • wtatsuru
    • powerdnsと格闘している
  • ふりかえり
    • powerdns 設定やってると壊しそうなのが怖い。サーバー壊す
  • 次やること
    • キャッシュ実装。次で終わらせる @polamjag
    • reaction の外側は剥がせる @Gurrium
    • powerdns調べる @tatsuru

@tatsuru
Copy link
Owner

tatsuru commented Nov 25, 2023

  • polamjag
    • 速くなったがスコアは上がらなかった
      • 続いてる
      • どこかのボトルネックを解消したら一気に上がるかも
  • gurrium
    • reaction の GET と N+1 を解消した
    • 中にある user, livestream の N+1 がある。後者をやろうとしている
  • wtatsuru
    • 手で動いたので自動化まで
  • ふりかえり
    • どこかのボトルネックを解消したら一気に上がるかも
    • 配信一覧の時に呼ばれる statistics 早くすると増えるんじゃないか
    • api/register 上がってきたの不気味→powerdns解消したら上がるんじゃないか
  • 次やること
    • 配信の statistics やる @polamjag
    • N+1 剥がす
    • powerdns MySQLはがし

@tatsuru
Copy link
Owner

tatsuru commented Nov 25, 2023

  • polamjag
    • livestream statistics キャッシュ化終わりつつある
  • gurrium
    • reaction 解消したと思ったが整合チェックで落ちた
  • wtatsuru
    • pdns タイミング見てベンチ回したい
  • ふりかえり
    • 休憩なしで行く
    • 再起動試験は 17:40 で
  • 次やること
    • ベンチは @Gurrium デバッグ
    • pdns MySQL剥がしやります @tatsuru
    • キャッシュ化終わったらデプロイ @polamjag

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tatsuru @polamjag @Gurrium