adaptations nécessaires pour TOPO ?

[landryb]

il y a a priori 2 rqts faisant une jointure sur le code voie de la table voie lors de la création des vues proprietebatie et proprietenonbatie:

• https://github.com/georchestra/cadastrapp/blob/master/database/sql/vues/ProprieteBatie.sql#L91
• https://github.com/georchestra/cadastrapp/blob/master/database/sql/vues/ProprieteNonBatie.sql#L77

les champs de la table voie utilisés sont uniquement:

• natvoi et libvoi pour proprietenonbatie
• voie, natvoi et libvoi pour proprietebatie

dans le code java:

• natvoi est utilisé a uniquement 2 endroits, et il est concaténé avec dvoilib (AS voie d'ailleurs):

  • https://github.com/georchestra/cadastrapp/blob/master/cadastrapp/src/main/java/org/georchestra/cadastrapp/helper/ProprieteHelper.java#L62
  • https://github.com/georchestra/cadastrapp/blob/master/cadastrapp/src/main/java/org/georchestra/cadastrapp/helper/ProprieteHelper.java#L258

• dvoilib est utilisé a plus d'endroits, dont la recherche par voie/adresse ?

• a priori le code/la clé voie n'est pas utilisé dans le code, uniquement lors de la création des 2 vues ?

cf 3liz/QgisCadastrePlugin#345

[landryb]

la création de la vue commune dans https://github.com/georchestra/cadastrapp/blob/master/database/sql/vues/Commune.sql#L6 dit explicitement qu'elle n'utilise pas les champs clerivili et typcom mais il y'a un filtre sur typcom non null.. de toute facon, avec la conversion TOPO->FANTOIR on a bien R ou N donc pour la table commune ca ne pose a priori pas de soucis.

[landryb]

pour l'instant, modifier les 2 jointures:

• l.voie=v.voie par l.ccodep||l.ccodir||l.ccocom||l.ccoriv=v.ccodep||v.ccodir||v.ccocom||v.ccoriv
• v.voie=p.voie par v.ccodep||v.ccodir||v.ccocom||v.ccoriv=p.ccodep||p.ccodir||p.ccocom||p.ccoriv

me donne des résultats probants.

reste a vérifier avec explain si c'est plus efficace de verifier l'égalité de 2 concaténations plutot qu'utiliser des LIKE, et ca a aussi l'avantage de fonctionner pour ceux qui utilisent un fantoir de 2023... ce qui n'est pas le cas si on utilise v.voie=substr(p.voie,0,7)||'00000'||substr(p.voie,12,4) (mais qui coute moins !)

EXPLAIN a l'air de dire que le cout est le meme, qu'on utilise substr(v.voie,0,7)||substr(v.voie,12,4)=substr(p.voie,0,7)||substr(p.voie,12,4) ou v.ccodep||v.ccodir||v.ccocom||v.ccoriv=p.ccodep||p.ccodir||p.ccocom||p.ccoriv comme jointure.