search

appsgate2015 / appsgate

Appsgate
0 stars 5 forks source link

L'interpreteur doit explicitement appeler unregister de la Clock #314

Closed snybril closed 9 years ago

snybril commented 9 years ago

Je suppose que c'est dans les NodeWait, NodeEvent, ou Node (?) Lorsque l'alarme a été produite ou que le programme est arrêté, if faut appeler "unregister" sur la Clock... (stocker & utiliser le alarmID récupéré au moment de l'enregistrement)

jrcourtois commented 9 years ago

Pourquoi ???

snybril commented 9 years ago

A partir du moment où tu as enregistré une alarme, elle va être déclenchée tous les jours, tout le temps, même quand le programme est arrêté.

Quand tu redémarre n fois un programme, à chaque fois ça t'enregistre une alarme. Donc si tu as démarré 12 fois ton programme, ça va te faire 12 évènements tous les jours.

-> J'imagine bien que ces évènements ne vont rien déclencher de fâcheux, mais ils trainent...

Ca ne pose pas beaucoup de problèmes de performances quand on fait des tests et qu'on redémarre le serveur tous les jours. Par contre si ça reste allumé pendant une semaine ou plus, ça devient embêtant.

Bref, un pattern habituel de Listener, quand on a plus besoin, il faut le désenregistrer...

jrcourtois commented 9 years ago

OK, ça semble fait, donc si des erreurs apparaissent le bug sera réouvert.