ClickHouse / ClickHouse/clickhouse-java

Progress Header Support

Abierto
#3,080 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Java
Estrellas
1.6k
Forks
636
Merge medio
2 d 23 h
PR fusionados (30 d)
29

Descripción

### Description

Support multiple `X-ClickHouse-Progress` for tracking progress.

**Note**:
Bumping httpclient5 to 5.6.4 also pulls httpcore5 5.4.3, which defaults incoming HTTP/1 messages to 100 headers and an 8192-byte line length (https://github.com/advisories/GHSA-hf6x-8p5f-cgmf). Both HTTP stacks build Http1Config with only a buffer size and pass DefaultHttpResponseParserFactory.INSTANCE, so those new limits apply. ClickHouse send_progress_in_http_headers can emit a new X-ClickHouse-Progress header every http_headers_progress_interval_ms (default 100ms), and queryAll/queryRecords force wait_end_of_query, so a long query can exceed 100 headers and fail at parse time instead of returning a result or server error.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza por la configuración del parser de respuestas HTTP/1 utilizada por queryAll y queryRecords, incluidos Http1Config y DefaultHttpResponseParserFactory, y revisa el comportamiento de la actualización de httpclient5/httpcore5. Reproduce una consulta larga con encabezados X-ClickHouse-Progress repetidos; se considera terminado cuando la respuesta se analiza correctamente en lugar de fallar al alcanzar el límite de encabezados.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
api
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Activo
Claridad
Bastante claro
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.