somewhat of an RFC - given that Tamas requested this, wanted to bring back this diff and discuss overall approach here
Details
Details
Diff Detail
Diff Detail
- Repository
- R1 dagster
- Branch
- task_retries
- Lint
Lint OK - Unit
No Unit Test Coverage
Event Timeline
python_modules/dagster/dagster/core/engine/retries.py | ||
---|---|---|
12–62 | hm - I feel like to get the stream of events we want we won't get away with just trying to wrap. I think we'll need to target https://dagster.phacility.com/source/dagster/browse/master/python_modules/dagster/dagster/core/engine/engine_inprocess.py$480-540 or shuffle that code around. My approximation is we'll want
| |
python_modules/dagster/dagster_tests/core_tests/engine_tests/test_retries.py | ||
28 | unrelated: having both metadata and step_metadata_fn on solid is odd - need to align the names or drop one |