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

あからさまに遅くなりそうなSQLクエリの見直しとインデックスを張る #127

Open
ryosan-470 opened this issue Jan 16, 2018 · 5 comments
Assignees

Comments

@ryosan-470
Copy link
Collaborator

複数のテーブルをJOINするクエリや、MySQLによる演算を行うクエリなどあからさまに重いであろうクエリが散見されているので書き直す必要がありそう。EXPLAINの結果などを貼りたい。

また、RDBMSの負荷分散のためにテーブルごとにサーバーを分けた場合、JOINは難しくなりそうなのでそこら辺も検討したい。

@HayatoDoi
Copy link
Member

HayatoDoi commented Jan 16, 2018

リクエストごとにJOINするページってあったけ?あんまりないと思う…。
この闇の深いところは、Redisにぶち込む時しか使わないので、インデックスとか張らなくても大丈夫。

https://github.com/HayatoDoi/HarekazeCTF-Competition/blob/6ae2cac5bf58dfe09eafcb950c55407f1ff9eee3/src/web/models/AnswerModel/AnswerModel.go#L94

https://github.com/HayatoDoi/HarekazeCTF-Competition/blob/6ae2cac5bf58dfe09eafcb950c55407f1ff9eee3/src/web/models/QuestionModel/QuestionModel.go#L90

@ryosan-470
Copy link
Collaborator Author

ランキング生成時にこのリクエストが呼ばれるなら改善すべきです。既にRedisのみであり、SQLクエリを呼ばないのであれば無視できますが現状そういうわけではなさそうなので、改善すべきでしょう。

@ryosan-470
Copy link
Collaborator Author

そもそも闇が深いクエリの存在自体許すべきではない気がする。

@hiww
Copy link
Member

hiww commented Jan 16, 2018

なるほど...🙇‍♂️

@HayatoDoi
Copy link
Member

ランキング生成は、Redisだけでデータ作りますね。1回だけredisに乗せるときに動くって感じです。

うーんこの僕の頭では、このクエリ以外思いつかなかったので、直せるならなおして欲しいな。

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

No branches or pull requests

3 participants