Skip to content

Conversation

@renaudhartert-db
Copy link
Contributor

@renaudhartert-db renaudhartert-db commented Aug 11, 2025

What changes are proposed in this pull request?

This PR is a first step in adding support for all the Databricks error detail types. Specifically, this PR introduces AutoValue classes for each error detail and make sure that these are parsed properly. However, the types are not exposed yet in the actual DatabricksError exception, this will come in a subsequent PR.

While it might not look as such, the change in ApiErrorBody is practically a no-op because:

  1. The server can only return one error info.
  2. Only error info types were extracted from the list of details.

How is this tested?

Unit tests focused on deserialization.

NO_CHANGELOG=true

@github-actions
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-java

Inputs:

  • PR number: 497
  • Commit SHA: 2f963ec451bc84648c6fac1fc7a1d3110b433bc8

Checks will be approved automatically on success.

@renaudhartert-db renaudhartert-db added this pull request to the merge queue Aug 13, 2025
Merged via the queue into main with commit d2b185a Aug 13, 2025
16 checks passed
@renaudhartert-db renaudhartert-db deleted the renaud-hartert_data/error1 branch August 13, 2025 08:52
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.

3 participants