[coverage] Conformance findings: METADATA-035,METADATA-036

Aperta
#473 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
58/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
nodejs, sql, typescript
Ambito
api

Direzione di ricerca

Inizia dal diff di tests/ della coverage PR e segui session.getTables() per METADATA-035, verificando il comportamento di tableTypes vuoto. Poi esamina il percorso dei metadati SEA per METADATA-036 e la relativa gestione della chiusura degli statements. Il lavoro è completato quando i casi xfail indicati diventano verdi sia per il comportamento di tableTypes=[] sia per gli statements di metadati CLOSED.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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 under tests/)
  • 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 under tests/)

Context

Lingua principale
TypeScript
Stelle
36
Fork
50
Merge medio
13h 46m
PR unite (30g)
9

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di databricks/databricks-sql-nodejs

Tutte le issue di databricks/databricks-sql-nodejs

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.