[coverage] Conformance findings: DATATYPE-041,STATEMENT-023

Open
#470 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
48/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Quiet
Tech stack
sql, typescript
Domain
api, backend, databases

Research direction

Start with the DATATYPE-041 and STATEMENT-023 expected-failure tests in the coverage PR, then compare the behavior with the reference databricks-sql-go PR. Done means the Node.js driver reports STRING for the untyped NULL column and returns the affected-row counts for the 3-row INSERT and 2-row UPDATE through the SEA backend.

Written by the indexing model from the issue text.

Description

Surfaced by the multi-language coverage fan-out while conformance-testing these SPEC-IDs against databricks/databricks-sql-nodejs.

Each is committed as an expected-failure (xfail) test in the coverage PR — the test asserts the CORRECT behavior and stays red until this driver is fixed.

  • DATATYPE-041: SEA backend reports an untyped NULL (VOID) column as TTypeId 16 (NULL) instead of the server-reported STRING type; the same query on Thrift correctly reports STRING, so the column type a caller sees depends on the transport
  • STATEMENT-023: SEA backend reports no affected-row count for real DML (numModifiedRows absent for a 3-row INSERT and a 2-row UPDATE) while Thrift reports 3 and 2; absent is returned for every statement alike, so a caller cannot distinguish a real modified-row count from "no count available"

Reference driver PR: https://github.com/databricks/databricks-sql-go/pull/416
Coverage PR (reproducing xfail tests): https://github.com/databricks/databricks-driver-test/pull/1010

Dominant language
TypeScript
Stars
36
Forks
50
Avg merge
13h 46m
Merged PRs (30d)
9

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from databricks/databricks-sql-nodejs

All issues in databricks/databricks-sql-nodejs

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.