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 2021/04/14 05:40:59 UTC

[GitHub] [airflow] dstandish commented on pull request #15285: Fixed #9387: Add State types for tasks and DAGs

dstandish commented on pull request #15285:
URL: https://github.com/apache/airflow/pull/15285#issuecomment-819243475


   it's possible to have a stringy enum class, did you explore that?
   
   see here https://stackoverflow.com/a/63028809/2761682
   
   one concern i have with this PR is the creation of a TaskState class.  
   
   there has been periodic discussion on mailing list re adding support for stateful tasks and of late some renewed interest in the topic.  adding DagState and TaskState classes would potentially be in conflict with that.  if we do create stateful tasks, i think that TaskState as the model through which we store state (i.e. an artifact representin the task's state) would be a better usage of the word than an enum representing the execution state.


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