[coverage] Conformance findings: METADATA-035,METADATA-036
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 58/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Quiet
- Tech stack
- nodejs, sql, typescript
- Domain
- api
Research direction
Start with the coverage PR's tests/ diff and trace session.getTables() for METADATA-035, checking the empty tableTypes behavior. Then inspect the SEA metadata path for METADATA-036 and its statement-close handling. Done means the referenced xfail cases become green for both tableTypes=[] behavior and CLOSED metadata statements.
Written by the indexing model from the issue text.
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 (adbc-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
- Dominant language
- TypeScript
- Stars
- 36
- Forks
- 50
- Avg merge
- 13h 46m
- Merged PRs (30d)
- 9
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from databricks/databricks-sql-nodejs
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
engineer-bot
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
databricks/databricks-sql-nodejs#274 · 1 comment · 1 reaction ·
-
Difficulty 3/5 1-2 days Newbie friendliness 68/100
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
All issues in databricks/databricks-sql-nodejs
Similar issues
-
clawsweeper:linked-pr-open clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:message-loss issue-rating: 🦞 diamond lobster maturity:stable P2
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Eynzof/Hermes-CN-Desktop#616 ·
-
ZCode 3.14.3 に対応する Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
supermomonga/zcode-acp#24 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
growthbook/growthbook#7100 ·
-
triage
Difficulty 1/5 1-3 hours Newbie friendliness 88/100