Re: [LAZY CONSENSUS] consolidate scheduling params with new param `schedule`

2022-08-10 Thread Felix Uellendall
+1 (non-binding) Best, Felix Sent from Proton Mail for iOS On Wed, Aug 10, 2022 at 19:29, Jeambrun Pierre wrote: > + 1 (non-binding) > > Regards, > Pierre Jeambrun

Re: [LAZY CONSENSUS] consolidate scheduling params with new param `schedule`

2022-08-10 Thread Jeambrun Pierre
+ 1 (non-binding) Regards, Pierre Jeambrun

Re: [LAZY CONSENSUS] consolidate scheduling params with new param `schedule`

2022-08-10 Thread Ping Zhang
updated my vote as binding vote +1 (binding) Thanks, Ping On Tue, Aug 9, 2022 at 3:36 PM Ping Zhang wrote: > +1 (non-binding) > > Thanks, > > Ping > > > On Tue, Aug 9, 2022 at 1:43 PM Drew Hubl > wrote: > >> +1 (non-binding) >> >> On Aug 9, 2022, at 9:29 AM, Josh Fell >> wrote: >> >> +1

Re: [LAZY CONSENSUS] consolidate scheduling params with new param `schedule`

2022-08-09 Thread Ping Zhang
+1 (non-binding) Thanks, Ping On Tue, Aug 9, 2022 at 1:43 PM Drew Hubl wrote: > +1 (non-binding) > > On Aug 9, 2022, at 9:29 AM, Josh Fell > wrote: > > +1 (binding) > > On Sat, Aug 6, 2022 at 2:55 PM Daniel Standish < > daniel.stand...@astronomer.io.invalid> wrote: > >> Proposal: >> >>-

Re: [LAZY CONSENSUS] consolidate scheduling params with new param `schedule`

2022-08-09 Thread Drew Hubl
+1 (non-binding) > On Aug 9, 2022, at 9:29 AM, Josh Fell > wrote: > > +1 (binding) > > On Sat, Aug 6, 2022 at 2:55 PM Daniel Standish > wrote: > Proposal: > add new DAG param `schedule` that accepts everything currently supported by > `schedule_interval`, `timetable`, and `schedule_on` >

Re: [LAZY CONSENSUS] consolidate scheduling params with new param `schedule`

2022-08-09 Thread Josh Fell
+1 (binding) On Sat, Aug 6, 2022 at 2:55 PM Daniel Standish wrote: > Proposal: > >- add new DAG param `schedule` that accepts everything *currently > *supported >by `schedule_interval`, `timetable`, and `schedule_on` > - (note schedule_on, added for AIP-48, has not been in any

[LAZY CONSENSUS] consolidate scheduling params with new param `schedule`

2022-08-06 Thread Daniel Standish
Proposal: - add new DAG param `schedule` that accepts everything *currently *supported by `schedule_interval`, `timetable`, and `schedule_on` - (note schedule_on, added for AIP-48, has not been in any release yet; only present in main) - remove param schedule_on immediately