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/08/01 15:31:43 UTC

[GitHub] [airflow] ashb commented on pull request #25419: WIP: Create a pluggable DatasetEventManager

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

   I think lets slim this down for now, and only add the methods/"entrypoints" that the scheduler needs, and then a custom class can user `super()` etc to do what it wants.
   
   I _think_ we might also want to move some of the query/creating logic from Daniel's PR  #24969 to exist in this class (i.e. just moving the code and calling it from inside SchedulerJob) -- need to think more about query pattern/only-one-lock vs flexibility here. /cc @dstandish 


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