Skip to content

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

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

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

Conversation

@cathleeny

@cathleeny cathleeny commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add the kernel-only, default-enabled enableGeoSpatialSupport connection option
  • return canonical { srid, wkb: Buffer } values when enabled and WKT / EWKT strings when disabled
  • pass the value through the typed kernel binding only; it is never forwarded to SEA

Dependency

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

Validation

  • production build, ESLint, and Prettier: passed
  • unit suite: 1,319 passed
  • live Reyden E2E using the exact kernel native addon: default-enabled, explicit-enabled, and disabled-string modes passed, including WKB equality and nulls

npm run type-check also reaches an existing token-federation example self-import harness failure; the production npm run build type-check is green and this change does not touch that harness.

Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>
Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>
Signed-off-by: Cathleen Yan <58714163+cathleeny@users.noreply.github.com>

This branch was successfully deployed

1 active deployment
azure-prod — edbb1614 Deployed Oct 1, 2026 by cathleeny via e2e-test (26) #1627
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