diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2024-04-18 11:16:08 +0200 |
---|---|---|
committer | Francois Kritzinger <francois@codesynthesis.com> | 2024-06-05 09:12:46 +0200 |
commit | 1ee750a80e9b7d8541e5957408c1aea87ee352bf (patch) | |
tree | 5b6459cd2110ecb160d4319e2a12c315295d3934 /mod/tenant-service.hxx | |
parent | 9bb4f1606d941aacc51626b370b84d976a836aba (diff) |
Review
Diffstat (limited to 'mod/tenant-service.hxx')
-rw-r--r-- | mod/tenant-service.hxx | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/mod/tenant-service.hxx b/mod/tenant-service.hxx index c9d0fa9..9a47459 100644 --- a/mod/tenant-service.hxx +++ b/mod/tenant-service.hxx @@ -54,15 +54,15 @@ namespace brep // the correct order, this is currently done by imposing delays (some // natural, such as building->built, and some artificial, such as // queued->building). As result, it is unlikely but possible to be notified - // about the state transitions in the wrong order, especially if the - // notifications take a long time. To minimize the chance of this happening, - // the service implementation should strive to batch the queued state - // notifications (or which there could be hundreds) in a single request if - // at all possible. Also, if supported by the third-party API, it makes - // sense for the implementation to protect against overwriting later states - // with earlier. For example, if it's possible to place a condition on a - // notification, it makes sense to only set the state to queued if none of - // the later states (e.g., building) are already in effect. + // about the state transitions in the wrong order, especially if processing + // notifications can take a long time. To minimize the chance of this + // happening, the service implementation should strive to batch the queued + // state notifications (of which there could be hundreds) in a single + // request if at all possible. Also, if supported by the third-party API, it + // makes sense for the implementation to protect against overwriting later + // states with earlier. For example, if it's possible to place a condition + // on a notification, it makes sense to only set the state to queued if none + // of the later states (e.g., building) are already in effect. // // Note also that it's possible for the build to get deleted at any stage // without any further notifications. This can happen, for example, due to |