[coverage] Conformance findings: CLOUDFETCH-012
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- sql, typescript
Línea de trabajo
Comienza con la prueba xfail del PR de cobertura para CLOUDFETCH-012 y las rutas del controlador que gestionan las opciones de executeStatement y databricks.cloudfetch.enabled. Compara el comportamiento previsto con el PR de referencia 225 y, después, sigue las solicitudes ExecuteStatement y CreateSession de SEA/kernel. Se considera terminado cuando se cumple el contrato: se utilizan INLINE y ARROW_STREAM, can_cloud_download está ausente, se devuelve al menos una fila y hay cero llamadas a cloud_download.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- CLOUDFETCH-012 [sea]: SEA/kernel path silently ignores CloudFetch-disable: both per-statement
useCloudFetch:false(driver logs "no-op on kernel") and the connection-leveldatabricks.cloudfetch.enabled=falseextraParameter are dropped, so ExecuteStatement still sends disposition=INLINE_OR_EXTERNAL_LINKS instead of INLINE- failing test:
CLOUDFETCH-012 — cloudfetch disabled: disposition=INLINE, no can_cloud_download conf, 0 cloud downloads [sea](see the coverage PR diff undertests/)
- failing test:
- CLOUDFETCH-012: SEA/kernel path silently ignores CloudFetch-disable: both per-statement
executeStatement(sql, {useCloudFetch:false})(driver logs "no-op on kernel") and the connection-leveldatabricks.cloudfetch.enabled=falseextraParameter are dropped, so ExecuteStatement still sends disposition=INLINE_OR_EXTERNAL_LINKS instead of INLINE and callers who disable CloudFetch keep receiving external links
Reproduce & Expected
CLOUDFETCH-012 — Validates that when CloudFetch is disabled, no CloudFetch activity occurs and results are fetched via the driver's inline result path instead.
Reproduce:
- Execute query with CloudFetch disabled
Expected (per the shared spec):
- completes without an exception
- result has at least 1 row(s)
- [thrift]
ExecuteStatementrequestcanDownloadResult== False - [thrift] exactly 0
cloud_downloadcall(s) - [sea]
ExecuteStatementrequestdisposition== 'INLINE' - [sea]
ExecuteStatementrequestformat== 'ARROW_STREAM' - [sea]
CreateSessionrequestsession_confs.can_cloud_downloadis absent - [sea] exactly 0
cloud_downloadcall(s) - full assertion contract:
result:
- no_exception: true
- row_count_min: 1
- result_not_null_with_data: true
protocol:
thrift:
- request_field:
method: ExecuteStatement
path: canDownloadResult
equals: false
- call_min:
method: FetchResults
min: 1
- call_count:
method: cloud_download
expected: 0
sea:
- request_field:
operation: ExecuteStatement
path: disposition
equals: INLINE
- request_field:
operation: ExecuteStatement
path: format
equals: ARROW_STREAM
- request_field:
operation: CreateSession
path: session_confs.can_cloud_download
present: false
- call_count:
method: cloud_download
expected: 0
Context
- The behavior was first fixed in a DIFFERENT driver — reference PR: https://github.com/databricks/databricks-sql-kernel/pull/225 — 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/1212
- Lenguaje dominante
- TypeScript
- Estrellas
- 36
- Forks
- 50
- Merge medio
- 13 h 46 min
- PR fusionados (30 d)
- 9
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de databricks/databricks-sql-nodejs
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Docs folder deleted in 1.8.4 Abiertoengineer-bot
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
databricks/databricks-sql-nodejs#274 · 1 comentario · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de databricks/databricks-sql-nodejs
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
ontola/atomic-server#1625 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
melgarafael/DeskcommCRM#1451 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
-
bug via-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
bot:ai-assisted component:compact-js status:untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
midnightntwrk/midnight-sdk#403 ·