Skip to content

Fix celery task error#9

Open
jaffrey-deepsource wants to merge 12 commits into
mainfrom
fix-celery-task-error
Open

Fix celery task error#9
jaffrey-deepsource wants to merge 12 commits into
mainfrom
fix-celery-task-error

Conversation

@jaffrey-deepsource
Copy link
Copy Markdown

No description provided.

NieRonghua and others added 12 commits January 19, 2026 17:25
…d support for secure storage mode.

Handled status update logic for workflow and node execution, ensuring terminal state is not overwritten by non-terminal state.

Added support for secure storage mode parameters.

Optimized database operations and added integrity error handling.
处理工作流执行和工作流节点执行更新时,确保终端状态不会被非终端状态覆盖
当插入出现IntegrityError时,改为记录警告并抛出异常让Celery重试
将工作流执行和节点执行中的终端状态集合提取为模块级常量,提高代码可读性和维护性
优化输入和输出数据的空值检查,使用显式的is not None判断
# Conflicts:
#	api/tasks/workflow_execution_tasks.py
#	api/tasks/workflow_node_execution_tasks.py
清理两个任务文件中未使用的sqlalchemy.orm.sessionmaker导入,简化依赖并提高代码整洁度
…celery-task-error

# Conflicts:
#	api/tasks/workflow_execution_tasks.py
#	api/tasks/workflow_node_execution_tasks.py
@deepsource-development
Copy link
Copy Markdown

deepsource-development Bot commented Feb 12, 2026

DeepSource Code Review

DeepSource reviewed changes in the commit range b76c8fa..da317ba on this pull request. Below is the summary for the review, and you can see the individual issues we found as review comments.

For detailed review results, please see the PR on DeepSource ↗

PR Report Card

Security × 0 issues Overall PR Quality   

Reliability × 0 issues
Complexity × 0 issues
Hygiene × 0 issues

Code Review Summary

Analyzer Status Summary Details
Python No new issues detected. Review ↗
Secrets No new issues detected. Review ↗
How are these analyzer statuses calculated?

Administrators can configure which issue categories are reported and cause analysis to be marked as failed when detected. This helps prevent bad and insecure code from being introduced in the codebase. If you're an administrator, you can modify this in the repository's settings.


💡 If you're a repository administrator, you can configure the quality gates from the settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants