Page MenuHomeElementl

[5/n] Fix mapping from execution time to partition
AbandonedPublicDraft

Authored by rexledesma on Apr 29 2021, 6:16 AM.

Details

Summary

Depends on D7656.

When mapping from execution time to partition value,
we disregard the minute/hour values by setting each to 0.

This results in incorrect behavior if we format the partition
value to include those minute/hour values.

Here, we remove the override to 0, so that we defer those values to
the cron schedule minute/hour settings.

Test Plan

new pytest that failed before this change

Diff Detail

Repository
R1 dagster
Branch
rl/fix-execution-time
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 29 2021, 6:35 AM
Harbormaster failed remote builds in B29705: Diff 36478!

saving this refactor for another day