Problem
There is an inconsistency between the Python task state string representations and what would be more standard naming:
Current Python representations:
TaskState.Succeeded → "success"
TaskState.Failed → "fail"
TaskState.Cancelled → "cancel"
Suggested consistent representations:
TaskState.Succeeded → "succeeded"
TaskState.Failed → "failed"
TaskState.Cancelled → "cancelled"
Impact
This change requires updates to both:
- C++ codebase (MariaDB table definitions)
- Python codebase (string mapping)
Context
This issue was identified during code review of PR #216 where the task state string mappings were introduced.
Backlinks:
Problem
There is an inconsistency between the Python task state string representations and what would be more standard naming:
Current Python representations:
TaskState.Succeeded→ "success"TaskState.Failed→ "fail"TaskState.Cancelled→ "cancel"Suggested consistent representations:
TaskState.Succeeded→ "succeeded"TaskState.Failed→ "failed"TaskState.Cancelled→ "cancelled"Impact
This change requires updates to both:
Context
This issue was identified during code review of PR #216 where the task state string mappings were introduced.
Backlinks: