fix: increase table client max retries to 5#32
Conversation
There was a problem hiding this comment.
Pull request overview
This PR increases the maximum retry attempts for Azure Table Storage client operations from the default to 5 retries, improving resilience against transient failures. The version is also bumped to 0.5.5.
- Adds retry configuration with
maxRetries: 5for all TableClient instances - Applies the retry configuration to both connection string and credential-based initialization paths
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/stores/DurableFunctionsWorkflowHistoryStore.ts | Adds retry options to TableClient instantiation for both history and instances tables |
| package.json | Bumps package version from 0.5.4 to 0.5.5 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| this.history = new TableClient(tableUrl, `${taskHubName}History`, credential, tableServiceClientOptions); | ||
| this.instances = new TableClient(tableUrl, `${taskHubName}Instances`, credential, tableServiceClientOptions); |
There was a problem hiding this comment.
The TableClient constructor signature appears incorrect. According to the Azure SDK, TableClient constructor takes (url, tableName, credential?, options?) but this code passes credential as the third parameter and options as the fourth. The credential and options parameters should be swapped or combined into the options object.
No description provided.