Map(K, V) type support issue
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Empieza en client-v2/src/main/java/com/clickhouse/client/api/data_formats/internal/BinaryStreamReader.java, en readMap(..), y luego sigue las sobrecargas de JDBC ResultSet getObject mencionadas en el issue. Reproduce la consulta con claves duplicadas e inspecciona cómo se leen los mapas de nivel superior, los mapas anidados y los mapas contenidos en arrays. Se considera terminado cuando el comportamiento predeterminado sigue siendo compatible, mientras que las sugerencias de tipo solicitadas conservan las entradas duplicadas de clave-valor en cada escenario indicado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
In Clickhouse Map(K, V) is not a collection of unique-by-key pairs, see https://clickhouse.com/docs/sql-reference/data-types/map :
i.e. a map can contain two elements with the same key
But there is no way to obtain any representation other than a Map<,> of unique entries.
All the key-value pairs are being delivered by the underlying protocol, but they are effectively merged while reading an instance of a map in the corresponding readMap(..) method.
The issue affects all scenarios of map usage: top-level value, nested map-in-map, map-in-array, etc.
Steps to reproduce
- Execute minimal reproduction program:
public class Program {
public static void main(String[] args) throws SQLException {
String queryText = "select map('key', 'X', 'key', 'Y')";
try (
Connection cnn = DriverManager.getConnection("jdbc:clickhouse://localhost:11049/default", "default", "");
PreparedStatement stmt = cnn.prepareStatement(queryText);
ResultSet rs = stmt.executeQuery()
) {
if (rs.next()) {
System.out.println(rs.getObject(1)); // gives {key=Y}
} else {
throw new IllegalStateException();
}
}
}
}
- Observe only one value per key returned.
- Any attempt to get other value representations than what
getObject()does gives the same or results in an exception being thrown.
Expected Behavior
Supporting existing behavior by default, it is expected to be able to use type hints to affect the actual return value of the getObject(..) method of the java.sql.ResultSet implementation (such overloads as <T> T getObject(int columnIndex, Class<T> type) and Object getObject(int columnIndex, java.util.Map<String,Class<?>> map) specifically, including their counterparts having columnLabel instead of columnIndex):
public class Program {
public static void main(String[] args) throws SQLException {
String queryText = "select map('key', 'X', 'key', 'Y')";
try (
Connection cnn = DriverManager.getConnection("jdbc:clickhouse://localhost:11049/default", "default", "");
PreparedStatement stmt = cnn.prepareStatement(queryText);
ResultSet rs = stmt.executeQuery()
) {
if (rs.next()) {
System.out.println(rs.getObject(1)); // gives {key=Y} by default
System.out.println(rs.getObject(1, List.class)); // to give [key=X, key=Y] only for top level
System.out.println(rs.getObject(1, Map.of("Map", List.class))); // to give [key=X, key=Y] same as above including nested values
} else {
throw new IllegalStateException();
}
}
}
}
Consider changing the default behavior by the time of a certain upcoming major release.
Code Example
select map('key', 'X', 'key', 'Y');
Environment
- Cloud
- Client version: 0.10.0-rc2
- Language version: 21
- OS: Windows 11
ClickHouse Server
- ClickHouse Server version: 24.5.3.5
- Lenguaje dominante
- Java
- Estrellas
- 1.6k
- Forks
- 639
- Merge medio
- 2 d 46 min
- PR fusionados (30 d)
- 46
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la 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 ClickHouse/clickhouse-java
-
[examples] Remove old Spring examplePosiblemente ocupada @polyglotAI-bot la tomó hace 11 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
ClickHouse/clickhouse-java#3111 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
bug client-api-v2 test
Dificultad 2/5 1-3 horas Aptitud para principiantes 92/100
ClickHouse/clickhouse-java#3076 ·
Los mantenedores suelen responder en 1 día
-
area:sql-parser bug client-v1
Dificultad 1/5 1-3 horas Aptitud para principiantes 92/100
ClickHouse/clickhouse-java#3066 ·
Los mantenedores suelen responder en 1 día
-
bug client-api-v2 jdbc-v2
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
ClickHouse/clickhouse-java#2957 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug client-v1 wontfix
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ClickHouse/clickhouse-java#2895 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de ClickHouse/clickhouse-java
Issues similares
-
Clock.MakeTime fails to validate hour, minute, and second ranges due to inert Calendar.set try-catchAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
mit-cml/appinventor-sources#4139 ·
Los mantenedores suelen responder en 1 día
-
[LOOTHUNT] Player Sleep % ToggleAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Vakore/ZappierGames#81 ·
-
proposal
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
beemdevelopment/Aegis#1843 · 1 reacción ·
-
[Bug] Logo style setting missing and `classic` style not applied across multiple platforms (v3.1.0)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Stirling-Tools/Stirling-PDF#8382 · 1 reacción ·
Los mantenedores suelen responder en 2 días
-
>enhancement needs:triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
elastic/elasticsearch#161191 ·
Los mantenedores suelen responder en 1 día