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

Add materialized view to improve the display of meshes on map #519

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

jpm-cbna
Copy link
Contributor

@jpm-cbna jpm-cbna commented Apr 5, 2024

See: #518

JOIN atlas.t_mailles_territoire AS m
ON (o.the_geom_point && m.the_geom)
GROUP BY o.cd_ref, date_part('year', o.dateobs), m.id_maille
ORDER BY o.cd_ref, annee
Copy link
Member

Choose a reason for hiding this comment

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

Est-ce que le order by sert pour la VM ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Est-ce que le order by sert pour la VM ?

Je n'ai pas vérifié... Mais le fait d'ordonner les enregistrements par cd_ref et annee fait que la requête qui agrège les données pour générer la carte à potentiellement moins de "pages" (au sens stockage par Postgresql) à consulter pour les récupérer. Cela peut donc participer à la rapidité d’exécution de la requête.

En outre, les index comme BRIN nécessite d'ordonner les lignes. Je n'ai pas tester ce type d'index ici mais c'est possiblement plus rapide et pourrait être une solution intéressante à tester...

Enfin, malgré le ORDER BY la requête s'exécute très rapidement même avec 28 millions de données. Donc, ce n'est pas un facteur limitant.

@jpm-cbna jpm-cbna self-assigned this Apr 16, 2024
@jpm-cbna jpm-cbna marked this pull request as ready for review April 16, 2024 08:46
@TheoLechemia TheoLechemia merged commit a45bf95 into develop Apr 16, 2024
2 checks passed
@camillemonchicourt camillemonchicourt deleted the feat/improve-meshes-map branch April 16, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants