Add ManualStorageSegment for direct HTTP integration with Segment ana…#398
Add ManualStorageSegment for direct HTTP integration with Segment ana…#398cshiels-ie wants to merge 1 commit into
Conversation
…lytics This new class implements a direct HTTP storage backend for Segment, bypassing the segment-analytics-python library to prevent issues with oversized POST requests. Key features include unique message IDs and timestamps for each chunk to avoid deduplication, and improved error handling for HTTP responses. The implementation retains chunking behavior from the existing StorageSegment class while enhancing reliability in event delivery.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|


…lytics
This new class implements a direct HTTP storage backend for Segment, bypassing the segment-analytics-python library to prevent issues with oversized POST requests. Key features include unique message IDs and timestamps for each chunk to avoid deduplication, and improved error handling for HTTP responses. The implementation retains chunking behavior from the existing StorageSegment class while enhancing reliability in event delivery.
[AAP-XXXXX]
Dont forget to link back issue to PR.
Description
Testing
Prerequisites
Steps to Test
Expected Results
Required Actions
Self-Review Checklist
Code Quality
- If updates are required in the handbook, create a separate PR for the handbook repo.
Notes for Reviewers
Add any additional context or instructions for reviewers here - for example screenshots if needed.