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

Use default DB in InfluxDBClient.write. #602

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

Conversation

bgusach
Copy link

@bgusach bgusach commented Jul 3, 2018

Centralize default parameters in a function, and reuse these defaults in
InfluxDBClient._write_points and .query.

Centralize default parameters in a function, and reuse these defaults in
InfluxDBClient._write_points and .query.
@patricktokeeffe
Copy link

A better approach would be to collect params inside write(), as was done for write_points() in 7fae65f

Also, write() should get a new database=None argument (like write_points has) that documents the use of connection's default database.

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

Successfully merging this pull request may close these issues.

None yet

4 participants