Skip to content

Conversation

@danfrankj
Copy link
Collaborator

@danfrankj danfrankj commented Jan 25, 2019

pandas finally merged in the multi-values insert so we no longer need it here.

Tested with a sqlalchemy client

@matthewwardrop
Copy link
Collaborator

Thanks for staying on top of this. This does bump our dependency on pandas to a very recent version, which may be concerning to some deployments. Do you think we should defer merging this for a while, or just go for it?

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.

4 participants