Make it clear in graph view that a disk write or disk read is a persist point

Description

- attributing this to the code generator, not sure if that's correct, but I thought this would probably be done via activity labelling and the persist name mangling.
I also thought all persist files were mangled such that they lived under scope "persist", but it seems not in the example we discussed, which used declared a persist like : PERSIST("~thor::blah")

Conclusion

None

Activity

Show:

Richard Chapman February 2, 2018 at 9:33 AM

I don't think the gain outweighs the compatibility concerns

Gavin Halliday February 2, 2018 at 8:46 AM

I think that would be sensible, but there are problems with backward compatibility.  I guess we could possible change it for 7.0 what do you think?

Jacob Cobbett-Smith January 26, 2018 at 11:33 AM

- the PR covers the graph activity label. What did you think about the other issue/observation, i.e. :

I also thought all persist files were mangled such that they lived under scope "persist", but it seems not in the example we discussed, which used declared a persist like : PERSIST("~thor::blah")

Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Components

Assignee

Reporter

Priority

Compatibility

Minor

Fix versions

Pull Request URL

Created November 17, 2017 at 1:14 PM
Updated February 2, 2018 at 9:33 AM
Resolved January 29, 2018 at 11:57 AM