You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Pour infos, j’en ai profité pour regarder le code de BtrPlace dans VMPlaceS.
Le mapping vers un model BtrPlace me semble correct. J’ai par contre des doutes sur l’application des migrations et le respect de l’ordonnancement calculé.
Votre apply() réagit au commit des actions et non au moment de leur lancement.
Il y aura donc une inconsistence entre ce qui a été demandé et ce qui va être executé: Une action qui était planifiée pour s’exécuter au moment 0 et finir au moment 15 va commencer au moment 15.
Une solution simple mais pas encore totalement safe consisterait à juste parcourir les actions et à les lancer au bon moment en regardant Action#getStart().
Pour être safe, il faudrait considérer les dépendances entre les actions pour ne pas lancer trop tôt une action.
Le plus simple étant d’utiliser DefaultReconfigurationPlanMonitor.
The text was updated successfully, but these errors were encountered:
In French sorry :
From Fabien Hermenier:
Salut
Pour infos, j’en ai profité pour regarder le code de BtrPlace dans VMPlaceS.
Le mapping vers un model BtrPlace me semble correct. J’ai par contre des doutes sur l’application des migrations et le respect de l’ordonnancement calculé.
Votre apply() réagit au commit des actions et non au moment de leur lancement.
Il y aura donc une inconsistence entre ce qui a été demandé et ce qui va être executé: Une action qui était planifiée pour s’exécuter au moment 0 et finir au moment 15 va commencer au moment 15.
Une solution simple mais pas encore totalement safe consisterait à juste parcourir les actions et à les lancer au bon moment en regardant Action#getStart().
Pour être safe, il faudrait considérer les dépendances entre les actions pour ne pas lancer trop tôt une action.
Le plus simple étant d’utiliser DefaultReconfigurationPlanMonitor.
The text was updated successfully, but these errors were encountered: