I am trying to take a stab at fixing the script for PROTM if there's any
issue there.
The script keeps running forever for me (or I am running out of patience
too soon), trying to check it out.

I will reach out to Briana if that's the case and try to figure it out.

Thanks & Regards,
Amogh Desai


On Tue, Oct 29, 2024 at 6:02 PM Amogh Desai <amoghdesai....@gmail.com>
wrote:

> Wow, I wasn't aware of the PR:
> https://github.com/apache/airflow/pull/42943.
>
> Removing those many lines is extremely brave. My vote goes for that one
> too!
>
>
>
> Thanks & Regards,
> Amogh Desai
>
>
> On Tue, Oct 29, 2024 at 12:55 AM Jarek Potiuk <ja...@potiuk.com> wrote:
>
>> I was sure there were a few more candidates this month - and I am
>> surprised
>> to not see them on the list (despite marking them with #protm - we
>> should likely look at the script :) ).
>>
>> My personal candidate (and my vote goes to):
>> https://github.com/apache/airflow/pull/42943 "Remove BackfillJobRunner
>> class" => with -3500 lines of code.
>>
>>
>>
>> On Mon, Oct 28, 2024 at 8:12 PM Briana Okyere
>> <briana.oky...@astronomer.io.invalid> wrote:
>>
>> > Hey All,
>> >
>> > It’s once again time to vote for the PR of the Month!
>> >
>> > With the help of the `get_important_pr_candidates` script in dev/stats,
>> > we've identified the following candidates:
>> >
>> > PR #42782: AIP-84 Migrate get connections to FastAPI API <
>> > https://github.com/apache/airflow/pull/42782>
>> >
>> > PR #43040: Masking configuration values irrelevant to DAG author <
>> > https://github.com/apache/airflow/pull/43040>
>> >
>> > PR #42080: Fix consistent return response from PubSubPullSensor <
>> > https://github.com/apache/airflow/pull/42080>
>> >
>> > PR #42959: AIP-84 | Public list tags API <
>> > https://github.com/apache/airflow/pull/42959>
>> >
>> > PR #42629: AIP-84 Migrate views /object/historical_metrics_data to Fast
>> API
>> > <https://github.com/apache/airflow/pull/42629>
>> >
>> > Please reply to this thread with your selection or offer your own
>> > nominee(s).
>> >
>> > Voting will close on Friday, Nov. 1st at 10 AM PST. The winner(s) will
>> be
>> > featured in the next issue of the Airflow newsletter.
>> >
>> > Also, if there’s an article or event that you think should be included
>> in
>> > this or a future issue of the newsletter, please drop me a line at <
>> > briana.oky...@astronomer.io>
>> >
>> > --
>> > Briana Okyere
>> > Community Manager
>> > Astronomer
>> >
>>
>

Reply via email to