datafuselabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.65k stars 726 forks source link

Return json null for nullable values in http api response data #16111

Closed everpcpc closed 1 month ago

everpcpc commented 1 month ago

cc @sundy-li

It seems https://github.com/datafuselabs/databend/issues/15960 should be reverted? cc @youngsofun

Server

Clients

youngsofun commented 1 month ago

@everpcpc no need to revert๏ผŒthe old JsonBlock only use json::Value::String, null is output as "NULL" like output format, change it to StringBlock is more clear.

but your PRs are good. the problem is https://github.com/datafuselabs/databend/issues/16076.

hantmac commented 1 month ago

databend-jdbc no need to modify because the data context defined as an object. https://github.com/datafuselabs/databend-jdbc/blob/main/databend-client/src/main/java/com/databend/client/QueryResults.java#L48