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 executemany functionality to RedshiftClient #28

Merged
merged 1 commit into from
Jun 14, 2024
Merged

Conversation

aaronfriedman6
Copy link
Contributor

In addition to unit tests, tested locally that this works as expected

In addition to unit tests, tested locally that this works as expected
Copy link
Contributor

@fatimarahman fatimarahman left a comment

Choose a reason for hiding this comment

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

Looks good! I like the use of the all function, very space efficient

@aaronfriedman6 aaronfriedman6 merged commit 86bb686 into main Jun 14, 2024
2 checks passed
@aaronfriedman6 aaronfriedman6 deleted the insert-many branch June 14, 2024 15:31
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.

2 participants