Skip to content

ci: set up validation test#151

Merged
lidavidm merged 1 commit intomainfrom
ci
Jan 20, 2026
Merged

ci: set up validation test#151
lidavidm merged 1 commit intomainfrom
ci

Conversation

@lidavidm
Copy link
Contributor

What's Changed

Use the existing CI environment.

@lidavidm
Copy link
Contributor Author

Hi @jadewang-db @eric-wang-1990 as discussed in December, this sets up testing of the Databricks driver using the existing CI secrets. It appears the tests hang currently, so we'll keep working on this, but just a heads up here.

Comment on lines 46 to 47
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be "main" and "adbc_testing"?

@lidavidm
Copy link
Contributor Author

CC @Mandukhai-Alimaa

FAILED validation/tests/test_connection.py::TestConnection::test_get_objects_column_not_exist[databricks:17] - adbc_driver_manager.InternalError: INTERNAL: failed to query tables with columns: databricks: execution error: failed to execute query: unexpected operation state ERROR_STATE: [TABLE_OR_VIEW_NOT_FOUND] The table or view `hive_metastore`.`information_schema`.`COLUMNS` cannot be found. Verify the spelling and correctness of the schema and catalog.
If you did not qualify the name with a schema, verify the current_schema() output, or qualify the name with the correct schema and catalog.
To tolerate the error on drop use DROP VIEW IF EXISTS or DROP TABLE IF EXISTS. SQLSTATE: 42P01; line 1 pos 97

Maybe we can merge this and work on the long tests and this error? (Maybe hive_metastore has a different information_schema?)

@Mandukhai-Alimaa
Copy link
Collaborator

@lidavidm, yes lets merge this in, and I will make a separate PR for the long tests and this error.

@lidavidm lidavidm marked this pull request as ready for review January 20, 2026 06:56
@lidavidm
Copy link
Contributor Author

@amoeba I need your approval to merge 😅

@amoeba
Copy link
Collaborator

amoeba commented Jan 20, 2026

Done!

@lidavidm lidavidm merged commit 24d4b0d into main Jan 20, 2026
12 of 13 checks passed
@lidavidm lidavidm deleted the ci branch January 20, 2026 09:09
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