[GitHub] [airflow] potiuk edited a comment on pull request #18506: Support for MariaDB >= 10.6

[GitBox <gi...@apache.org>]

potiuk edited a comment on pull request #18506:
URL: https://github.com/apache/airflow/pull/18506#issuecomment-961162163


   > I'm not sure how I feel about adding another DB to the test matrix.
   > 
   > I think I'm leaning towards leaving MariaDB support as "best effort" but not actively tested.
   
   I'd take slightly different approach. I think what we could do actually, is to make another (long overdue) refactor of our test suites to only run part of the tests for all databases. For example almost all provider tests do not neeed to be run for all databases, similarly we could refactor/identify/mark some of the modules/packages in core that do not require multi-db testing as they won't need to access the db at all.  That will require a bit more automation (and for example some way of verification that potentially new DB-specific tests are not added to some excluded packages).
   
   I want to wait with that however until we rewrite the Breeze / CI to Python (which we plan to do with the Outreachy interns) and then we can likely think about implementing more elaborated logic in the selective-tests part especially. Then Adding a new DB will be far less of a pain.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org