kaxil edited a comment on pull request #18506: URL: https://github.com/apache/airflow/pull/18506#issuecomment-961743999
I disagree completely with "no test" as "no effort". We made sure even if the experience was sub par for Scheduler HA, to atleast make sure it works. Adding to CI is a big NO - CI already fails in certain cases where the frustration would grow for users to see the red cross for random failures that are DB related or something new. We will need to main the CI DockerImage for any mariadb libraries that it is upto date too. There are no huge gains for the maintenance effort. We should really spend the time to building new features in Core Airflow. We know some of the issues with both MySQL and Mariadb and Cloud Composer shifting to Postgres is also a good example. If the underlying DB has some issues like "deadlocks" which won't be easily tested via tests as they are sporadic, we will have to spend time to investigate and fix those if we officially support it. We should really reduce the surface area over here for the things that we officially support. -- 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]
