Fix: Optimized the get_by_id method to resolve the issue of missing exceptions and improve query performance ### What problem does this PR solve? Optimized the get_by_id method to resolve the issue of missing exceptions and improve query performance. Optimization details: 1. The original method used a custom query method that required concatenating SQL, which impacted performance. 2. The query method returned a list, which needed to be accessed by index, posing a risk of index out-of-bounds errors. 3. The original method used except Exception to catch all errors, which is not a best practice in Python programming and may lead to missing exceptions. The get_or_none method accurately catches DoesNotExist errors while allowing other errors to be raised normally. ### Type of change - [x] Bug Fix (non-breaking change which fixes an issue) - [x] Performance Improvement |
||
|---|---|---|
| .. | ||
| __init__.py | ||
| api_service.py | ||
| canvas_service.py | ||
| common_service.py | ||
| conversation_service.py | ||
| dialog_service.py | ||
| document_service.py | ||
| file2document_service.py | ||
| file_service.py | ||
| knowledgebase_service.py | ||
| llm_service.py | ||
| task_service.py | ||
| user_canvas_version.py | ||
| user_service.py | ||