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:
[email protected]


With regards,
Apache Git Services

Reply via email to