Conversation
|
04:12:59.956686 初期化リクエストに成功しました 実装言語:go |
|
15:04:21.728381 初期化リクエストに成功しました 実装言語:perl |
|
16:14:35.248519 初期化リクエストに成功しました 実装言語:perl |
https://github.com/isucon/isucon12-qualify/actions/runs/2681032304 |
| use DBIx::Tracer; | ||
| use POSIX qw(strftime); | ||
| use Fcntl qw(O_CREAT O_APPEND O_WRONLY); | ||
| use JSON::PP; |
There was a problem hiding this comment.
Cpanel::JSON::XSが、現時点ではsort_byのコードブロックをサポートしていなかったので、JSON::PPを利用しています。
https://github.com/isucon/isucon12-qualify/actions/runs/2681934032 |
| statement => $statement, | ||
| args => $bind_params, | ||
| query_time => sprintf('%.6f', $query_time), | ||
| dbname => $dbname, |
There was a problem hiding this comment.
affected_rowsを取得するには、DBIx::Tracerをカスタマイズする必要があるので、一旦、対応なしにしています。
代わりに（？）に、dbnameがあった方が便利だと思ったので載せています。
https://github.com/isucon/isucon12-qualify/actions/runs/2682023708 |
https://github.com/isucon/isucon12-qualify/actions/runs/2683806177 |
| try { | ||
| $v = $self->parse_viewer($c); | ||
| } | ||
| catch ($e) { | ||
| if ($e isa Kossy::Exception && $e->{code} == HTTP_UNAUTHORIZED) { | ||
| return $c->render_json({ | ||
| status => true, | ||
| data => { | ||
| tenant => $tenant_detail, | ||
| me => undef, | ||
| role => ROLE_NONE, | ||
| logged_in => false, | ||
| } | ||
| }, MeHandlerSuccess); | ||
| } | ||
|
|
||
| fail($c, HTTP_INTERNAL_SERVER_ERROR, sprintf('error parse viewer: %s', $e)); | ||
| } |
|
|
||
| my $role = $token->{role}; | ||
| unless ($role eq ROLE_ADMIN || $role eq ROLE_ORGANIZER || $role eq ROLE_PLAYER) { | ||
| fail($c, HTTP_UNAUTHORIZED, sprintf("invalid token: %s is invalid role: %s", $role, $token_str)); |
There was a problem hiding this comment.
Goの場合はroleのキャストが成功するかわからなかったので、ここでいう$roleをエラーメッセージに乗せるのはやめましたが、Perlの場合は問題なさそうなのでこれで良さそう
mackee
left a comment
mackee
left a comment
There was a problem hiding this comment.
コメントしましたが動作には問題ない箇所です
テナント請求の計算式コメントが間違っている部分だけ最低でも直してもらえればOKです
Co-authored-by: mackee <macopy123@gmail.com>
https://github.com/isucon/isucon12-qualify/actions/runs/2697341286 |
|
本番imageを焼いてEC2でベンチしたところ、整合性チェックで以下のエラーが出ました。(後で調べます) |
https://github.com/isucon/isucon12-qualify/actions/runs/2704109202 |
https://github.com/isucon/isucon12-qualify/actions/runs/2704145039 |
https://github.com/isucon/isucon12-qualify/actions/runs/2704150051 |
3 participants
TODO