ariatosca-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Maxim Orlov (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (ARIA-333) Remove model instrumentation code
Date Sun, 30 Jul 2017 15:28:00 GMT

    [ https://issues.apache.org/jira/browse/ARIA-333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16106531#comment-16106531
] 

Maxim Orlov edited comment on ARIA-333 at 7/30/17 3:27 PM:
-----------------------------------------------------------

please note that the instrumentation not only enables the unpacking of parameters via {{.value}},
but:
1. cascade wrapping of parameters. (e.g. {{node.outbound_relationships[0].source_node.attributes}}
will still be instrumented)
2. propagation of changes. (i.e. updating of a subdict/sublist should be reflected in the
model as well).


was (Author: mxmrlv):
please note that the instrumentation not only enables the unpacking of parameters via {{ .value
}}, but:
1. cascade wrapping of parameters. (e.g. {{ node.outbound_relationships[0].source_node.attributes
}} will still be instrumented)
2. propagation of changes. (i.e. updating of a subdict/sublist should be reflected in the
model as well).

> Remove model instrumentation code
> ---------------------------------
>
>                 Key: ARIA-333
>                 URL: https://issues.apache.org/jira/browse/ARIA-333
>             Project: AriaTosca
>          Issue Type: Story
>            Reporter: Tal Liron
>
> Our model instrumentation code is some of the most complex part of our codebase. It is
difficult to understand and maintain.
> Why do we have this complex mechanism? Because we want it to be easier to access values
in parameters dicts without having to add {{.value}}.
> In the case of the ctx proxy, there is a much simpler solution. We could just check if
the final value is a {{models.Parameter}}, and if so unpack the value from it.
> For uses of ctx within Python there indeed is no simple solution. However, in my opinion
the solution we chose is far too complex, and the cons far outweigh the pros.
> In the end, it's really seems not too much to expect a Python programmer to add {{.value}}
where necessary.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message