You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/03/15 19:37:14 UTC

[GitHub] [airflow] ephraimbuddy commented on pull request #22177: Add test to run offline SQL generation in the CI

ephraimbuddy commented on pull request #22177:
URL: https://github.com/apache/airflow/pull/22177#issuecomment-1068389701


   I have dropped support for MSSQL until we fix its auto-generated primary keys that are difficult to guess. The only way we can currently drop MSSQL primary keys is to do this: https://github.com/apache/airflow/blob/dadb1325a73d161000d1537a1a26edf21aeb5bbd/airflow/migrations/versions/0104_4eaab2fe6582_migrate_rtif_to_use_run_id_and_map_index.py#L127-L130.
   
   We can overcome this limitation by using a naming convention which I'm experimenting currently


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