Skip to content

Conversation

@allisonwang-db
Copy link
Contributor

What changes were proposed in this pull request?

This PR fixes a few invalid error class references and adds more tests. Two error classes are invalid:

  • PYTHON_DATA_SOURCE_TYPE_MISMATCH -> DATA_SOURCE_TYPE_MISMATCH
  • PYTHON_DATA_SOURCE_READ_RETURN_SCHEMA_MISMATCH -> DATA_SOURCE_RETURN_SCHEMA_MISMATCH

Why are the changes needed?

To fix invalid error class references.

Does this PR introduce any user-facing change?

No

How was this patch tested?

New unit tests.

Was this patch authored or co-authored using generative AI tooling?

No

@allisonwang-db
Copy link
Contributor Author

cc @HyukjinKwon

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending CI

@HyukjinKwon
Copy link
Member

Merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants