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

Add temp table bulk loading #17

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

davidtme
Copy link

Proposed Changes

This allows you to bulk load temp tables which can then be used in queries such as merging large amounts of data.

I added a similar feature to FSharp.Data.SqlClient as I use this in a ETL process.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • Build and tests pass locally
  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have added necessary documentation (if appropriate)

Notes

The naming of the function such as createTempTable, tempTableData and loadTempTable as well as the structure of their use completely open debate but I found this parameter structure easy to work with in the Analyzer

As the structure could change I will add documentation once it has been agreed.

@davidtme
Copy link
Author

I'm not sure how to get the appveyor working but I'm not sure it's todo with my PR.

@Numpsy
Copy link

Numpsy commented Feb 14, 2021

This is a bit of a guess (my knowledge of appveyor is quite slim), but the appveyor config file might need to specify which CI image to use to make sure it has the new enough tools (netcore3 etc).

Possibly try adding

image: Visual Studio 2019

to the appveyor.yml file?

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.

None yet

2 participants