Thank you Masahiko, Tristan, Christophe, and Zsolt for the reviews and
feedback. Addressing the feedback in a single email.

Attached v2 addressing the feedback:

- Moved infinity check before timestamp arithmetic [Masahiko]
- Used INT64CONST() for the 48-bit constant [Masahiko]
- Added documentation for the valid timestamp range [Masahiko, Christophe]
- Added a test for '292230 years' to cover the overflow path
 caught by pg_add_s64_overflow() [Zsolt Parragi]

0001 - Reject infinite intervals
0002 - Reject pre-epoch timestamps (with overflow-safe epoch conversion)
0003 - Reject timestamps beyond the 48-bit limit

Let me know if I have missed anything.

Thanks,
Baji Shaik.

Attachment: v2-0001-Reject-infinite-intervals-in-uuidv7-interval.patch
Description: Binary data

Attachment: v2-0003-Reject-timestamps-beyond-48-bit-limit-in-uuidv7-i.patch
Description: Binary data

Attachment: v2-0002-Reject-pre-epoch-timestamps-in-uuidv7-interval.patch
Description: Binary data

Reply via email to