Add poison message handling to Azure Storage#1366

Open

sophiatev wants to merge 55 commits into

mainfrom

stevosyan/add-poison-message-handling

Commits on Mar 19, 2026

initial implementation
Sophia Tevosyan
committed

Commits on Mar 24, 2026

fixing the compilation errors
Sophia Tevosyan
committed
addressing copilot comments
Sophia Tevosyan
committed
Merge branch 'main' into stevosyan/add-poison-message-handling
sophiatev
authored
fixing the error in the logger where I was incorrectly calling DiscardingMessage
Sophia Tevosyan
committed

Commits on Mar 25, 2026

moved the max dispatch count from IOrchestrationService to dispatch parameters
Sophia Tevosyan
committed

Commits on Apr 1, 2026

updated the implementations to remove all exception-throwing in the case of poison message handling, except for entity unlock requests
Sophia Tevosyan
committed
comment updates
Sophia Tevosyan
committed
fixed a typo, added an argument range check for the max dispatch count
Sophia Tevosyan
committed

Commits on Jun 2, 2026

Commits on Jun 3, 2026

redid the implementation to follow an interface format
Sophia Tevosyan
committed
mroe cleanup and PR comments
Sophia Tevosyan
committed
updated to wait for the async calls
Sophia Tevosyan
committed
Merge branch 'main' into stevosyan/add-poison-message-handling
Sophia Tevosyan
committed

Commits on Jun 4, 2026

fixing a bug related to the results count and something incorrect i had for trace activities
Sophia Tevosyan
committed

Commits on Jun 10, 2026

Commits on Jun 12, 2026

addressing the first round of PR comments
Sophia Tevosyan
committed
missed changing one place from error to warning
Sophia Tevosyan
committed
removed the IsPoisonMessage method in favor of MaxDispatchCount
Sophia Tevosyan
committed
updated variable names to be more descriptive
Sophia Tevosyan
committed

Commits on Jun 19, 2026

Commits on Jul 13, 2026

Merge branch 'main' into stevosyan/add-poison-message-handling
Sophia Tevosyan
committed

Commits on Jul 15, 2026

azure storage implementation done
Sophia Tevosyan
committed
slight update to avoid looking for specific strings
Sophia Tevosyan
committed

Commits on Jul 16, 2026

added one more invalid work item test
Sophia Tevosyan
committed
fixed bug in adding failure results for poison messages
Sophia Tevosyan
committed
tiny test updates
Sophia Tevosyan
committed

Commits on Jul 17, 2026

fixing a bug related to storing poison messages for lock release messages twice, also changed the logic for invalid work items to just delete the incoming messages
Sophia Tevosyan
committed
a few more stylistic things
Sophia Tevosyan
committed
fixed the tests
Sophia Tevosyan
committed
Potential fix for pull request finding 'Useless assignment to local variable'

sophiatev
and
github-code-quality[bot]
authored
Apply suggestions from code review

sophiatev
and
github-code-quality[bot]
authored

Commits on Jul 18, 2026

some more updates to the dispatchers based on copilot comments
Sophia Tevosyan
committed

Commits on Jul 20, 2026

slight update to only return a result if the entity request is not a signal
Sophia Tevosyan
committed

Commits on Jul 22, 2026

new queue implementation
Sophia Tevosyan
committed
addressing copilot comments
Sophia Tevosyan
committed
Apply suggestions from code review

sophiatev
and
Copilot
authored
slight updates
Sophia Tevosyan
committed
Merge branch 'stevosyan/add-poison-message-handling' of https://github.com/Azure/durabletask into stevosyan/add-poison-message-handling
Sophia Tevosyan
committed
added a comment
Sophia Tevosyan
committed
reverted some useful but unreltaed changes, added a sanitize blob name test
Sophia Tevosyan
committed
addressing PR comments
Sophia Tevosyan
committed

Commits on Jul 23, 2026

settings updates, more blob name sanitation, moving to just one blob container, log update
Sophia Tevosyan
committed
addressing copilot comments
Sophia Tevosyan
committed
Potential fix for pull request finding

sophiatev
and
Copilot
authored
more copilot comments
Sophia Tevosyan
committed