Merge branch 'cloudfetch' into main #154

mattdeekay · 2023-08-04T21:07:31Z

Supports executing queries with Cloud Fetch for increased performance and caching.

Steps taken:

Synced fork mattdeekay for both cloudfetch and main branches
On mattdeekay:main, ran git merge --squash cloudfetch
Resolved merge conflicts
Fixed cloudfetch_test.go end-to-end test to (WithEnableCloudFetch -> WithCloudFetch)
Commit and create PR
Fix connector_test.go to add cloud fetch (forgot to add earlier)
Add link expiration test to batchloader_test.go
Fix arrowRows_test.go
golangci-lint run

Signed-off-by: Matthew Kim <11141331+mattdeekay@users.noreply.github.com>

Merge branch 'cloudfetch' into main

4be8ace

Signed-off-by: Matthew Kim <11141331+mattdeekay@users.noreply.github.com>

mattdeekay requested review from susodapop, yunbodeng-db, andrefurlan-db and rcypher-databricks as code owners August 4, 2023 21:07

mattdeekay self-assigned this Aug 4, 2023

Fix connector_test.go and gofmt lint

da16c0f

Signed-off-by: Matthew Kim <11141331+mattdeekay@users.noreply.github.com>

mattdeekay force-pushed the main branch from 86b6ec4 to da16c0f Compare August 4, 2023 21:25

mattdeekay added 4 commits August 4, 2023 15:10

Add link expiration test for batchloader_test.go

fdc0181

Signed-off-by: Matthew Kim <11141331+mattdeekay@users.noreply.github.com>

End-to-end test should not be a _test.go

fc9d9cb

Signed-off-by: Matthew Kim <11141331+mattdeekay@users.noreply.github.com>

Fix arrowRows_test.go

66b7905

Signed-off-by: Matthew Kim <11141331+mattdeekay@users.noreply.github.com>

Remove unused err field

b8c87f7

Signed-off-by: Matthew Kim <11141331+mattdeekay@users.noreply.github.com>

rcypher-databricks approved these changes Aug 8, 2023

View reviewed changes

rcypher-databricks merged commit 7e079fd into databricks:main Aug 8, 2023
3 checks passed

Provide feedback