You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@airflow.apache.org by Daniel Standish <da...@astronomer.io.INVALID> on 2022/08/06 18:54:54 UTC

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

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
   - deprecate schedule_interval and timetable

PR (WIP): https://github.com/apache/airflow/pull/25410

Discussion thread [DISCUSS] consolidate dag scheduling params
<https://lists.apache.org/thread/do76d17wmt64nc7nps0ld0x1tgmo944m>.

After 72 hours without objection the proposal will be adopted.

Thanks

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

Posted by Felix Uellendall <fe...@pm.me.INVALID>.
+1 (non-binding)

Best,
Felix

Sent from Proton Mail for iOS

On Wed, Aug 10, 2022 at 19:29, Jeambrun Pierre <pi...@gmail.com> wrote:

> + 1 (non-binding)
>
> Regards,
> Pierre Jeambrun

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

Posted by Jeambrun Pierre <pi...@gmail.com>.
+ 1 (non-binding)

Regards,
Pierre Jeambrun

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

Posted by Ping Zhang <pi...@umich.edu>.
updated my vote as binding vote

+1 (binding)

Thanks,

Ping


On Tue, Aug 9, 2022 at 3:36 PM Ping Zhang <pi...@umich.edu> wrote:

> +1 (non-binding)
>
> Thanks,
>
> Ping
>
>
> On Tue, Aug 9, 2022 at 1:43 PM Drew Hubl <dr...@astronomer.io.invalid>
> wrote:
>
>> +1 (non-binding)
>>
>> On Aug 9, 2022, at 9:29 AM, Josh Fell <jo...@astronomer.io.INVALID>
>> wrote:
>>
>> +1 (binding)
>>
>> On Sat, Aug 6, 2022 at 2:55 PM Daniel Standish <
>> daniel.standish@astronomer.io.invalid> 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
>>>       release yet; only present in main)
>>>    - remove param schedule_on immediately
>>>    - deprecate schedule_interval and timetable
>>>
>>> PR (WIP): https://github.com/apache/airflow/pull/25410
>>>
>>> Discussion thread [DISCUSS] consolidate dag scheduling params
>>> <https://lists.apache.org/thread/do76d17wmt64nc7nps0ld0x1tgmo944m>.
>>>
>>> After 72 hours without objection the proposal will be adopted.
>>>
>>> Thanks
>>>
>>
>>

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

Posted by Ping Zhang <pi...@umich.edu>.
+1 (non-binding)

Thanks,

Ping


On Tue, Aug 9, 2022 at 1:43 PM Drew Hubl <dr...@astronomer.io.invalid>
wrote:

> +1 (non-binding)
>
> On Aug 9, 2022, at 9:29 AM, Josh Fell <jo...@astronomer.io.INVALID>
> wrote:
>
> +1 (binding)
>
> On Sat, Aug 6, 2022 at 2:55 PM Daniel Standish <
> daniel.standish@astronomer.io.invalid> 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 release
>>       yet; only present in main)
>>    - remove param schedule_on immediately
>>    - deprecate schedule_interval and timetable
>>
>> PR (WIP): https://github.com/apache/airflow/pull/25410
>>
>> Discussion thread [DISCUSS] consolidate dag scheduling params
>> <https://lists.apache.org/thread/do76d17wmt64nc7nps0ld0x1tgmo944m>.
>>
>> After 72 hours without objection the proposal will be adopted.
>>
>> Thanks
>>
>
>

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

Posted by Drew Hubl <dr...@astronomer.io.INVALID>.
+1 (non-binding)

> On Aug 9, 2022, at 9:29 AM, Josh Fell <jo...@astronomer.io.INVALID> wrote:
> 
> +1 (binding)
> 
> On Sat, Aug 6, 2022 at 2:55 PM Daniel Standish <da...@astronomer.io.invalid> 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 release yet; only present in main)
> remove param schedule_on immediately
> deprecate schedule_interval and timetable
> PR (WIP): https://github.com/apache/airflow/pull/25410 <https://github.com/apache/airflow/pull/25410>
> 
> Discussion thread [DISCUSS] consolidate dag scheduling params <https://lists.apache.org/thread/do76d17wmt64nc7nps0ld0x1tgmo944m>.
> 
> After 72 hours without objection the proposal will be adopted.
> 
> Thanks


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

Posted by Josh Fell <jo...@astronomer.io.INVALID>.
+1 (binding)

On Sat, Aug 6, 2022 at 2:55 PM Daniel Standish
<da...@astronomer.io.invalid> 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 release
>       yet; only present in main)
>    - remove param schedule_on immediately
>    - deprecate schedule_interval and timetable
>
> PR (WIP): https://github.com/apache/airflow/pull/25410
>
> Discussion thread [DISCUSS] consolidate dag scheduling params
> <https://lists.apache.org/thread/do76d17wmt64nc7nps0ld0x1tgmo944m>.
>
> After 72 hours without objection the proposal will be adopted.
>
> Thanks
>