trino icon indicating copy to clipboard operation
trino copied to clipboard

Use UUIDv7 for OpenLineage runId

Open dolfinus opened this issue 6 months ago • 2 comments
trafficstars

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`)

dolfinus avatar Apr 28 '25 11:04 dolfinus

lgtm 🙏🏻

mgorsk1 avatar May 19 '25 14:05 mgorsk1

@wendigo could you please take a look?

dolfinus avatar May 30 '25 08:05 dolfinus

Could someone take a look? This is important change to make integration compatible with some OpenLineage backends

dolfinus avatar Jun 24 '25 08:06 dolfinus

Thank you very much!

dolfinus avatar Jun 25 '25 06:06 dolfinus