[GitHub] [airflow] potiuk commented on issue #7473: [AIRFLOW-6852] Force reschedule mode for sensors when running on SequentialExecutor

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

potiuk commented on issue #7473: [AIRFLOW-6852] Force reschedule mode for sensors when running on SequentialExecutor
URL: https://github.com/apache/airflow/pull/7473#issuecomment-590327691
 
 
   > The SequentialExecutor _stops heartbeating_ when running a task, so this is very very unlikely to be used in production, as you'd constantly get "The scheduler is not running" notices in the UI.
   > 
   > I'm not even sure we should have Sequential executor anymore. LocalExecutor with a single process would be much much better.
   
   SequentialExecutor is currently the only one allowed for SQLIte
   
   ```
       def _validate(self):
           if (
                   self.get("core", "executor") not in ('DebugExecutor', 'SequentialExecutor') and
                   "sqlite" in self.get('core', 'sql_alchemy_conn')):
               raise AirflowConfigException(
                   "error: cannot use sqlite with the {}".format(
                       self.get('core', 'executor')))
   ```
   
   But indeed, if we change it to enforce the LocalExecutor and count ==1 this might work and we can get rid of the Sequential executor.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services