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

Separate NuGet Package to Reduce Unnecessary Dependencies #517

Open
agilenut opened this issue Sep 1, 2022 · 1 comment
Open

Separate NuGet Package to Reduce Unnecessary Dependencies #517

agilenut opened this issue Sep 1, 2022 · 1 comment
Assignees
Labels
feature status-triage_done Initial triage done, will be further handled by the driver team

Comments

@agilenut
Copy link

agilenut commented Sep 1, 2022

This library currently brings in a lot of unwanted dependencies. I'd like to propose a multi-package structure that creates a better separation of concerns and improves dependency management.

This doesn't have to be exact but I'm imagining something like this:

  • Snowflake.Data.Core - Responsible for authenticating to Snowflake and providing a type-safe wrapper around the API. Should ideally have no dependencies other than the .net framework.
  • Snowflake.Data.Ado - References Snowflake.Data.Core and adds the ADO.Net implementations
  • Snowflake.Data.Aws - References Snowflake.Data.Core and AWS libs and adds AWS specific implementations. There would be similar packages for Azure and Google.

Then you could create a light-weight dapper style mapper like Snowflake.Client for those of us that would prefer a lighter weight interface to map to POCOs and don't need ADO.

  • Snowflake.Data.Querying - References Snowflake.Data.Core and adds dapper style POCO mapping extensions.
@sfc-gh-dszmolka sfc-gh-dszmolka self-assigned this Jun 8, 2023
@sfc-gh-dszmolka sfc-gh-dszmolka added status-triage Issue is under initial triage and removed status-triage Issue is under initial triage labels Jun 8, 2023
@sfc-gh-dszmolka
Copy link
Contributor

hello and thank you for submitting this issue! also apologies for the long period without response. we'll take a look and see if and how this could be put on the roadmap.

@sfc-gh-dszmolka sfc-gh-dszmolka added the status-triage_done Initial triage done, will be further handled by the driver team label Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature status-triage_done Initial triage done, will be further handled by the driver team
Projects
None yet
Development

No branches or pull requests

4 participants