Conversation
5811b38 to
fdc550b
Compare
fdc550b to
496d9d6
Compare
|
Maybe the argument should be optional, and default to the configured timezone? That’s what the cron expression schedule does (although we didn’t implement it at the timetable level; the default timezone is inherited from the DAG object) |
Didn't know we have that, feel like I'll need to tweak |
a4c2db5 to
3f8d15b
Compare
|
We don’t need to do it in this PR either, or even in 2.3.0 (since adding a default is entirely backward compatible). |
yep, the issue I'm now wondering is whether timezone is handled correctly in the backfilled partitioned Dag run. Will fix this first and see whether I can resolve the thing you commented |
dbd2f63 to
c673b6d
Compare
6b3c415 to
12d4a34
Compare
12d4a34 to
db5fbd9
Compare
Why
Currently,
CronPartitionedTimetabledoes not generate a timezone-aware partition key. Temporal partition mappers do not support timezone either. This might make the result misleadingWhat
Make
CronParitionedTimetablegenerate a timezone-aware partition key by default and make partition mappers parse the time info and then output a timezone aware partition keyWas generative AI tooling used to co-author this PR?
Generated-by: [Claude] following the guidelines
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.