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

Add support for messagepack #733

Closed
lovasoa opened this issue Jul 10, 2019 · 4 comments · Fixed by #734
Closed

Add support for messagepack #733

lovasoa opened this issue Jul 10, 2019 · 4 comments · Fixed by #734

Comments

@lovasoa
Copy link
Contributor

lovasoa commented Jul 10, 2019

The preferred response format for InfluxDB is now MessagePack, for reasons listed in influxdata/influxdb#8330 .
Supporting it and using it by default when the server supports it too would solve several open issues, such as #665 and #715. It would also improve performance, and lower the required amount of data to be transmitted between the server and the client.

@philip-jones
Copy link

I fully support that, but I think JSON vs MessagePack is a completely separate issue to whether the HTTP POST to InfluxDB is gzip'd or not, so perhaps don't conflate the two issues or get sidetracked from implementing gzip support in influxdb-python.

Gzipping will benefit either transfer format.

@lovasoa
Copy link
Contributor Author

lovasoa commented Jul 12, 2019

To be clear: message pack is used only from the server to the client. gzip is less useful on msgpack-encoded data since it has a high entropy, but it would definitely help for POST data sent from the client to the server.

@philip-jones
Copy link

Sorry folks, probably posted my comment on the wrong issue, should have been on #718 not this one.

@http600
Copy link

http600 commented Jun 10, 2021

ya, but how to make server return in json

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

Successfully merging a pull request may close this issue.

3 participants