Skip to content

Conversation

@pl
Copy link
Contributor

@pl pl commented Nov 27, 2025

Problem

When timestamp validation in cookieless manager fails, it throws an error which is handled by the generic try-catch, causing all cookieless events in the batch to be dropped.

Changes

  • Refactors the cookieless manager to drop only the messages with timestamps out of range.
  • Adds ingestion warnings for events submitted with out of range timestamps

How did you test this code?

  • Added new tests for timestamp validation
  • Existing regression tests

@pl pl requested review from a team and robbie-c November 27, 2025 23:17
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

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.

2 participants