Skip to content

[matjaz-3-sdks] [AI-FSSDK] (DO NOT REVIEW) [FSSDK-12248] Increase max retry time interval to 3 secs#487

Closed
Mat001 wants to merge 1 commit intomasterfrom
ai/mat001/FSSDK-12248-retry-interval
Closed

[matjaz-3-sdks] [AI-FSSDK] (DO NOT REVIEW) [FSSDK-12248] Increase max retry time interval to 3 secs#487
Mat001 wants to merge 1 commit intomasterfrom
ai/mat001/FSSDK-12248-retry-interval

Conversation

@Mat001
Copy link
Contributor

@Mat001 Mat001 commented Feb 4, 2026

Summary

Increase the maximum retry time interval from 1 second to 3 seconds in the ODP event manager retry logic, providing better resilience to temporary network issues.

Changes

  • Updated max_retry_interval from 1.0 to 3.0 seconds in optimizely/odp/odp_event_manager.py
  • Updated related comments to reflect the new 3-second cap
  • Added comprehensive test to verify the 3-second cap behavior

Quality Assurance (AI-Driven)

Smart Exit: ✅ Exited after 1/5 iterations (perfect on first try)
Test Results: ✅ 24/24 tests PASSED (100%)
Code Review: ✅ APPROVED (0 critical issues, 0 warnings)
Test Coverage: ✅ All retry logic paths covered

Quality Gates

  • ✅ Unit Tests: All 24 tests pass (23 existing + 1 new)
  • ✅ Code Review: No critical issues or warnings
  • ✅ Performance: Acceptable trade-off for improved reliability

Time Saved

Smart exit saved approximately 12-15 minutes by achieving perfect quality on the first iteration instead of running all 5 iterations.

Related

Jira: FSSDK-12248

🤖 Generated with AI-FSSDK automation

@Mat001 Mat001 changed the title [matjaz-test] [AI-FSSDK] (DO NOT REVIEW) [FSSDK-12248] Increase max retry time interval to 3 seconds [matjaz-3-sdks] [AI-FSSDK] (DO NOT REVIEW) [FSSDK-12248] Increase max retry time interval to 3 secs Feb 5, 2026
- Updated max_retry_interval from 1.0 to 3.0 seconds in ODP event manager
- Allows longer retry delays for improved resilience during temporary network issues

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@Mat001 Mat001 force-pushed the ai/mat001/FSSDK-12248-retry-interval branch from 2f7e708 to b0335b8 Compare February 6, 2026 19:08
@Mat001 Mat001 closed this Feb 6, 2026
@Mat001 Mat001 deleted the ai/mat001/FSSDK-12248-retry-interval branch February 6, 2026 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant