Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wiki: SQLDrivers #67200

Open
sdavids opened this issue May 6, 2024 · 2 comments
Open

wiki: SQLDrivers #67200

sdavids opened this issue May 6, 2024 · 2 comments
Labels
Documentation help wanted NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.

Comments

@sdavids
Copy link

sdavids commented May 6, 2024

Drivers marked with [*] are both included in and pass the compatibility test suite

https://go.dev/wiki/SQLDrivers

The list of drivers tested has not been updated in years:

https://github.com/bradfitz/go-sql-test/blob/master/src/sqltest/drivers.go


I suggest you:

a) update the drivers to be tested and the update the wiki page afterward (and update it periodically)
b) remove the mention of the compatibility test suite
c) state that the compatibility test was done in 2019 (5 years ago)

@mauri870 mauri870 added Documentation NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. help wanted labels May 6, 2024
@mauri870
Copy link
Member

mauri870 commented May 6, 2024

I'm wondering if there would be any benefit from the Go team adopting this test suite, perhaps integrating it into x/tools.

Apart from that, updates to the documentation page would be appreciated.

cc @bradfitz

@mauri870
Copy link
Member

mauri870 commented May 6, 2024

The list of drivers seems to be updated with a certain frequency. I guess the question is more about the test suite itself, I highly doubt the most recent entries added to the list were tested with go-sql-test.

@mauri870 mauri870 added NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation help wanted NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Projects
None yet
Development

No branches or pull requests

2 participants