You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@airflow.apache.org by Michael Robinson <mi...@astronomer.io.INVALID> on 2023/04/25 21:08:46 UTC
[VOTE] April 2023 PR of the Month
Hello all,
It’s once again time to vote for the PR of the Month.
With the help of the script in dev/stats, I’ve identified the following candidates:
#30255 by @potiuk: “Decouple 'job under' from BaseJob ORM model” <https://github.com/apache/airflow/pull/30255>
#29940 by @mobuchowski: “Add base OpenLineage provider implementation” <https://github.com/apache/airflow/pulls?q=is%3Apr+29940>
#30495 by @vandonr-amz: “Preload Airflow imports before DAG parsing to save time” <https://github.com/apache/airflow/pull/30495>
#30032 by @syedahsn: “Add support for deferrable operators in AMPP” <https://github.com/apache/airflow/pull/30032>
#30705 by @potiuk: “Optimize parallel test execution for unit tests” <https://github.com/apache/airflow/pull/30705>
#30175 by @dylanbstorey: “Add provider for Apache Kafka” <https://github.com/apache/airflow/pull/30175>
#30375 by @RNHTTR: “Simplify logic to resolve tasks stuck in queued despite stalled_task_timeout” <https://github.com/apache/airflow/pull/30375>
#30373 by @bbovenzi: “Improve task & run actions UX in grid view” <https://github.com/apache/airflow/pull/30373>
Please reply to the thread with your selection or offer your own nominee(s).
Voting will close on Friday, April 28th at 4 pm ET. The winner 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, please drop me a line.
-Michael
Re: [VOTE] April 2023 PR of the Month
Posted by Jorrick Sleijster <jo...@gmail.com>.
+1 for #30373. Can't wait to test it 😁
On Wed, 26 Apr 2023, 08:12 Pankaj Koti, <pa...@astronomer.io.invalid>
wrote:
> +1 for #30373 by @bbovenzi: “Improve task & run actions UX in grid view” <
> https://github.com/apache/airflow/pull/30373>
>
> Regards,
>
>
>
> Pankaj Koti
>
> *Senior Software Engineer, *OSS Engineering Team.
> Location: Pune, India
>
> Timezone: Indian Standard Time (IST)
>
> Email: pankaj.koti@astronomer.io
>
> Mobile: +91 9730079985
>
>
> On Wed, Apr 26, 2023 at 11:37 AM Abhishek Bhakat
> <ab...@astronomer.io.invalid> wrote:
>
> > +1 for #30375 is must to have.
> >
> > Thanks,
> > Abhishek Bhakat
> > Airflow Engineer
> >
> > On 26-Apr-2023 at 01:31:22, Jarek Potiuk <ja...@potiuk.com> wrote:
> >
> > > Indeed. Extremely tough choice this time, but I'd go for 30175 for me
> ->
> > > Kafka provider has been long in the making and Dylan had done a lot of
> > > effort to make it work well and respond to plenty of comments,
> including
> > > adding new integration in breeze which means that the new provider is
> > > automatically tested for regressions in our CI with "real" dockerised
> > Kafka
> > > running during the tests.
> > >
> > > On Wed, Apr 26, 2023 at 3:16 AM Mehta, Shubham
> <shubhx@amazon.com.invalid
> > >
> > > wrote:
> > >
> > > That's an amazing set of PRs. I vote for #29940. Kudos to Maciej for
> the
> > >
> > > effort he put into it and laying out the foundation for OL work.
> > >
> > >
> > > Shubham
> > >
> > >
> > > On 2023-04-25, 6:08 PM, "Michael Robinson"
> > >
> > > <michael.robinson@astronomer.io.INVA <mailto:
> > >
> > > michael.robinson@astronomer.io.INVA>LID> wrote:
> > >
> > >
> > >
> > > CAUTION: This email originated from outside of the organization. Do not
> > >
> > > click links or open attachments unless you can confirm the sender and
> > know
> > >
> > > the content is safe.
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> > > Hello all,
> > >
> > >
> > >
> > > It’s once again time to vote for the PR of the Month.
> > >
> > >
> > >
> > > With the help of the script in dev/stats, I’ve identified the following
> > >
> > > candidates:
> > >
> > >
> > >
> > > #30255 by @potiuk: “Decouple 'job under' from BaseJob ORM model” <
> > >
> > > https://github.com/apache/airflow/pull/30255> <
> > >
> > > https://github.com/apache/airflow/pull/30255>>
> > >
> > > #29940 by @mobuchowski: “Add base OpenLineage provider implementation”
> <
> > >
> > > https://github.com/apache/airflow/pulls?q=is%3Apr+29940> <
> > >
> > > https://github.com/apache/airflow/pulls?q=is%3Apr+29940>>
> > >
> > > #30495 by @vandonr-amz: “Preload Airflow imports before DAG parsing to
> > >
> > > save time” <https://github.com/apache/airflow/pull/30495> <
> > >
> > > https://github.com/apache/airflow/pull/30495>>
> > >
> > > #30032 by @syedahsn: “Add support for deferrable operators in AMPP” <
> > >
> > > https://github.com/apache/airflow/pull/30032> <
> > >
> > > https://github.com/apache/airflow/pull/30032>>
> > >
> > > #30705 by @potiuk: “Optimize parallel test execution for unit tests” <
> > >
> > > https://github.com/apache/airflow/pull/30705> <
> > >
> > > https://github.com/apache/airflow/pull/30705>>
> > >
> > > #30175 by @dylanbstorey: “Add provider for Apache Kafka” <
> > >
> > > https://github.com/apache/airflow/pull/30175> <
> > >
> > > https://github.com/apache/airflow/pull/30175>>
> > >
> > > #30375 by @RNHTTR: “Simplify logic to resolve tasks stuck in queued
> > >
> > > despite stalled_task_timeout” <
> > >
> > > https://github.com/apache/airflow/pull/30375> <
> > >
> > > https://github.com/apache/airflow/pull/30375>>
> > >
> > > #30373 by @bbovenzi: “Improve task & run actions UX in grid view” <
> > >
> > > https://github.com/apache/airflow/pull/30373> <
> > >
> > > https://github.com/apache/airflow/pull/30373>>
> > >
> > >
> > >
> > > Please reply to the thread with your selection or offer your own
> > >
> > > nominee(s).
> > >
> > >
> > >
> > > Voting will close on Friday, April 28th at 4 pm ET. The winner 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, please drop me a line.
> > >
> > >
> > >
> > > -Michael
> > >
> > >
> > >
> > >
> > >
> > > ---------------------------------------------------------------------
> > >
> > > To unsubscribe, e-mail: dev-unsubscribe@airflow.apache.org
> > >
> > > For additional commands, e-mail: dev-help@airflow.apache.org
> > >
> > >
> > >
> >
>
Re: [VOTE] April 2023 PR of the Month
Posted by Pankaj Koti <pa...@astronomer.io.INVALID>.
+1 for #30373 by @bbovenzi: “Improve task & run actions UX in grid view” <
https://github.com/apache/airflow/pull/30373>
Regards,
Pankaj Koti
*Senior Software Engineer, *OSS Engineering Team.
Location: Pune, India
Timezone: Indian Standard Time (IST)
Email: pankaj.koti@astronomer.io
Mobile: +91 9730079985
On Wed, Apr 26, 2023 at 11:37 AM Abhishek Bhakat
<ab...@astronomer.io.invalid> wrote:
> +1 for #30375 is must to have.
>
> Thanks,
> Abhishek Bhakat
> Airflow Engineer
>
> On 26-Apr-2023 at 01:31:22, Jarek Potiuk <ja...@potiuk.com> wrote:
>
> > Indeed. Extremely tough choice this time, but I'd go for 30175 for me ->
> > Kafka provider has been long in the making and Dylan had done a lot of
> > effort to make it work well and respond to plenty of comments, including
> > adding new integration in breeze which means that the new provider is
> > automatically tested for regressions in our CI with "real" dockerised
> Kafka
> > running during the tests.
> >
> > On Wed, Apr 26, 2023 at 3:16 AM Mehta, Shubham <shubhx@amazon.com.invalid
> >
> > wrote:
> >
> > That's an amazing set of PRs. I vote for #29940. Kudos to Maciej for the
> >
> > effort he put into it and laying out the foundation for OL work.
> >
> >
> > Shubham
> >
> >
> > On 2023-04-25, 6:08 PM, "Michael Robinson"
> >
> > <michael.robinson@astronomer.io.INVA <mailto:
> >
> > michael.robinson@astronomer.io.INVA>LID> wrote:
> >
> >
> >
> > CAUTION: This email originated from outside of the organization. Do not
> >
> > click links or open attachments unless you can confirm the sender and
> know
> >
> > the content is safe.
> >
> >
> >
> >
> >
> >
> >
> > Hello all,
> >
> >
> >
> > It’s once again time to vote for the PR of the Month.
> >
> >
> >
> > With the help of the script in dev/stats, I’ve identified the following
> >
> > candidates:
> >
> >
> >
> > #30255 by @potiuk: “Decouple 'job under' from BaseJob ORM model” <
> >
> > https://github.com/apache/airflow/pull/30255> <
> >
> > https://github.com/apache/airflow/pull/30255>>
> >
> > #29940 by @mobuchowski: “Add base OpenLineage provider implementation” <
> >
> > https://github.com/apache/airflow/pulls?q=is%3Apr+29940> <
> >
> > https://github.com/apache/airflow/pulls?q=is%3Apr+29940>>
> >
> > #30495 by @vandonr-amz: “Preload Airflow imports before DAG parsing to
> >
> > save time” <https://github.com/apache/airflow/pull/30495> <
> >
> > https://github.com/apache/airflow/pull/30495>>
> >
> > #30032 by @syedahsn: “Add support for deferrable operators in AMPP” <
> >
> > https://github.com/apache/airflow/pull/30032> <
> >
> > https://github.com/apache/airflow/pull/30032>>
> >
> > #30705 by @potiuk: “Optimize parallel test execution for unit tests” <
> >
> > https://github.com/apache/airflow/pull/30705> <
> >
> > https://github.com/apache/airflow/pull/30705>>
> >
> > #30175 by @dylanbstorey: “Add provider for Apache Kafka” <
> >
> > https://github.com/apache/airflow/pull/30175> <
> >
> > https://github.com/apache/airflow/pull/30175>>
> >
> > #30375 by @RNHTTR: “Simplify logic to resolve tasks stuck in queued
> >
> > despite stalled_task_timeout” <
> >
> > https://github.com/apache/airflow/pull/30375> <
> >
> > https://github.com/apache/airflow/pull/30375>>
> >
> > #30373 by @bbovenzi: “Improve task & run actions UX in grid view” <
> >
> > https://github.com/apache/airflow/pull/30373> <
> >
> > https://github.com/apache/airflow/pull/30373>>
> >
> >
> >
> > Please reply to the thread with your selection or offer your own
> >
> > nominee(s).
> >
> >
> >
> > Voting will close on Friday, April 28th at 4 pm ET. The winner 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, please drop me a line.
> >
> >
> >
> > -Michael
> >
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> >
> > To unsubscribe, e-mail: dev-unsubscribe@airflow.apache.org
> >
> > For additional commands, e-mail: dev-help@airflow.apache.org
> >
> >
> >
>
Re: [VOTE] April 2023 PR of the Month
Posted by Abhishek Bhakat <ab...@astronomer.io.INVALID>.
+1 for #30375 is must to have.
Thanks,
Abhishek Bhakat
Airflow Engineer
On 26-Apr-2023 at 01:31:22, Jarek Potiuk <ja...@potiuk.com> wrote:
> Indeed. Extremely tough choice this time, but I'd go for 30175 for me ->
> Kafka provider has been long in the making and Dylan had done a lot of
> effort to make it work well and respond to plenty of comments, including
> adding new integration in breeze which means that the new provider is
> automatically tested for regressions in our CI with "real" dockerised Kafka
> running during the tests.
>
> On Wed, Apr 26, 2023 at 3:16 AM Mehta, Shubham <sh...@amazon.com.invalid>
> wrote:
>
> That's an amazing set of PRs. I vote for #29940. Kudos to Maciej for the
>
> effort he put into it and laying out the foundation for OL work.
>
>
> Shubham
>
>
> On 2023-04-25, 6:08 PM, "Michael Robinson"
>
> <michael.robinson@astronomer.io.INVA <mailto:
>
> michael.robinson@astronomer.io.INVA>LID> wrote:
>
>
>
> CAUTION: This email originated from outside of the organization. Do not
>
> click links or open attachments unless you can confirm the sender and know
>
> the content is safe.
>
>
>
>
>
>
>
> Hello all,
>
>
>
> It’s once again time to vote for the PR of the Month.
>
>
>
> With the help of the script in dev/stats, I’ve identified the following
>
> candidates:
>
>
>
> #30255 by @potiuk: “Decouple 'job under' from BaseJob ORM model” <
>
> https://github.com/apache/airflow/pull/30255> <
>
> https://github.com/apache/airflow/pull/30255>>
>
> #29940 by @mobuchowski: “Add base OpenLineage provider implementation” <
>
> https://github.com/apache/airflow/pulls?q=is%3Apr+29940> <
>
> https://github.com/apache/airflow/pulls?q=is%3Apr+29940>>
>
> #30495 by @vandonr-amz: “Preload Airflow imports before DAG parsing to
>
> save time” <https://github.com/apache/airflow/pull/30495> <
>
> https://github.com/apache/airflow/pull/30495>>
>
> #30032 by @syedahsn: “Add support for deferrable operators in AMPP” <
>
> https://github.com/apache/airflow/pull/30032> <
>
> https://github.com/apache/airflow/pull/30032>>
>
> #30705 by @potiuk: “Optimize parallel test execution for unit tests” <
>
> https://github.com/apache/airflow/pull/30705> <
>
> https://github.com/apache/airflow/pull/30705>>
>
> #30175 by @dylanbstorey: “Add provider for Apache Kafka” <
>
> https://github.com/apache/airflow/pull/30175> <
>
> https://github.com/apache/airflow/pull/30175>>
>
> #30375 by @RNHTTR: “Simplify logic to resolve tasks stuck in queued
>
> despite stalled_task_timeout” <
>
> https://github.com/apache/airflow/pull/30375> <
>
> https://github.com/apache/airflow/pull/30375>>
>
> #30373 by @bbovenzi: “Improve task & run actions UX in grid view” <
>
> https://github.com/apache/airflow/pull/30373> <
>
> https://github.com/apache/airflow/pull/30373>>
>
>
>
> Please reply to the thread with your selection or offer your own
>
> nominee(s).
>
>
>
> Voting will close on Friday, April 28th at 4 pm ET. The winner 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, please drop me a line.
>
>
>
> -Michael
>
>
>
>
>
> ---------------------------------------------------------------------
>
> To unsubscribe, e-mail: dev-unsubscribe@airflow.apache.org
>
> For additional commands, e-mail: dev-help@airflow.apache.org
>
>
>
Re: [VOTE] April 2023 PR of the Month
Posted by Jarek Potiuk <ja...@potiuk.com>.
Indeed. Extremely tough choice this time, but I'd go for 30175 for me ->
Kafka provider has been long in the making and Dylan had done a lot of
effort to make it work well and respond to plenty of comments, including
adding new integration in breeze which means that the new provider is
automatically tested for regressions in our CI with "real" dockerised Kafka
running during the tests.
On Wed, Apr 26, 2023 at 3:16 AM Mehta, Shubham <sh...@amazon.com.invalid>
wrote:
> That's an amazing set of PRs. I vote for #29940. Kudos to Maciej for the
> effort he put into it and laying out the foundation for OL work.
>
> Shubham
>
> On 2023-04-25, 6:08 PM, "Michael Robinson"
> <michael.robinson@astronomer.io.INVA <mailto:
> michael.robinson@astronomer.io.INVA>LID> wrote:
>
>
> CAUTION: This email originated from outside of the organization. Do not
> click links or open attachments unless you can confirm the sender and know
> the content is safe.
>
>
>
>
>
>
> Hello all,
>
>
> It’s once again time to vote for the PR of the Month.
>
>
> With the help of the script in dev/stats, I’ve identified the following
> candidates:
>
>
> #30255 by @potiuk: “Decouple 'job under' from BaseJob ORM model” <
> https://github.com/apache/airflow/pull/30255> <
> https://github.com/apache/airflow/pull/30255>>
> #29940 by @mobuchowski: “Add base OpenLineage provider implementation” <
> https://github.com/apache/airflow/pulls?q=is%3Apr+29940> <
> https://github.com/apache/airflow/pulls?q=is%3Apr+29940>>
> #30495 by @vandonr-amz: “Preload Airflow imports before DAG parsing to
> save time” <https://github.com/apache/airflow/pull/30495> <
> https://github.com/apache/airflow/pull/30495>>
> #30032 by @syedahsn: “Add support for deferrable operators in AMPP” <
> https://github.com/apache/airflow/pull/30032> <
> https://github.com/apache/airflow/pull/30032>>
> #30705 by @potiuk: “Optimize parallel test execution for unit tests” <
> https://github.com/apache/airflow/pull/30705> <
> https://github.com/apache/airflow/pull/30705>>
> #30175 by @dylanbstorey: “Add provider for Apache Kafka” <
> https://github.com/apache/airflow/pull/30175> <
> https://github.com/apache/airflow/pull/30175>>
> #30375 by @RNHTTR: “Simplify logic to resolve tasks stuck in queued
> despite stalled_task_timeout” <
> https://github.com/apache/airflow/pull/30375> <
> https://github.com/apache/airflow/pull/30375>>
> #30373 by @bbovenzi: “Improve task & run actions UX in grid view” <
> https://github.com/apache/airflow/pull/30373> <
> https://github.com/apache/airflow/pull/30373>>
>
>
> Please reply to the thread with your selection or offer your own
> nominee(s).
>
>
> Voting will close on Friday, April 28th at 4 pm ET. The winner 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, please drop me a line.
>
>
> -Michael
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@airflow.apache.org
> For additional commands, e-mail: dev-help@airflow.apache.org
>
Re: [VOTE] April 2023 PR of the Month
Posted by "Mehta, Shubham" <sh...@amazon.com.INVALID>.
That's an amazing set of PRs. I vote for #29940. Kudos to Maciej for the effort he put into it and laying out the foundation for OL work.
Shubham
On 2023-04-25, 6:08 PM, "Michael Robinson" <michael.robinson@astronomer.io.INVA <ma...@astronomer.io.INVA>LID> wrote:
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe.
Hello all,
It’s once again time to vote for the PR of the Month.
With the help of the script in dev/stats, I’ve identified the following candidates:
#30255 by @potiuk: “Decouple 'job under' from BaseJob ORM model” <https://github.com/apache/airflow/pull/30255> <https://github.com/apache/airflow/pull/30255>>
#29940 by @mobuchowski: “Add base OpenLineage provider implementation” <https://github.com/apache/airflow/pulls?q=is%3Apr+29940> <https://github.com/apache/airflow/pulls?q=is%3Apr+29940>>
#30495 by @vandonr-amz: “Preload Airflow imports before DAG parsing to save time” <https://github.com/apache/airflow/pull/30495> <https://github.com/apache/airflow/pull/30495>>
#30032 by @syedahsn: “Add support for deferrable operators in AMPP” <https://github.com/apache/airflow/pull/30032> <https://github.com/apache/airflow/pull/30032>>
#30705 by @potiuk: “Optimize parallel test execution for unit tests” <https://github.com/apache/airflow/pull/30705> <https://github.com/apache/airflow/pull/30705>>
#30175 by @dylanbstorey: “Add provider for Apache Kafka” <https://github.com/apache/airflow/pull/30175> <https://github.com/apache/airflow/pull/30175>>
#30375 by @RNHTTR: “Simplify logic to resolve tasks stuck in queued despite stalled_task_timeout” <https://github.com/apache/airflow/pull/30375> <https://github.com/apache/airflow/pull/30375>>
#30373 by @bbovenzi: “Improve task & run actions UX in grid view” <https://github.com/apache/airflow/pull/30373> <https://github.com/apache/airflow/pull/30373>>
Please reply to the thread with your selection or offer your own nominee(s).
Voting will close on Friday, April 28th at 4 pm ET. The winner 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, please drop me a line.
-Michael
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@airflow.apache.org
For additional commands, e-mail: dev-help@airflow.apache.org
Re: [VOTE] April 2023 PR of the Month
Posted by Collin McNulty <co...@astronomer.io.INVALID>.
So many great ones this month. My vote is for 30375 on account of having
been personally affected by the behavior.
On Tue, Apr 25, 2023 at 8:08 PM Michael Robinson
<mi...@astronomer.io.invalid> wrote:
> Hello all,
>
> It’s once again time to vote for the PR of the Month.
>
> With the help of the script in dev/stats, I’ve identified the following
> candidates:
>
> #30255 by @potiuk: “Decouple 'job under' from BaseJob ORM model” <
> https://github.com/apache/airflow/pull/30255>
> #29940 by @mobuchowski: “Add base OpenLineage provider implementation” <
> https://github.com/apache/airflow/pulls?q=is%3Apr+29940>
> #30495 by @vandonr-amz: “Preload Airflow imports before DAG parsing to
> save time” <https://github.com/apache/airflow/pull/30495>
> #30032 by @syedahsn: “Add support for deferrable operators in AMPP” <
> https://github.com/apache/airflow/pull/30032>
> #30705 by @potiuk: “Optimize parallel test execution for unit tests” <
> https://github.com/apache/airflow/pull/30705>
> #30175 by @dylanbstorey: “Add provider for Apache Kafka” <
> https://github.com/apache/airflow/pull/30175>
> #30375 by @RNHTTR: “Simplify logic to resolve tasks stuck in queued
> despite stalled_task_timeout” <
> https://github.com/apache/airflow/pull/30375>
> #30373 by @bbovenzi: “Improve task & run actions UX in grid view” <
> https://github.com/apache/airflow/pull/30373>
>
> Please reply to the thread with your selection or offer your own
> nominee(s).
>
> Voting will close on Friday, April 28th at 4 pm ET. The winner 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, please drop me a line.
>
> -Michael
--
Collin McNulty
Lead Airflow Engineer
Email: collin@astronomer.io <jo...@astronomer.io>
Time zone: US Central (CST UTC-6 / CDT UTC-5)
<https://www.astronomer.io/>