Awesome - thanks for digging all these up
Today
Today
yuhan added inline comments to D7476: monitor sensor prototype.
prha requested review of D7928: make pandas support dict-based metadata.
Yesterday
Yesterday
cdecarolis requested review of D7918: [docs] Update solid examples with context optional changes.
cdecarolis updated the test plan for D7926: [RFC] Context managers for building input/output contexts.
cdecarolis updated the summary of D7926: [RFC] Context managers for building input/output contexts.
cdecarolis published D7926: [RFC] Context managers for building input/output contexts for review.
bengotow accepted D7897: [dagit] Use Yarn v2.
Looks good to me and patches ok! Excited to give this a spin for a bit and see what else they've improved 🙌
rexledesma requested review of D7923: Add documentation for Dagster Helm Chart versioning.
sandyryza added a comment to D7476: monitor sensor prototype.
im proposing naming it pipeline_failure_sensor for now without exporting it, prototyping the above steps and see if we need a new noun
prha accepted D7878: monitor sensor 1/ rename cursor to before_cursor, add arg after_cursor and ascending to RunStorage.get_runs.
I think this looks good!
johann updated the summary of D7920: K8s executor non-step-isolated.
johann updated the summary of D7915: [RFC] Non step isolated executor.
johann requested review of D7920: K8s executor non-step-isolated.
johann retitled D7915: [RFC] Non step isolated executor from Non step isolated executor to [RFC] Non step isolated executor.
cdecarolis updated the diff for D7914: [docs] Re-add example for accessing resources within solid.
up
johann requested review of D7915: [RFC] Non step isolated executor.
[dagit] Increase hit area to expand error logs
Looks good!
dish requested review of D7917: [dagit] Increase hit area to expand error logs.
[dagit] Repair reload tooltip
dish requested review of D7916: [dagit] Repair reload tooltip.
[dagit] Link to timestamped asset from Run logs
cdecarolis requested review of D7914: [docs] Re-add example for accessing resources within solid.
spurious failure
cdecarolis updated the test plan for D7832: [dagster-dbt] re-enable asset materializations when running `dbt cli` solids.
cdecarolis updated the diff for D7832: [dagster-dbt] re-enable asset materializations when running `dbt cli` solids.
Preserve metadata for backcompat
dish requested review of D7912: [dagit] Fix config nubs.
dish requested review of D7911: [dagit] Link to timestamped asset from Run logs.
dgibson added a comment to D7901: Refactor step handler to only use ExecuteStepArgs.
"I think the abstraction is actually more useful if we limit it's parameters." - maybe this is the part I'm not following - what becomes more useful here? Is the idea that lots of step handlers will involve executing an execute_step CLI call in some environment, so it reduces boilerplate to have that all packaged up already?