[coverage] Conformance findings: LOGGING-005
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 55/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Quiet
- Tech stack
- node.js, sql, typescript
- Domain
- backend, observability
Research direction
Start with the LOGGING-005 failing tests in the coverage PR diff under tests/ and compare their expected behavior with the reference databricks-odbc PR. Then trace the corresponding Thrift and SEA logging paths in this Node.js driver and run the reproducing tests; done means both backends satisfy the expected logging behavior and the tests turn green.
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
- LOGGING-005 [thrift]: Thrift backend emits one "Fetched chunk of size: N from operation with id: …" record per fetch at LogLevel.debug (DEBUG volume scales with fetch count) and emits no aggregate per-result-set summary at teardown; the exported LogLevel enum has no
tracemember, so there is no less-verbose level to demote the per-fetch detail to.- failing test:
LOGGING-005 — driver bug: per-fetch record emitted at DEBUG with no per-result-set summary [thrift](see the coverage PR diff undertests/)
- failing test:
- LOGGING-005 [sea]: SEA kernel backend emits one "Fetched chunk of size: N from operation with id: …" record per fetch at LogLevel.debug (DEBUG volume scales with fetch count) and emits no aggregate per-result-set summary at teardown, including for a result set abandoned after 2 of 100 rows; the exported LogLevel enum has no
tracemember, so there is no less-verbose level to demote the per-fetch detail to.- failing test:
LOGGING-005 — driver bug: per-fetch record emitted at DEBUG with no per-result-set summary [sea](see the coverage PR diff undertests/)
- failing test:
- LOGGING-005: At LogLevel.debug the driver logs one "Fetched chunk of size: N from operation with id: …" record per fetch (unbounded in fetch count — 1-row metadata-browsing rowsets flood the log) and emits no aggregate per-result-set summary at teardown; the per-fetch detail also has no less-verbose level to be demoted to because the exported LogLevel enum has no
tracemember. Affects both the Thrift backend and the SEA kernel.
Context
- The behavior was first fixed in a DIFFERENT driver — reference PR: https://github.com/databricks/databricks-odbc/pull/252 — 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/1267
- Dominant language
- TypeScript
- Stars
- 36
- Forks
- 50
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 12
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 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
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
All issues in databricks/databricks-sql-nodejs
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
check:passed streams:add
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100