Yedidyah Bar David has posted comments on this change.
Change subject: core: rewrite buildSequence using toposort
......................................................................
Patch Set 3:
In current implementation:
Priority is not enforced:
- It's currently just a hint, simply defining the initial order.
- We do rely on it in a few cases.
- Before/After and Stage silently overrule priority.
Before/After are not enforced:
- Stage silently overrules Before/After.
It's not deterministic:
- Example: A "working" set of events can become non-working ("loop detected")
merely by removing parts of it. This is today much a larger problem than it was
in the past, because setup now has many more optional components.
--
To view, visit http://gerrit.ovirt.org/28775
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9a7e9a26ac68543331dc869ab151883ac8bd5b30
Gerrit-PatchSet: 3
Gerrit-Project: otopi
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Lev Veyde <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Simone Tiraboschi <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
