satish-chinthanippu commented on issue #40382:
URL: https://github.com/apache/airflow/issues/40382#issuecomment-2186439150

   > > Can you please remove teradata #40378 PR from this 2.3.0 release as we 
wanted to release compute cluster functionality along with this. We are in 
process of creating new PR for compute cluster and we would like to release 
2.3.0 with these two features as per our roadmap.
   > 
   > It's of course @eladkal (Release Manager's) decision - but no 
@satish-chinthanippu , this is not how our provider's release process work. We 
release every changed provider from main, and we only exclude a provider (not 
individual PRs) from release if a serious bug has been found. Manipulating and 
manually modifying stuff during the release is not easy, takes time and effort, 
might break various parts of the process (like documentation generation, or 
package preparation, publishing, signing and verification) and introduces 
serious overhead for ther release process that we don't want.
   > 
   > We have more than 90 providers and we cannot afford individual treatment 
and such "custom" approach.
   > 
   > If there is a bug / regression in the existing functionality that is a 
blocker - we might remove whole provider from the set of the providers being 
voted - but that's about all flexibility, and unless there is a bug in Teradata 
provider, it will be released as-is.
   > 
   > We do not look at other's roadmaps - this is a bit of price to pay for 
making the provider a "community managed" one and we've been very clear about 
the process when we accepted Teradata - one of the reasons why Teradata could 
choose to release their own provider, was that this could help them to manage 
their own release roadmap and schedule. Once this is a community provider, we 
do expect Teradata to keep it updated, with dashboard and fix bugs (in their 
own interest) but this also means that anyone can contribute changes (and 
Airflow committers make decisions what goes in and out) but also that we 
release it together with other providers with the same cadence.
   > 
   > I hope that explains how it works :). This is not a complaint or being 
nasty (we appreciate all the work done, system test dashboards, and all the new 
features you guys, add). It's just the way how we have to manage 90+ providers 
in release has to have some limitations and structure, and also the governance 
requirements of the ASF is very clear that once the code is submitted to our 
repo it has to follow the rules of ASF where only Airflow committers can decide 
on the code modifications. So I wanted to make it clear.
   
   Thank you @potiuk  for your detailed information. Understood the process and 
considerations regarding the release. Actually, @eladkal  suggested to raise 
individual PRs for each related functionality implemented in Airflow Teradata 
Provider to make the review process simpler. So, in line with this suggestion, 
we thought of raising individual PRs for the two features we have planned for 
this release.
   We're committed to aligning with the community standards and appreciate the 
governance framework outlined by ASF.
   Given this understanding, we'll proceed with the new PR for the compute 
cluster functionality alongside Teradata's provider updates as per the standard 
release cadence. We'll ensure that our contributions meet the necessary 
criteria and are compatible with the overall release process.
   Please let us know if there are any specific guidelines or additional steps 
we should follow as we prepare these updates with multiple PRs for a single 
release.
   


-- 
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]

Reply via email to