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

feat: improve mongo indexes #77

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

mtlmatt
Copy link
Contributor

@mtlmatt mtlmatt commented Jul 9, 2024

No description provided.

@mtlmatt mtlmatt requested a review from beltschatsar July 9, 2024 18:16
@mtlmatt mtlmatt force-pushed the feature/mongo-performance-improvement branch from 486de91 to 9acd4e6 Compare July 9, 2024 18:24
@mtlmatt mtlmatt force-pushed the feature/mongo-performance-improvement branch from 9acd4e6 to be2949e Compare July 10, 2024 12:20
@steph-lebl
Copy link
Contributor

steph-lebl commented Jul 10, 2024

Peux-tu stp ajuster les tests de charge pour que le nouveau standard soit 90 req/sec?

Je n'ai pas vraiment fait de revue, je regardais le code pour un exemple pour un autre projet et j'ai remarqué cela en passant. Je laisse Daniel faire la revue.

@mtlmatt
Copy link
Contributor Author

mtlmatt commented Jul 11, 2024

Peux-tu stp ajuster les tests de charge pour que le nouveau standard soit 90 req/sec?

Je n'ai pas vraiment fait de revue, je regardais le code pour un exemple pour un autre projet et j'ai remarqué cela en passant. Je laisse Daniel faire la revue.

Une PR est en cours pour les ajustements aux tests de charge et le nouveau standard est 90rps pour 2h => https://github.com/VilledeMontreal/montreal-taxi-registry/pull/76/files#diff-6ad7896087100fdcba54fefb5a0c52c96d25ef08f9be51d0a17e92e4ce178790

@mtlmatt mtlmatt merged commit 0ff775c into develop Jul 17, 2024
3 checks passed
@mtlmatt mtlmatt deleted the feature/mongo-performance-improvement branch July 17, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants