Edan Coben
04/13/2023, 3:01 PMTim Castillo
04/13/2023, 5:43 PMTim Castillo
04/13/2023, 5:44 PMclaire
04/13/2023, 7:40 PMTimeWindowPartitionsDefinition(cron_schedule="0 0 1 1 *")
Derek Chang
06/13/2023, 8:24 AMCronPartitionsDefinition
. This would allow users to have the flexibility to choose any partition time partitions they need. Also, a cron_partitioned_config
decorator can be created to ensure consistency with other partitioned configs. Here is the code snippet that outlines the features:claire
06/13/2023, 8:31 PMCronPartitionsDefinition
and build_schedule_from_cron_partitioned_job
are just wrappers around the existing TimeWindowPartitionsDefinition
and build_schedule_from_partitioned_job
functions, right?
I think it makes sense to introduce CronPartitionsDefinition
and cron_partitioned_config
for symmetry with HourlyPartitionsDefinition
, hourly_partitioned_config
etc. Happy to review PRs if you have the bandwidth to contribute. I think we don't need to support build_schedule_from_cron_partitioned_job
since we don't have similar functions for hourly partitioned jobs, daily partitioned jobs, etc.