[GitHub] [airflow] hussein-awala commented on a diff in pull request #33397: Remove `supports_ad_hoc_ti_run` from executor API

["hussein-awala (via GitHub)" <gi...@apache.org>]

hussein-awala commented on code in PR #33397:
URL: https://github.com/apache/airflow/pull/33397#discussion_r1294051849


##########
airflow/executors/base_executor.py:
##########
@@ -104,7 +104,6 @@ class BaseExecutor(LoggingMixin):
     :param parallelism: how many jobs should run at one time. Set to ``0`` for infinity.
     """
 
-    supports_ad_hoc_ti_run: bool = False

Review Comment:
   `BaseExecutor` is a part of the public interface of Airflow, is it safe to remove this config?
   
   We don't use this config in Airflow, but we don't know if the users use it in custom plugins or custom executors.



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