[coverage] Conformance findings: CLOUDFETCH-018
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 62/100
Línea de trabajo
Comienza con TestLinkFetchFailureSurfacesATerminalError en el PR de cobertura bajo tests/ y, después, reproduce el escenario de CloudFetch de múltiples chunks con un fallo persistente al obtener el enlace. Sigue el fallo de CloudFetch y la ruta de drain del driver de Go. Se considera terminado cuando el primer drain produce un error en un plazo de 180 segundos, el drain repetido retorna en un plazo de 30 segundos y el protocolo confirma que se realizó una obtención de chunk.
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-go. 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-go) is fixed, then flips green as a tripwire.
Findings
- CLOUDFETCH-018 [sea]: A permanently-failing CloudFetch link fetch (403 on GET .../result/chunks/{n}) never returns: after 32 attempts the failure is logged and a waiter woken, but it is never recorded as terminal provider state and end-of-stream is left unset, so the drain blocks past 180s with 0 rows instead of raising a terminal error (kernel #319; the pure-Go Thrift path conforms, giving up in 35s after 5 attempts)
- failing test:
TestLinkFetchFailureSurfacesATerminalError(see the coverage PR diff undertests/)
- failing test:
Reproduce & Expected
CLOUDFETCH-018 — A permanently-failing CloudFetch LINK FETCH must surface a TERMINAL error to the consumer, and must do so in bounded time.
Reproduce:
- Enable this run's protocol scenario (see protocol_scenarios above) so the
link-fetch call fails persistently. The result must be large enough to span
MULTIPLE chunks, so the driver has to ask for links it does not already hold --
otherwise every link arrives inline, no link fetch is issued, and the injected
failure never fires (the protocol guard below catches that vacuous case). - Drain a multi-chunk CloudFetch result whose link fetch fails permanently.
Expect the drain to raise -- not to block, and not to report a clean
end-of-stream over a truncated row set. - Fetch again on the SAME result after the terminal error. Must return promptly
(end-of-stream or the same error), not hang.
Expected (per the shared spec):
- full assertion contract:
result:
- label: first_drain
exception_thrown: true
- label: first_drain
elapsed_seconds_range:
max: 180
- label: repeat_drain
either_condition_true:
- row_count: 0
- exception_thrown: true
- label: repeat_drain
elapsed_seconds_range:
max: 30
protocol:
thrift:
- label: first_drain
call_min:
method: FetchResults
min: 1
sea:
- label: first_drain
call_min:
operation: FetchResultChunk
min: 1
Context
- The behavior was first fixed in a DIFFERENT driver — reference PR: https://github.com/databricks/databricks-sql-kernel/pull/322 — which seeded the shared language-neutral spec. This issue tracks the same conformance gap in databricks/databricks-sql-go; 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/1571
- Lenguaje dominante
- Go
- Estrellas
- 53
- Forks
- 66
- Merge medio
- 7 h 57 min
- PR fusionados (30 d)
- 16
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-go
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
databricks/databricks-sql-go#476 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
databricks/databricks-sql-go#481 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
databricks/databricks-sql-go#480 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
databricks/databricks-sql-go#475 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
databricks/databricks-sql-go#474 ·
Todos los issues de databricks/databricks-sql-go
Issues similares
-
nix: vendorHash is outdated Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
Bob Shell support Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
santhosh-tekuri/jsonschema#276 ·