Skip to content

Conversation

0xRobin
Copy link

@0xRobin 0xRobin commented Oct 10, 2025

Note

Adds Snowflake and BigQuery SQL drivers and updates dependencies (incl. AWS SDK) and codegen to support them.

  • Drivers:
    • Enable github.com/snowflakedb/gosnowflake and github.com/solcates/go-sql-bigquery via drivers.go and add to drivers_gen.go custom list.
  • Codegen:
    • Import github.com/dave/jennifer/jen and use it in drivers_gen.go for generating drivers.go.
  • Dependencies:
    • Add direct deps: [email protected], [email protected].
    • Bump github.com/aws/aws-sdk-go-v2 to v1.38.1 and update related AWS packages.
    • Add/update numerous indirect GCP/BigQuery, thrift, OpenTelemetry, and other transitive deps in go.mod/go.sum.

Written by Cursor Bugbot for commit 331a703. This will update automatically on new commits. Configure here.

@0xRobin 0xRobin requested a review from a-monteiro October 10, 2025 13:34
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment @cursor review or bugbot run to trigger another review on this PR

//"github.com/mithrandie/csvq-driver",
"github.com/trinodb/trino-go-client/trino",
"github.com/databricks/databricks-sql-go",
"github.com/snowflakedb/gosnowflake",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Snowflake Driver Duplicate in Driver Lists

The github.com/snowflakedb/gosnowflake driver is duplicated in both the extra and custom driver lists. This creates an inconsistent categorization, which can lead to confusion for maintainers and potential issues if driver generation logic changes.

Fix in Cursor Fix in Web

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