Skip to content

fix: SimpleAggregateFunction(anyLast, JSON)#1645

Merged
SpencerTorres merged 1 commit intoClickHouse:mainfrom
bripkens:saf-json-bug
Sep 23, 2025
Merged

fix: SimpleAggregateFunction(anyLast, JSON)#1645
SpencerTorres merged 1 commit intoClickHouse:mainfrom
bripkens:saf-json-bug

Conversation

@bripkens
Copy link
Copy Markdown
Contributor

@bripkens bripkens commented Sep 3, 2025

Why

Attempting to use a SimpleAggregateFunction(anyLast, JSON) column type fails through clickhouse-go.

What

Implement the necessary CustomSerialization interface to transmit serialization version information to ClickHouse.

Fixes #1644

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Sep 3, 2025

CLA assistant check
All committers have signed the CLA.

# Why
Attempting to use a `SimpleAggregateFunction(anyLast, JSON)`
column type fails through clickhouse-go.

# What
Implement the necessary `CustomSerialization` interface to transmit
serialization version information to ClickHouse.
@tomekim
Copy link
Copy Markdown

tomekim commented Sep 12, 2025

also probably fixes #1636

@intmboris
Copy link
Copy Markdown

intmboris commented Sep 23, 2025

@SpencerTorres what is the ETA of merging this fix, please?

@mshustov mshustov requested a review from kavirajk September 23, 2025 07:40
@SpencerTorres SpencerTorres merged commit f2cb5de into ClickHouse:main Sep 23, 2025
14 checks passed
@kavirajk
Copy link
Copy Markdown
Contributor

thanks for the contribution @bripkens :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Columns of type SimpleAggregateFunction(anyLast, JSON) break on insertion

6 participants