search

benoitdm-oslandia / pg_featureserv

Apache License 2.0
1 stars 0 forks source link

DRAFT: poc(pg_notify/listen): working listen/notify test - [closed] #115

Closed benoitdm-oslandia closed 1 year ago

benoitdm-oslandia commented 1 year ago

In GitLab by @azarz on Oct 3, 2022, 12:06

_Merges feature/listennotify -> develop

ref #42

benoitdm-oslandia commented 1 year ago

In GitLab by @azarz on Oct 3, 2022, 15:08

Commented on internal/util/test_db.go line 125

Le faire sur le SELECT aussi ? @nrevelant

benoitdm-oslandia commented 1 year ago

In GitLab by @azarz on Oct 3, 2022, 15:15

added 1 commit

Compare with previous version

benoitdm-oslandia commented 1 year ago

In GitLab by @azarz on Oct 3, 2022, 15:30

Commented on internal/util/test_db.go line 139

Mettre le trigger avant les insert ? (déplacer l133 et l134 à l128 ?) -> cela peuplera le cache @nrevelant

benoitdm-oslandia commented 1 year ago

In GitLab by @nrevelant on Oct 3, 2022, 15:55

Commented on internal/util/test_db.go line 139

Yep! Mais est-ce que c'est simplement l'INSERT ou le COMMIT sur la transaction liée qui doit déclencher ? Je pense notamment au fait d'une erreur et un besoin de ROLLBACK. Ça signifierait qu'il faille déclencher une suppression de l'entrée en cache en cas de ROLLBACK ou qu'il faille attendre que la transaction soit COMMIT avant de peupler le cache ?

benoitdm-oslandia commented 1 year ago

In GitLab by @azarz on Oct 7, 2022, 18:12

added 1 commit

Compare with previous version

benoitdm-oslandia commented 1 year ago

il faut aussi ajouter le schéma !

benoitdm-oslandia commented 1 year ago

Ce n'est pas nécessaire. On récupérera les xmin lorsque un client fera une requête GET, sinon on va avoir le trigger qui va être appelé pour chaque SELECT mais pour ceux des appli tierces et le serveur ne va pas aimer.

benoitdm-oslandia commented 1 year ago

à faire plutôt à l'initialisation de catalog_db

benoitdm-oslandia commented 1 year ago

Par contre cela est aussi à faire à l'initialisation de catalog_db, il va falloir scanner toutes les tables des schemas. À discuter ou alors passer par un event trigger. À discuter !

benoitdm-oslandia commented 1 year ago

le trigger est AFTER donc pas de pbm de rollback

benoitdm-oslandia commented 1 year ago

Ne devrait pas être nécessaire

benoitdm-oslandia commented 1 year ago

In GitLab by @azarz on Oct 10, 2022, 11:17

Commented on internal/service/db_test/listen_test.go line 53

De toute manière ce test est voué à disparaître : je m'en sers actuellement pour vérifier le bon fonctionnement du listen, que je déplacerai (il me semble) dans une goroutine une fois le cache implémenté.

benoitdm-oslandia commented 1 year ago

Je ferme car on passe au sprint 3