Skip to content

Conversation

@pan3793
Copy link
Member

@pan3793 pan3793 commented Dec 17, 2025

Why are the changes needed?

https://datatracker.ietf.org/doc/html/rfc9562#name-uuid-version-7

https://uuidv7.org/

The Key Benefits of UUIDv7
UUIDv7 is rapidly gaining traction for various use cases across many industries. Here are some of the primary benefits:

  1. Time-Ordered and Sortable: UUIDv7 encodes the time of creation as part of the identifier, making it naturally sortable. This is especially valuable for systems where data needs to be ordered by time without needing an additional timestamp field.
    ...

I think it's very useful - e.g., use UUIDv7 as session ID, then we can infer the session creation time from the UUID itself, or use UUIDv7 as a staging dir name, then we can easily clean dangling staging folders (for example, created 3 months ago) with a prefix.

Implementation is inspired by apache/iceberg#14700

How was this patch tested?

New UTs are added.

Was this patch authored or co-authored using generative AI tooling?

No.

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (8baaba5) to head (1dc1801).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...rc/main/java/org/apache/kyuubi/util/UuidUtils.java 0.00% 17 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #7277   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         697     698    +1     
  Lines       43610   43627   +17     
  Branches     5893    5893           
======================================
- Misses      43610   43627   +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pan3793 pan3793 requested a review from turboFei December 18, 2025 02:16
@pan3793 pan3793 added this to the v1.11.0 milestone Dec 18, 2025
@pan3793 pan3793 self-assigned this Dec 18, 2025
@pan3793 pan3793 requested a review from cxzl25 December 18, 2025 02:17
@pan3793 pan3793 closed this in 63095f0 Dec 18, 2025
@pan3793
Copy link
Member Author

pan3793 commented Dec 18, 2025

thanks, merged to master

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.

4 participants