Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ベンチマーカの重複予約判定について #2

Open
btk15049 opened this issue Jul 18, 2022 · 1 comment
Open

ベンチマーカの重複予約判定について #2

btk15049 opened this issue Jul 18, 2022 · 1 comment

Comments

@btk15049
Copy link

ISUCONの練習でこちらを使わせていただいております。

高速化を進めていたところ、Level 3 あたりで以下の部分で 400 が発生するようになりました。
https://github.com/nri-isucon/nri-isucon2022/blob/main/webapp/backend/go/main.go#L1099-L1112

ログを見たところ実際に同一アクティビティへの予約が発生しており、アプリケーションの挙動としてはおかしくないものの、ベンチマーカにて減点がされていました。
アプリケーション側の仕様を変更し、予約リクエストは必ず成功させ、既に同一日への予約があってもDBへ書き込むようにするとベンチマーカの減点がなくなりました。
これらの経緯から、ベンチマーカの実装にバグがあるのではないかと疑っています。

@w2-yamaguchi
Copy link
Member

@btk15049 はじめまして、ご報告ありがとうございます。
また、練習へのご活用、ありがとうございます。

対応のお約束はできませんが、何かのタイミングで調査・対応をさせていただければと思います。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants