[GitHub] [airflow] ashb opened a new pull request, #25727: Separate "public" Dataset class from SQLA model

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

ashb opened a new pull request, #25727:
URL: https://github.com/apache/airflow/pull/25727

   Importing all of SQLA can be very heavy weight (slowing import times),
   and it can also make it harder if libraries want to build on top of the
   Dataset concept.
   
   Not to mention that the interal-API/db-isolation AIP should mean we
   don't want to import any SQLA models directly in user code.


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

[GitHub] [airflow] ashb merged pull request #25727: Separate "public" Dataset class from SQLA model

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

ashb merged PR #25727:
URL: https://github.com/apache/airflow/pull/25727


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

[GitHub] [airflow] ashb commented on pull request #25727: Separate "public" Dataset class from SQLA model

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

ashb commented on PR #25727:
URL: https://github.com/apache/airflow/pull/25727#issuecomment-1215733556

   Overzealous refactoring ;) will fix those in the morning 


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