Skip to content
This repository has been archived by the owner on Oct 29, 2024. It is now read-only.

Default database not being used in write method. #601

Open
bgusach opened this issue Jul 2, 2018 · 3 comments
Open

Default database not being used in write method. #601

bgusach opened this issue Jul 2, 2018 · 3 comments

Comments

@bgusach
Copy link

bgusach commented Jul 2, 2018

Hi,

I had some confusing problems while using the write method, and after some debugging realized that DB passed to the constructor is not used at all. On the other hand, the method write_points does use this DB, and I think this is the desired behaviour.

Is there any reason behing this? or it is just a bug?

I can submit a patch if deemed necessary.

@xginn8
Copy link
Collaborator

xginn8 commented Jul 3, 2018

I can't find anything specifically as to why that's not around, feel free to submit a patch! Note as well that passing a params={'db': 'your_db'} should unblock you in the interim

@bgusach
Copy link
Author

bgusach commented Jul 3, 2018

Hi,

I just created a pull request: #602

@patricktokeeffe
Copy link

I also find this discrepancy between write and write_points confusing and unnecessary. I specified a default database for a reason...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants