search

databricks / databricks-sql-go

Golang database/sql driver for Databricks SQL.
Apache License 2.0
37 stars 41 forks source link

Pass ctx properly in internal client.ExecuteStatement #242

Closed shelldandy closed 1 month ago

shelldandy commented 3 months ago

https://github.com/databricks/databricks-sql-go/blob/93222abd7f5be64af98bd90fe8d60972d365eda6/internal/client/client.go#L162-L171

This needs to actually pass ctx so custom authenticators can make use of the context.

The bug was introduced here:

https://github.com/databricks/databricks-sql-go/pull/60/files#diff-3b1f8149deb45d304c69c856bcd1b9b628961fb6000486cb39172a99cb8de514L91-R90