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