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

Vues decompte objets #187

Open
wants to merge 10 commits into
base: gestiongeo
Choose a base branch
from

Conversation

romain-jault
Copy link
Contributor

@romain-jault romain-jault commented Sep 22, 2022

Créer une vue décomptant les objets actifs par cla_inu de TA_POINT_TOPO_F.

Faire un decompte par classe des objets contenues dans les tables TA_POINT_TOPO_F, TA_LIG_TOPO_F en réalisant une jointure sur la table TA_CLASSE. Cela permettrait par exemple de repérer les objets valides qui ont un cla_inu invalide.

Dans ce but deux vues ont été crées sur l'instance de test:

  • v_decompte_entite_classe_ta_point_topo_f
  • v_decompte_entite_classe_ta_lig_topo_f

Actions à mener

  • Ecrire les requêtes des vues et les tester en PROD
  • Créer les vues sur l'instance de test
  • Proposer les requêtes sur notre plate forme git
  • un reviewer a été ajouté
  • le code et les noms d'objets suivent la syntaxe

@romain-jault romain-jault added the A merger Pull request pouvant être mergée selon son auteur label Sep 22, 2022
Copy link
Contributor

@BaptisteJacq BaptisteJacq left a comment

Choose a reason for hiding this comment

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

Problèmes :

le code de tes vues décomptent tous les objets des tables de prod dont la classe est invalide, ce qui diffère du commentaire de vue.

Proposition :

Ne serait-il pas plus intéressant de décompter les objets des tables de prod par classe et ce, peu importe leur validité ? Cela permettrait notamment de savoir s'il reste des objets en prod dont la classe est invalide.



-- 2. Commentaire de la vue.
COMMENT ON TABLE GEO.V_DECOMPTE_ENTITE_CLASSE_TA_LIG_TOPO_F IS 'Vue qui comptabilise les objets de la table GEO.TA_LIG_TOPO_F par classe d''objet.';
Copy link
Contributor

Choose a reason for hiding this comment

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

précise qu'il s'agit des entités valides uniquement.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est corrigé

COMMENT ON COLUMN GEO.V_DECOMPTE_ENTITE_CLASSE_TA_POINT_TOPO_F.CLA_INU IS 'Identifiant de la classe d''objet';
COMMENT ON COLUMN GEO.V_DECOMPTE_ENTITE_CLASSE_TA_POINT_TOPO_F.CLA_CODE IS 'Code de la classe d''objet';
COMMENT ON COLUMN GEO.V_DECOMPTE_ENTITE_CLASSE_TA_POINT_TOPO_F.CLA_LI IS 'Libelle de la classe';
COMMENT ON COLUMN GEO.V_DECOMPTE_ENTITE_CLASSE_TA_POINT_TOPO_F.VALIDITE_CLASSE IS 'Validite de la classe, 1: classe valide, 0 classe non valide';
Copy link
Contributor

Choose a reason for hiding this comment

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

précise qu'il s'agit des entités valides uniquement.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

C'est corrigé

Correction des commentaires des vues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A merger Pull request pouvant être mergée selon son auteur
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants