Skip to content

docs: tracker — fix PRs sent for OpenSearch #101/#102 and ClickHouse's parameter-array half - #150

Merged
singhpratech merged 1 commit into
mainfrom
docs/tracker-588-opensearch-prs
Sep 26, 2026
Merged

singhpratech merged 1 commit into
mainfrom
docs/tracker-588-opensearch-prs

Conversation

@singhpratech

Copy link
Copy Markdown
Owner

Three tracker rows catch up with pull requests sent this week. None has landed; each row now names its PR, what it changes, and how far it was verified — which differs a lot between them, and the rows say so.

  • OpenSearch #102 → fix PR #104. OPENSEARCHAPI_ConnectError never cleared the connection error after reporting it, so unixODBC's SQLDriverConnect(W) loop over SQLErrorW never reached SQL_NO_DATA_FOUND and a failed connect spun at 100% CPU. Four lines, following the pattern EN_get_error() already uses for the environment.
  • OpenSearch #101 → fix PR #105. A character column asked for as a datetime came back as today's date under SQL_SUCCESS; character sources now go through timestamp2stime() and an unreadable string returns 22018 — which required adding that SQLSTATE, since STMT_STRING_CONVERSION_ERROR had none.
  • ClickHouse #582, parameter-array half → fix PR #588. The row previously said this half "stays open for its own PR"; it now has one, plus the maintainer's review and the error-path work it asked for.

Verification is stated honestly per row. The OpenSearch pair is syntax-checked only — that repo's CMake hardcodes a 2019 kernel-headers path and its tests need vcpkg, the AWS SDK and a live cluster — so their CI is the first real run. The ClickHouse fix was built and run against a live 26.7.5.10 with the project's own suite, 391 of 392 passing, the one failure being environmental and identical on the released driver.

docs/compatibility.json regenerated as CI requires (no content change — these rows are in UPSTREAM.md, not the compatibility table).

…s parameter-array half

opensearch-project/sql-odbc#104 (ConnectError never clears its error, so a
failed connect spins forever) and #105 (a character value asked for as a
datetime came back as today's date) were opened 2026-09-24. ClickHouse/
clickhouse-odbc#588 covers the second half of #582; the maintainer's review
asked for the ODBC error-path contract, which was added the same day and
verified on a driver built from the branch.

None of the three has landed. The rows say what each PR does and how far it
was verified, which for the OpenSearch pair is a syntax check only.
@singhpratech
singhpratech merged commit 394e337 into main Sep 26, 2026
6 checks passed
@singhpratech
singhpratech deleted the docs/tracker-588-opensearch-prs branch September 26, 2026 17:17
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.

1 participant