Pass ctx properly in internal client.ExecuteStatement

[shelldandy]

databricks-sql-go/internal/client/client.go

Lines 162 to 171 in 93222ab

	// ExecuteStatement is a wrapper around the thrift operation ExecuteStatement
	// If RecordResults is true, the results will be marshalled to JSON format and written to ExecuteStatement<index>.json
	func (tsc *ThriftServiceClient) ExecuteStatement(ctx context.Context, req *cli_service.TExecuteStatementReq) (*cli_service.TExecuteStatementResp, error) {
	ctx = startClientMethod(ctx, clientMethodExecuteStatement)
	var log *logger.DBSQLLogger
	log, ctx = LoggerAndContext(ctx, req)
	msg, start := log.Track("ExecuteStatement")
	// We use context.Background to fix a problem where on context done the query would not be cancelled.
	resp, err := tsc.TCLIServiceClient.ExecuteStatement(context.Background(), req)

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