trino
trino copied to clipboard
Use UUIDv7 for OpenLineage runId
Description
Previously OpenLineage run.id was generated as UUIDv3(hash(queryId)). Now it is generated as UUIDv7(queryCreateTime, hash(queryId)), using UUIDUtils.generateStaticUUID method.
Additional context and related issues
According to OpenLineage documentation, it is recommended to use UUID v7 for run.id values.
Related: #25534
Release notes
( ) This is not user-visible or is docs only, and no release notes are required. ( ) Release notes are required. Please propose a release note for me. (X) Release notes are required, with the following suggested text:
## OpenLineage
* Use UUIDv7 for OpenLineage `runId` ({issue}`25534`)
lgtm 🙏🏻
@wendigo could you please take a look?
Could someone take a look? This is important change to make integration compatible with some OpenLineage backends
Thank you very much!