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

Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2023/02/15 01:36:32 UTC

[GitHub] [airflow] potiuk commented on pull request #29188: AIP-44 Migrate Dagbag.sync_to_db to internal API.

potiuk commented on PR #29188:
URL: https://github.com/apache/airflow/pull/29188#issuecomment-1430626233

   Sorry, i have not looked at this before, I want to have a closer look at that after we discuss the approach i proposed in https://github.com/apache/airflow/pull/29513#discussion_r1106553896. - I have a feeling that once we agree this is a good idea and we apply it here, we might implement some simple refactor that will let us avoid any complexity (but it's too late for me now to think clearly).


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