[coverage] Conformance findings: LOGGING-005

Open
#498 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
55/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Quiet

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 trace member, 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 under tests/)
  • 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 trace member, 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 under tests/)
  • 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 trace member. Affects both the Thrift backend and the SEA kernel.

Context

Dominant language
TypeScript
Stars
36
Forks
50
Avg merge
1d 6h
Merged PRs (30d)
12

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.