Skip to content

Conversation

@devdavidkarlsson
Copy link
Contributor

Summary

2524-setSocketLinger type in config properties not matching docs, socket linger is expecting an integer of seconds, but config properties validates to require boolean

Closes #2524

@CLAassistant
Copy link

CLAassistant commented Oct 17, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 🤙

💡 To request another review, post a new comment with "/windsurf-review".

…ket linger is expecting an integer of seconds, but config properties validates to require boolean
@devdavidkarlsson devdavidkarlsson force-pushed the set-socket-linger-type-fix branch from f69900f to fa91ae0 Compare October 17, 2025 10:30
@mshustov mshustov requested a review from chernser October 17, 2025 13:33
@chernser chernser merged commit d70d5b7 into ClickHouse:main Oct 20, 2025
32 of 39 checks passed
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.

socket_linger client config Int vs Boolean

3 participants