docs: clickhouse-odbc SQL_NULL_DATA fix merged upstream (#586) - #148
Merged
Merged
Conversation
Merged 2026-09-24 by the maintainer, commit eb0fdb2654, six days after it was opened and with no changes requested. Recorded as half fixed rather than fixed, deliberately: issue #582 reported two defects and this is the first. The parameter-array half is a different mechanism -- SQLMoreResults reports result sets, not parameter sets -- so #582 stays open upstream and the status widget will keep showing it that way. Not in a release either. The reproducer was re-run against 1.5.5.20260810 on 2026-09-24 against ClickHouse 26.7.5.10 and still shows all five problems, so the compatibility flag stays and the shipped-in line waits for a tag.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Our fix ClickHouse/clickhouse-odbc#586 was merged on 2026-09-24 by the maintainer (slabko), commit
eb0fdb2654— six days after opening, no changes requested. That makes ClickHouse the fifth project to merge a fix from this work, and the second fix in that driver after #585.Recorded as half fixed, not fixed. Issue #582 reported two defects together and this is the first of them. The parameter-array half —
SQLMoreResultsexecuting the next set and answeringSQL_NO_DATA, so a five-set array lands two rows — is a different mechanism and needs its own change. So #582 stays open upstream, the status widget will keep showing it open, and the row says so rather than implying the report is closed. I offered on the pull request to prepare that second fix.Verified, and it changes nothing for users yet. The reproducer from #582 was re-run today against the current release, 1.5.5.20260810, on ClickHouse 26.7.5.10: still five problems — the
Stringcolumn silently storing'', and theInt32/Float64/Date/Decimalparameters failing server-side. So the compatibility caveat stays in place, scoped to 1.5.5, and the "fixed in" line waits for a tag that carries the change.Changes:
docs/UPSTREAM.md— the row moves to half fixed, with the merge commit, what it fixes, and what it does not.docs/COMPATIBILITY.md— the ClickHouse caveat is scoped to 1.5.5 and links the merged fix.docs/compatibility.json— regenerated, as CI requires.The note at
/notes/clickhouse-odbc-findings/gains an "Update, 2026-09-24" callout with the same facts; it publishes with the next site build.