Skip to content

chore: use Uuid type for Transaction.uuid instead of String #6346

@wjones127

Description

@wjones127

Transaction.uuid is typed as String in lance/src/dataset/transaction.rs, but it always holds a UUID value. The proto field (pb::Transaction.uuid) is also a string.

Change Transaction.uuid to Uuid, parsing from the proto string in TryFrom<pb::Transaction> and formatting with .hyphenated() at the one callsite that builds the .txn filename. Also update TransactionBuilder::uuid to accept Uuid instead of String, which removes .to_string() calls at builder callsites that already hold a Uuid.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions