Skip to content

BED-8746: migrate to app.defer#18

Closed
StranDutton wants to merge 1 commit into
mainfrom
feature/BED-8746-migrate-app-defer
Closed

BED-8746: migrate to app.defer#18
StranDutton wants to merge 1 commit into
mainfrom
feature/BED-8746-migrate-app-defer

Conversation

@StranDutton

Copy link
Copy Markdown
Contributor

Replaces @dlt.defer with @app.defer in organization.py for the _workflow_file_contents function.

@dlt.defer functions execute lazily outside the generator iteration loop, so exceptions in them bypass the safe resource wrapper and crash the entire pipeline. @app.defer wraps them with error handling that logs the exception allows the pipeline to continue.

Depends on OpenHound PR #37

@StranDutton StranDutton self-assigned this Jun 24, 2026
@StranDutton StranDutton added the bug Something isn't working label Jun 24, 2026
@StranDutton

Copy link
Copy Markdown
Contributor Author

wrong branch prefix, closing because it should be patch not feature

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant