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.

Sign up for GitHub

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

Jump to bottom

Increase trade query performance by indexing database columns #151

Closed
Daniel-J-Mason opened this issue Mar 15, 2024 · 1 comment · Fixed by #152 or #161
Closed

Increase trade query performance by indexing database columns #151

Daniel-J-Mason opened this issue Mar 15, 2024 · 1 comment · Fixed by #152 or #161
Assignees
@Daniel-J-Mason
Labels
enhancement New feature or request
Milestone
trade-module

Comments

@Daniel-J-Mason
Copy link
Collaborator

Daniel-J-Mason commented Mar 15, 2024 •
edited
Loading

Current trade database is quite bare-bones. It is worth starting to index the frequently searched fields so we can improve these query response times.

Proposed indexes are:
station: arrival distance
latest_market_data: supply, demand, price
@Daniel-J-Mason Daniel-J-Mason added the enhancement New feature or request label Mar 15, 2024
@Daniel-J-Mason Daniel-J-Mason added this to the trade-module milestone Mar 15, 2024
@Daniel-J-Mason Daniel-J-Mason self-assigned this Mar 15, 2024
@Daniel-J-Mason Daniel-J-Mason moved this to 🚧 In Progress in 🔙 EDPN - Backend Mar 15, 2024
@Daniel-J-Mason
Copy link
Collaborator Author

Current trade database is quite bare-bones. It is worth starting to index the frequently searched fields so we can improve these query response times.

Proposed indexes are: station: arrival distance latest_market_data: supply, demand, price

removing max_landing_pad_size from station indexes, cardinality is too low to be useful

@github-project-automation github-project-automation bot moved this from 🚧 In Progress to 🎉 Done in 🔙 EDPN - Backend Mar 18, 2024
@Daniel-J-Mason Daniel-J-Mason linked a pull request Mar 18, 2024 that will close this issue
151 increase trade query performance #152
Merged
@Daniel-J-Mason Daniel-J-Mason reopened this Apr 1, 2024
@Daniel-J-Mason Daniel-J-Mason linked a pull request Apr 1, 2024 that will close this issue
151-2 Add additional indexes to optimize trade query #161
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Daniel-J-Mason Daniel-J-Mason

Labels
enhancement New feature or request
Projects
🔙 EDPN - Backend
Status: 🎉 Done
Milestone
trade-module
1 participant
@Daniel-J-Mason