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: query postgres extended voie #479

Merged
merged 14 commits into from
Oct 16, 2024

Conversation

fufeck
Copy link
Collaborator

@fufeck fufeck commented Sep 23, 2024

CONTEXT

Les voie mettent beaucoup trop de temps a se charger

FIX

  • Requete opti pour récupèrer le nombre de certifié et les commentaire
  • Ajout du champ bbox a l'entitie voie avec le calcule de la bbox en meme temps que le centroid

PR

BENCHMARK

Environ 1200ms pour la prod
Capture d’écran 2024-10-14 à 14 31 56

Environ 600ms pour la staging
Capture d’écran 2024-10-14 à 14 31 46

Réduction du temps de la requète de moitié

@fufeck fufeck changed the title Fufeck fix query postgres extended voie Feat: query postgres extended voie Oct 1, 2024
@fufeck fufeck requested a review from MaGOs92 October 1, 2024 14:41
Copy link
Collaborator

@MaGOs92 MaGOs92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests and code LGTM

migrations/1726841060714-bbox.ts Outdated Show resolved Hide resolved
@fufeck fufeck merged commit 863c0d3 into master Oct 16, 2024
3 checks passed
@fufeck fufeck deleted the fufeck_fix_query-postgres-extended-voie branch October 16, 2024 19:22
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.

2 participants