ashb commented on a change in pull request #6760: [AIRFLOW-6157] Separate out common protocol for executors. URL: https://github.com/apache/airflow/pull/6760#discussion_r355994742
########## File path: airflow/executors/base_executor.py ########## @@ -42,7 +43,124 @@ QueuedTaskInstanceType = Tuple[CommandType, int, Optional[str], SimpleTaskInstance] -class BaseExecutor(LoggingMixin): +class BaseExecutorProtocol(LoggingMixin): + """ + Base Protocol implemented by all executors including multiple executors. + """ + + def __init__(self): + super().__init__() Review comment: If this has code it's not a "Protocol" ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
