Page MenuHomePhabricator

(3/n k8s-pod-deletion) Add step keys filter to get_run_step_stats
ClosedPublic

Authored by sashank on Oct 21 2020, 3:01 PM.

Details

Summary

This diff adds an optional step_keys argument to get_run_step_stats to allow more efficient querying for step stats. This is used in an upcoming diff where we only need step stats for a single step.

Test Plan

unit

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

sashank edited the summary of this revision. (Show Details)

up

alangenfeld added a subscriber: alangenfeld.

maybe test postgres too?

This revision is now accepted and ready to land.Oct 21 2020, 10:23 PM
python_modules/dagster/dagster/core/storage/event_log/sql_event_log.py
277–291

;)