création de requêtes permettant la maj des statistiques des tables gestiongeo et leur sélection

meldig / SQL

1 stars 3 forks source link

création de requêtes permettant la maj des statistiques des tables gestiongeo et leur sélection #127

Closed BaptisteJacq closed 3 years ago

BaptisteJacq commented 3 years ago

Objectif

Mise à jour des statistiques des tables de gestiongeo ;
Sélection des champs permettant une étude statistique de ces tables et de leur utilisation ;

Actions à mener

[x] un reviewer a été ajouté
[x] le code et les noms d'objets suivent la syntaxe

BaptisteJacq commented 3 years ago

Je viens de vérifier et ma requête utilise bien 100% des données de chaque table pour faire la mise à jour des statistiques. Pour cela j'ai comparé les statistiques issues de la requête de la PR avec celles renvoyées par le code ci-dessous pour la table TA_GG_DOSSIER (en DEV) et j'obtiens exactement les mêmes valeurs :

begin DBMS_STATS.GATHER_TABLE_STATS ( ownname => '"GEO"', tabname => '"TA_GG_DOSSIER"', estimate_percent => 100 ); end;

BaptisteJacq commented 3 years ago

Je viens de regarder la documentation sur le champ density et en fait il ne s'agit pas du tout de la densité d'utilisation du champ, mais d'une estimation permettant de savoir si ce champ est intéressant pour faire des conditions dessus :

1 : ce champ permet de faire des requêtes très sélectives ;
0 : ce champ n'a aucun intérêt à être utilisé dans la condition d'une sélection ; Voici la doc à partir de laquelle je fais cette analyse :
- https://docs.oracle.com/en/database/oracle/oracle-database/12.2/refrn/ALL_TAB_COLUMNS.html#GUID-F218205C-7D76-4A83-8691-BFD2AD372B63 ;
- https://datacadamia.com/db/oracle/statistics/density ;

Du coup je vais l'enlever de ma requête.

BaptisteJacq commented 3 years ago

Cette PR doit être mergée après la PR 128, sans quoi, je pense que des conflits pourraient apparaître : La PR 128 sert normalement à mettre à jour le fichier "Statistiques_tables_gestion_geo", mais les dernières modifications ne s'y trouvent pas. Elles se trouvent dans cette PR 127 (une erreur de ma part en voulant aller trop vite).

Jean-Roc commented 3 years ago

Je te laisse m'indiquer quand faire le merge.

Après c'est l’intérêt de travailler en multipliant les branches en local ça te permet de rebaser l'une par rapport à l'autre pour éviter les conflits et d'avoir un ordre dans tes PR

BaptisteJacq commented 3 years ago

En effet. C'est corrigé, tu peux merger.