Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

Use default DB in InfluxDBClient.write.#602

Open
bgusach wants to merge2 commits intoinfluxdata:master
base:master
Choose a base branch
Loading
frombgusach:master

Conversation

bgusach
Copy link

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 inInfluxDBClient._write_points and .query.
@patricktokeeffe
Copy link

A better approach would be to collect params insidewrite(), as was done forwrite_points() in7fae65f

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

Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers
No reviews
Assignees

@sebito91sebito91

Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@bgusach@patricktokeeffe@sebito91@xginn8
[8]ページ先頭
©2009-2025 Movatter.jp