potiuk commented on PR #33211: URL: https://github.com/apache/airflow/pull/33211#issuecomment-1670095460
> > @pankajkoti @Lee-W what about adding try/except for that setter on your side, and going with rc2 release? > > Since we don't have an upper found for the google provider https://github.com/astronomer/astronomer-providers/blob/main/setup.cfg#L64, our previous releases won't have the try/except block and unfortunately it would break the provider :( I'd be for making RC3 with the fix I proposed and @mobuchowski implemented in https://github.com/apache/airflow/pull/33218. While not technically breaking, this is breatking some ways with which Open-Lineage integration works for existing open-lineage integration currently existing in Astronomer. It's a bit funny how Astronomer DOS'ed itself with it a bit - :rofl: - but hey, I think it's important not to break it - especially that this is likely version of the provider that will come with 2.7.0. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
