Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-12-29 Thread via GitHub
jscheffl closed pull request #35861: Add a migration script tooling support to migrate off mssql URL: https://github.com/apache/airflow/pull/35861 -- 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

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-12-29 Thread via GitHub
jscheffl commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1872320271 Migrated script to https://github.com/apache/airflow-mssql-migration -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-12-07 Thread via GitHub
ephraimbuddy commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1844873329 +1 to keep this separate from the Airflow repo considering the added maintenance time it would cost and of course the CVE issues -- This is an automated message from the Apache G

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-11-28 Thread via GitHub
potiuk commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1829761337 > > I see no problem with that. Mssql is considered experimental. We can publish it under statement of best effort. In the mailing list thread one of the comments was that we should

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-11-26 Thread via GitHub
eladkal commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1826833780 > making it part of our repository in general I see no problem with that. Mssql is considered experimental. We can publish it under statmant of best effort. In the mailing l

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-11-26 Thread via GitHub
potiuk commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1826824513 And... Fantastic you are doing it ! Thanks for that! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-11-26 Thread via GitHub
jscheffl commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1826823839 Hi @potiuk Yes, maybe this is a mis-understanding. I did NOT want to make this script a release piece. If we see we need it in other cases we _could_ leverage it. So it is now i

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-11-26 Thread via GitHub
potiuk commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1826819752 I wonder - what others think about it. Especially those who were in Airflow when we migrated to Airflow 2. from 1.10.15. And just to add a bit more context. We did release https:/

Re: [PR] Add a migration script tooling support to migrate off mssql [airflow]

2023-11-26 Thread via GitHub
potiuk commented on PR #35861: URL: https://github.com/apache/airflow/pull/35861#issuecomment-1826815324 I am not going to block it in this form - but - as explained in a number of messages - I have a huge concern about a) making it part of official airflow tooling b) making it p