Repository navigation
[coverage] Conformance findings: METADATA-035,METADATA-036 #473
Description
Summary
Surfaced by the multi-language coverage fan-out while conformance-testing these SPEC-IDs against databricks/databricks-sql-nodejs. Each finding is committed as an expected-failure (xfail) test in the coverage PR — the test asserts the CORRECT (post-fix) behavior and stays red until THIS driver (databricks/databricks-sql-nodejs) is fixed, then flips green as a tripwire.
Findings
- METADATA-035 [thrift, sea]: session.getTables() treats an empty tableTypes array as "match none" and returns 0 rows instead of behaving like a null filter and matching ALL table types (fix(csharp): correctly handle empty table types filter on
GetTablesadbc-drivers/hiveserver2#72)- failing test:
getTables — tableTypes=[] (empty, non-null) matches ALL table types [xfail: driver bug] [thrift], getTables — tableTypes=[] (empty, non-null) matches ALL table types [xfail: driver bug] [sea](see the coverage PR diff undertests/)
- failing test:
- METADATA-036 [sea]: the SEA metadata path sends a redundant CloseStatement DELETE even when the metadata ExecuteStatement response reported status.state=CLOSED; the driver's regular-query path honors the terminal state correctly, so this is metadata-specific
- failing test:
METADATA-036 — metadata statement close honors ExecuteStatement status.state [sea](see the coverage PR diff undertests/)
- failing test:
Context
- The behavior was first fixed in a DIFFERENT driver — reference PR: https://github.com/databricks/databricks-sql-kernel/pull/183 — which seeded the shared language-neutral spec. This issue tracks the same conformance gap in databricks/databricks-sql-nodejs; the reference PR is for cross-referencing the intended behavior, NOT a change to this repo.
- Coverage PR carrying the reproducing xfail test(s): https://github.com/databricks/databricks-driver-test/pull/1021
Activity
vuanhphung commentedon Nov 13, 2026
Re-checked on
main(7be705e) against a live warehouse, on both the Thrift and kernel backends:getTablesreturns 972 rows with notableTypesfilter and 0 rows withtableTypes: []. That is what the shared spec now requires: fix(csharp): empty tableTypes filter matches none, not all (JDBC parity) adbc-drivers/hiveserver2#83 reversed this case, so an empty, non-null filter matches no table types, the same as databricks-jdbc.close(), so any extra request comes from the Rust kernel. If it still reproduces, it needs a fix in databricks-sql-kernel.Closing, since nothing is left to change in this driver.