Closed nicolaparo closed 9 months ago
The idea is definitely interesting. I have some consideration about your suggestion:
Gpt
instead of GPT
for the project nameDatabaseGpt.Core
, I would prefer DatabaseGpt.Abstractions
, using the same naming convention that is adopted by MicrosoftDatabaseGptConsole
project should be moved to a samples folder.I have assigned the task to you @nicolaparo, thank your for collaboration!
DatabaseGPT currently supports only scenarios where we have SqlServer databases. It doesn't currently support the case of a different SQL DBMS like Postgres, SQLite, MySql, etc... The major difference between all these DBMS is mainly the dialect of the SQL language used in the database and the required drivers themselves.
Since a user who is interested in using DatabaseGPT to query a SqlServer might not be interested in having the drivers also for another DBMS, it is preferrable to not include all the possible database drivers in a single package. I suggest then the refactor of the solution as follows:
If the idea is valid and interesting, I can start to design a more detailed refactoring plan and the database-specific implementations for SqlServer and Postgres.