Skip to content

feat: support geospatial results on the kernel path - #492

Draft
cathleeny wants to merge 2 commits into
mainfrom
cathleeny/geospatial-results
Draft

cathleeny wants to merge 2 commits into
mainfrom
cathleeny/geospatial-results

Conversation

@cathleeny

@cathleeny cathleeny commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add WithEnableGeoSpatialSupport and the enableGeoSpatialSupport DSN parameter
  • default to enabled: return raw WKB through database/sql while preserving the canonical {srid,wkb} Arrow struct through GetArrowBatches
  • return WKT / EWKT strings when disabled
  • pass the value through the typed kernel C ABI only; it is never forwarded to SEA

Dependency

Depends on databricks/databricks-sql-kernel#361 at b7e9310b27be16a6c42490e58c4b5c4a525320bc.

Validation

  • uncached full go test -tags databricks_kernel ./... against the exact kernel Rustls archive: passed
  • generated C header exactly matches kernel
  • live Reyden E2E against the exact archive: default-enabled, explicit-enabled, and disabled-string modes passed, including logical type metadata, WKB equality, and nulls

Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>
@cathleeny
cathleeny force-pushed the cathleeny/geospatial-results branch from ab7567d to 1d78b88 Compare September 30, 2026 20:37
Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>

This branch has not been deployed

No deployments
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