Skip to content

fix(api): require warehouse-backed sync surfaces#184

Merged
josh-writer merged 2 commits intosnowflake-provider-storage-warehouse-20260327from
snowflake-api-warehouse-surfaces-20260327
Apr 14, 2026
Merged

fix(api): require warehouse-backed sync surfaces#184
josh-writer merged 2 commits intosnowflake-provider-storage-warehouse-20260327from
snowflake-api-warehouse-surfaces-20260327

Conversation

@jonathanhaaswriter
Copy link
Copy Markdown
Collaborator

Summary

  • remove the remaining API sync fallback to direct Snowflake clients and require the configured warehouse surface instead
  • update API handlers to report warehouse not configured for warehouse-backed data endpoints
  • rewrite sync/API regression tests to use the generic warehouse test double instead of Snowflake clients

Validation

  • go test ./internal/api
  • go test -race ./...
  • GOTOOLCHAIN=go1.26.1 go run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.8.0 run --timeout 5m ./...

@jonathanhaaswriter jonathanhaaswriter force-pushed the snowflake-provider-storage-warehouse-20260327 branch from 01ae291 to 0d496b3 Compare March 28, 2026 02:20
@jonathanhaaswriter jonathanhaaswriter force-pushed the snowflake-api-warehouse-surfaces-20260327 branch from 6205c41 to a32dc23 Compare March 28, 2026 02:26
@jonathanhaaswriter jonathanhaaswriter force-pushed the snowflake-provider-storage-warehouse-20260327 branch from fc5b2e8 to fc3eb92 Compare April 1, 2026 17:59
@jonathanhaaswriter jonathanhaaswriter force-pushed the snowflake-api-warehouse-surfaces-20260327 branch from b805a5a to 26d300e Compare April 1, 2026 18:01
@josh-writer josh-writer merged commit e573f3e into snowflake-provider-storage-warehouse-20260327 Apr 14, 2026
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.

2 participants