Hi All, The Vote passed out close with a 3-way tie! So I'll get all of them in there, just a little condensed
Best, John On Fri, Oct 28, 2022 at 7:41 AM Jarek Potiuk <[email protected]> wrote: > I was out most of October, but from my catch-up I have one favourite > candidate (actually two-in-one): > > Fix IntegrityError during webserver startup #27297 > https://github.com/apache/airflow/pull/27297 > > This is quite related to this one (at least by FAB permission area both > touch) > > Add missing AUTOINC/SERIAL for FAB tables #26885 > https://github.com/apache/airflow/pull/26885 > > Both connected to FAB permissions and both exposed during migrations > and sometimes causing our users to go back or hold on with the > migration > > J. > > On Fri, Oct 28, 2022 at 12:17 PM Elad Kalif <[email protected]> wrote: > > > > I think https://github.com/apache/airflow/pull/25717 Add > SQLExecuteQueryOperator which was merged this month is a good candidate > (sorry missed the #protm tagging) > > This operator is something we wanted for a really long time (initial PR > was in 2020) > > It took a while to get this done, it involved a lot of changes and at > the end I think it also has a big impact on Airflow users as 1 operator for > almost all sql related providers. > > > > So my vote goes to https://github.com/apache/airflow/pull/25717 > > > > On Fri, Oct 28, 2022 at 2:30 AM John Thomas > > <[email protected]> > wrote: > >> > >> Correction: Voting will close on 10/31, not 9/31. Voting closes in the > future, not the past > >> > >> On Thu, Oct 27, 2022 at 5:28 PM John Thomas <[email protected]> > wrote: > >>> > >>> Hey All! > >>> > >>> It's that time again to vote for the PR of the Month for the upcoming > Airflow Newsletter. Below are some candidates that came out of the > `get_important_pr_candidates.py` script in /dev/stats. Remember that if > there's a PR you think is a good candidate you can nominate it with the > #protm tag in a comment on the PR :) > >>> > >>> Please vote by selecting the most important, interesting, or impactful > PR from the list below (or an entirely new one!) and replying with the > number. Voting will close on 9/31 at 8:15 AM PT > >>> > >>> [26942] Filtering datasets by recent update events > >>> https://github.com/apache/airflow/pull/26942 > >>> > >>> [27076] Add RdsStopDbOperator and RdsStartDbOperator > >>> https://github.com/apache/airflow/pull/27076 > >>> > >>> [26886] Adding preserve_file_name param to S3Hook.download_file method > >>> https://github.com/apache/airflow/pull/26886 > >>> > >>> [27141] Fix some bug in web ui dags list page (auto-refresh & jump > search null state) > >>> https://github.com/apache/airflow/pull/27141 > >>> > >>> [26815] Move user-facing string to template > >>> https://github.com/apache/airflow/pull/26815 > >>> > >>> [26880] Add separate error handler for 405(Method not allowed) errors > >>> https://github.com/apache/airflow/pull/26880 > >>> > >>> Thanks! > >>> John > >>> >
