fix update_many with new columns
fixes a bug where updating many records with new columns and ensuring those columns are created would raise a sqlalchemy.exc.CompileError exception as reported in #409
the pull request includes two new unit-tests:
- ensure the bug is fixed
- demonstrate the behaviour when new columns are present but the
ensureargument is set toFalse
in making these changes i did not consider the cases under point 2, i'm uncertain what the right behaviour should be under those circumstances
as stated by @4l1fe in #409 this solution may work on other methods
a way to improve the insert_many(), upsert_many(), update_many() methods is to add a new wrapper _sync_columns_many()
i wouldn't mind working on this if there is some indication the work will get merged ping @pudo, @gka
Hi @medecau and thanks a lot for your contribution. It's been a while since I last worked on this project, but I'll see if I can get @pudo's attention somehow!
thanks @gka pinged you because i noticed your handle on pypi's page for dataset