sor4chi / isucon13

ISUCON13 本戦のリポジトリ

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

fillLivestreamResponse の N+1

Ryoga-exe opened this issue · comments

for ループの中で fillLivestreamResponse が呼ばれているので実質的に N+1 になっていそう

livestreams := make([]Livestream, len(livestreamModels))
for i := range livestreamModels {
livestream, err := fillLivestreamResponse(ctx, tx, *livestreamModels[i])
if err != nil {
return echo.NewHTTPError(http.StatusInternalServerError, "failed to fill livestream: "+err.Error())
}
livestreams[i] = livestream
}

livestreams := make([]Livestream, len(livestreamModels))
for i := range livestreamModels {
livestream, err := fillLivestreamResponse(ctx, tx, *livestreamModels[i])
if err != nil {
return echo.NewHTTPError(http.StatusInternalServerError, "failed to fill livestream: "+err.Error())
}
livestreams[i] = livestream
}

livestreams := make([]Livestream, len(livestreamModels))
for i := range livestreamModels {
livestream, err := fillLivestreamResponse(ctx, tx, *livestreamModels[i])
if err != nil {
return echo.NewHTTPError(http.StatusInternalServerError, "failed to fill livestream: "+err.Error())
}
livestreams[i] = livestream
}