)]}'
{"specs/queens/approved/relationship-will_cause.rst":[{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"4af3e302bd79e8057d2c6f70c94365af0e0c4251","unresolved":false,"context_lines":[{"line_number":33,"context_line":"Proposed change"},{"line_number":34,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Define a new relationship ``will_cause``, which will be interpreted as scenarios and equivalences described above."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1f485f77_70e067cd","line":36,"range":{"start_line":36,"start_character":0,"end_line":36,"end_character":114},"updated":"2017-11-14 13:22:36.000000000","message":"I think it is very confusing. The template is made of {entities, relationships, scenarios}, and in addition we have the equivalences files. Now you are suggesting that a single relationship will be interpreted as:\nrelationship + a few scenarios + equivalence. But when you look at the scenario section you will not see it.\n\n\nI do see the benefit of having a \"shortcut\" definition. How about we introduce a new section in the template for shortcuts? (you can probably find a better name for it). Such shortcuts should be well documented, and I assume we will only have one or two types (maybe one with suspect and the other without suspect, just the raise-alarm + set-state + mark-causal-relationship).\n\n\nIn any case, I see it as something independent of the proactive RCA use case. It can help with this use case (as well as with others), but it is not mandatory.","commit_id":"3875b833c37f8a0f869b09a9bd575ca2e6497640"},{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"4af3e302bd79e8057d2c6f70c94365af0e0c4251","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"None."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"``will_cause`` is not an edge in entity graph, so no impact."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1f485f77_90064386","line":79,"range":{"start_line":79,"start_character":0,"end_line":79,"end_character":60},"updated":"2017-11-14 13:22:36.000000000","message":"Once again, having a relationship that is not really an edge is very confusing IMO.","commit_id":"3875b833c37f8a0f869b09a9bd575ca2e6497640"}]}