[coverage] Conformance findings: LOGGING-005

オープン
#498 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
55/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
静か
技術スタック
node.js, sql, typescript

調査の方向性

tests/ 配下の coverage PR diff にある失敗している LOGGING-005 テストから始め、期待される動作をリファレンスの databricks-odbc PR と比較します。次に、この Node.js ドライバー内の対応する Thrift と SEA のロギング経路を追跡し、再現テストを実行します。両方のバックエンドが期待されるロギング動作を満たし、テストがグリーンになることが完了の条件です。

索引モデルが issue の本文から書いたものです。

説明

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

主要言語
TypeScript
スター
36
フォーク
50
平均マージ
13時間 46分
マージ済み PR(30日)
9

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

databricks/databricks-sql-nodejs のほかの issue

databricks/databricks-sql-nodejs の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。