Fix Data Loading Issue #3207
Fix Data Loading Issue #3207
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR fixes a data loading issue by refining exception handling boundaries. Instead of having a single try/except block around the entire loading process, exceptions are now caught at more granular levels to prevent one plugin's failure from stopping the entire loading process.
- Moved exception handling from method-level to individual plugin and task processing
- Added more granular error handling for plugin loading and async task execution
- Maintained existing debug logging for exceptions
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| await self._load_extensions() | ||
| await self._load_data_encoders(plugins) | ||
| await self.create_or_update_everything_adversary() | ||
| await self._verify_data_sets() | ||
|
|
There was a problem hiding this comment.
These critical operations at the end of the method are no longer wrapped in exception handling. If any of these methods fail, they will cause the entire _load method to fail and potentially crash the application. Consider adding try/except blocks around these operations or ensure they have their own internal error handling.
Copilot uses AI. Check for mistakes.
Description
Fixes #3206 by reducing the try/except boundary.
Type of change
How Has This Been Tested?
Local startup test.
Checklist: