Skip to content

perf: Better performance at quest play

Dosugamea requested to merge Dosugamea/server:perf/player-quest-srv into main

Summary

Some users plays quests really much. If the quest logs be bigger with them quest plays, the server had performance bottleneck at some places. This PR fixes the some of quest performance problems and make it more comfortable to play.

Related Issues

#169

Changes Made

  • Performance
    • internal\interfaces\api_player_get_all_service.go
      • Get only required fields at /api/player/get_all
    • internal/domain/model/user/quest_log_model.go
      • Add composite index for MySQL DB at UserId and QuestId

Testing

  • The performance is better than before

Checklist

  • No extra unnecessary files are included.
  • No extra fmt.Println() or fmt.Printf() statements are present.
  • The game client can accept the response (optional)

Additional Information

  • Actually, it need more database model changes since the problem is critical.
    • We have no enough man power...
Edited by Dosugamea

Merge request reports