Skip to content

Use source instead of message: source.to_string() #3224

@wjones127

Description

@wjones127

Right now, our error messages can end up being huge chains of junk prefixes:

InvalidInputException: Invalid Input Error: arrow_scan: get_next failed(): Invalid: C Data interface error: Io error: Execution error: External error: Execution error: LanceError(IO): failed to read all bytes from file: expected 480, got 437, /home/runner/work/lance/lance/rust/lance/src/io/local.rs:121:31

We should:

  • change our error enum to use source field.
  • Make sure to add an IO wrapper that has a field for data file path we are reading / writing
  • find some way to pass through full errors in the C data interface (or write our own RBR implementation)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions