influxdata / influxdb-client-python

InfluxDB 2.0 python client
https://influxdb-client.readthedocs.io/en/stable/
MIT License
724 stars 187 forks source link

fix(write-api): accept 201 response to write #663

Closed jacobmarble closed 3 months ago

jacobmarble commented 3 months ago

Helps internal issue https://github.com/influxdata/idpe/issues/18710

Proposed Changes

InfluxDB v3 Serverless will soon return 201 or 204, in cases where InfluxDB v1 and v2 only return 204. This change anticipates this API change. Public docs will also be updated.

InfluxDB v3 Dedicated and Clustered already have this behavior, and it is already documented.

Checklist

codecov-commenter commented 3 months ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.43%. Comparing base (653af46) to head (9b4cf12). Report is 13 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #663 +/- ## ======================================= Coverage 90.43% 90.43% ======================================= Files 40 40 Lines 3524 3524 ======================================= Hits 3187 3187 Misses 337 337 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jacobmarble commented 3 months ago

@bednar I don't have permission to merge, so I'll leave it to you. Thanks for your attention!