Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for compression #30

Open
espenhogbakk opened this issue May 27, 2024 · 2 comments
Open

Add support for compression #30

espenhogbakk opened this issue May 27, 2024 · 2 comments
Assignees
Labels
feature Product feature

Comments

@espenhogbakk
Copy link

There seems to be no support for compressing the data transferred between server and client. I had a look at the internals and i see there are some flags in the internalConfig for compression that defaults to false. I tried enabling it but seems there is no decompression implemented.

Would be really nice if compression could be added as we see some slow responses related to transfers taking some time.

@kaklakariada kaklakariada added the feature Product feature label May 28, 2024
@kaklakariada
Copy link
Collaborator

The "compression" flag will compress the messages after the login with zlib (see https://github.com/exasol/websocket-api/blob/master/docs/WebsocketAPIV3.md#compression). I assume this would improve performance for large queries or result sets. If this is the case I will add it to the roadmap.

@espenhogbakk
Copy link
Author

Yes large queries would surely see a benfit of that. And it makes sense for the decompression to be handled by the driver instead of outside i think? And even if I wanted to try to do the decompression outside of the driver there is currently no supported way to configure the driver to use compression since the compression is not part of the Config interface we can pass into the initializer.

@pj-spoelders pj-spoelders self-assigned this Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Product feature
Projects
None yet
Development

No branches or pull requests

3 participants