19913d85-f662-4518-a9a1-7adc1793dfd4.png){kind=small}
Channels
#announcements
- s
schrockn
08/28/2018, 12:15 PMno compute node/no logical execution plan - s
schrockn
08/28/2018, 12:15 PM(I call the compute node graph the logical execution plan interchangably) - s
schrockn
08/28/2018, 12:15 PM2) I added the compute node stuff while keeping the execution API (relatively) constant - s
schrockn
08/28/2018, 12:15 PM3) I never consolidated them - s
schrockn
08/28/2018, 12:15 PMhowever - s
schrockn
08/28/2018, 12:16 PMnot still convinced they should be the same - s
schrockn
08/28/2018, 12:16 PMit depends on what we want to report to API consumers - s
schrockn
08/28/2018, 12:16 PMe.g. should the compute node stuff be considered a pure implementation detail? - s
schrockn
08/28/2018, 12:16 PMso I kicked the can down the road on that q - f
freiksenet
08/28/2018, 12:17 PMso with current impl, there is nothing added by execution result - f
freiksenet
08/28/2018, 12:17 PMbut - f
freiksenet
08/28/2018, 12:17 PMone thing that could be done is unifying expectations with their corresponding solids - s
schrockn
08/28/2018, 12:17 PMexactly - f
freiksenet
08/28/2018, 12:17 PMwhich could be better for the user consumption - s
schrockn
08/28/2018, 12:17 PMalso - s
schrockn
08/28/2018, 12:17 PMmultiple outputs - f
freiksenet
08/28/2018, 12:18 PMI don't quite get how ti works with compute node - s
schrockn
08/28/2018, 12:18 PMwhich part - f
freiksenet
08/28/2018, 12:18 PMmultiple outputs - f
freiksenet
08/28/2018, 12:19 PMit yields multiple compute node results? - s
schrockn
08/28/2018, 12:19 PMya - f
freiksenet
08/28/2018, 12:19 PMso we have multilple execution step results? - s
schrockn
08/28/2018, 12:20 PMyup. one of the most broken parts of the API right now - f
freiksenet
08/28/2018, 12:20 PMok - f
freiksenet
08/28/2018, 12:20 PMso I think we def keep PipelineResult - f
freiksenet
08/28/2018, 12:21 PMPipeline result should be able to get results per solid - f
freiksenet
08/28/2018, 12:21 PMand then that should be some wrapper that holds all solid outputs and expectation results associated with solid - s
schrockn
08/28/2018, 12:21 PMyes that makes sense - s
schrockn
08/28/2018, 12:22 PMthe results coming out of execution.py - f
freiksenet
08/28/2018, 12:22 PMI now agree that compute nodes shoulds be distinc