isucon / isucon11-qualify

ISUCON11 予選 (ISUCONDITION)
https://isucon.net/archives/55821036.html
MIT License
166 stars 83 forks source link

fix(jiaapi-mock) ISU UUIDのみが重複しているISU登録のリクエストが来た場合、新たにposterを立てないようにした #1321

Closed ockie1729 closed 3 years ago

ockie1729 commented 3 years ago

やったこと

対応issue

セルフチェック

備考

以下の ここ の部分には到達しない想定ですが、念の為残しています

func (m *IsuConditionPosterManager) StartPosting(targetURL *url.URL, isuUUID string) error {
    key := getKey(targetURL.String(), isuUUID)
    conflict := func() bool {
        m.activatedIsuMtx.Lock()
        defer m.activatedIsuMtx.Unlock()
        if _, ok := m.activatedIsu[key]; ok {
            return true  # ここ
        }
ShotaKitazawa commented 3 years ago

utils.go の key を isuUUID にするだけじゃダメですか?

ockie1729 commented 3 years ago

あーですね、、直します!

ockie1729 commented 3 years ago

revertするのが大変そうだったので、このPRは閉じて、あらたに #1348 を立てました