red-balloon-89377
09/19/2019, 1:28 PMExecutionProcessRequests
to be able to return timing data as an extra field. This would probably take the form of ExecuteProcessResponse
having an extra field, similar to execution_attempts
, named something like timing_data
. This data would be a map, probably, and it would have the shape of {"local": "200ms", "remote": "100ms"}
.
It would be transmitted across the FFI boundary, but not cached in the V2 Process Execution Cache or the Node graph.
Does it sound “super useful, let’s do it”, or “let’s don’t edit the EPR API if we don’t need to”?hundreds-father-404
09/19/2019, 1:32 PMred-balloon-89377
09/19/2019, 2:27 PMaverage-vr-56795
09/19/2019, 2:34 PMmetadata
- we’ll need to make sure it doesn’t factor into cache keys, but I’ve also been thinking about the same thing 😄red-balloon-89377
09/19/2019, 2:48 PMtiming_metadata
average-vr-56795
09/19/2019, 2:49 PMtiming
prefix - I’m sure more metadata will come up toored-balloon-89377
09/19/2019, 2:49 PMwitty-crayon-22786
09/19/2019, 3:04 PMred-balloon-89377
09/19/2019, 3:04 PMwitty-crayon-22786
09/19/2019, 3:04 PMred-balloon-89377
09/19/2019, 3:05 PMWorkunitStore
witty-crayon-22786
09/19/2019, 3:05 PMaverage-vr-56795
09/19/2019, 3:05 PMred-balloon-89377
09/19/2019, 3:06 PMExecutionGraph
to remoting/speculation.witty-crayon-22786
09/19/2019, 3:06 PMred-balloon-89377
09/19/2019, 3:07 PMmax(remote, local)
instead of the min(remote, local)
unless we implement cancellation for local nodes.average-vr-56795
09/19/2019, 3:07 PMinclude_metadata
boolean on EPR
so that you could grab metadata where you want it…red-balloon-89377
09/19/2019, 3:07 PMwitty-crayon-22786
09/19/2019, 3:09 PMred-balloon-89377
09/19/2019, 3:14 PMExecutionGraph
a bit imprecise, but only in the boundary casewitty-crayon-22786
09/19/2019, 3:16 PMred-balloon-89377
09/19/2019, 3:21 PMQueued - Local ():
zinc(...)
zinc(...)
Queued - Remote ():
zinc(...)
....
Unstarted:
But that fundamentally opposes speculation.witty-crayon-22786
09/19/2019, 3:23 PMred-balloon-89377
09/19/2019, 3:26 PMwitty-crayon-22786
09/19/2019, 3:28 PM@rules
to be rendering UI with --v2red-balloon-89377
09/19/2019, 3:29 PMwitty-crayon-22786
09/19/2019, 3:29 PMred-balloon-89377
09/19/2019, 3:30 PMaloof-angle-91616
09/19/2019, 6:32 PMwitty-crayon-22786
09/19/2019, 6:41 PM