Skip to content

Conversation

@sgbalogh
Copy link
Member

Two changes:

  • Introduces a StreamPosition type for capturing seqNum/timestamp pairs (in append ack, and checkTail responses)
  • Removes Instant conversions when dealing with timestamp
    • This was assuming that all timestamps are epoch ms; this is not necessarily the case if a client supplies their own timestamp and opts to use a different convention, so I've moved to long

@sgbalogh sgbalogh marked this pull request as ready for review June 18, 2025 00:57
@sgbalogh sgbalogh merged commit 53d4db8 into main Jun 18, 2025
2 checks passed
@sgbalogh sgbalogh deleted the update-jun17 branch June 18, 2025 01:14
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.

3 participants