[coverage] Conformance findings: METADATA-035,METADATA-036
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 58/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- nodejs, sql, typescript
- 領域
- api
調査の方向性
coverage PR の tests/ diff から始め、METADATA-035 の session.getTables() を追跡して、空の tableTypes の動作を確認します。次に、METADATA-036 の SEA メタデータパスと、その statement のクローズ処理を調べます。tableTypes=[] の動作と CLOSED メタデータ statements の両方について、参照されている xfail ケースが green になれば完了です。
索引モデルが 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
- 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
- 主要言語
- TypeScript
- スター
- 36
- フォーク
- 50
- 平均マージ
- 13時間 46分
- マージ済み PR(30日)
- 9
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
databricks/databricks-sql-nodejs のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
engineer-bot
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
databricks/databricks-sql-nodejs#274 · コメント 1 件 · リアクション 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
databricks/databricks-sql-nodejs の issue をすべて見る
似ている issue
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
Mend: dependency security vulnerability untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100