ClickHouse / clickhouse-java

ClickHouse Java Clients & JDBC Driver
https://clickhouse.com
Apache License 2.0
1.44k stars 530 forks source link

[client-v2] primitive reading into pojo #1826

Open chernser opened 1 week ago

chernser commented 1 week ago

Summary

Current implementation reads all values into objects even it can be done using primitive types. It was done to simplify code, but it may be not good for memory consumption. Current PR:

Checklist

Delete items not relevant to your PR:

mzitnik commented 1 week ago

Do we support POJO with types of Long, Integer, Short, and more class

chernser commented 1 week ago

Do we support POJO with types of Long, Integer, Short, and more class

We do but with 1-to-1 match. But with primitives it should be easier to handle. I will update this part.

sonarcloud[bot] commented 4 days ago

Quality Gate Failed Quality Gate failed

Failed conditions
77.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud