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

Groundwork to prepare for arrays #89

Closed
amunra opened this issue Mar 11, 2025 · 0 comments · Fixed by #94 · May be fixed by #96
Closed

Groundwork to prepare for arrays #89

amunra opened this issue Mar 11, 2025 · 0 comments · Fixed by #94 · May be fixed by #96

Comments

@amunra
Copy link
Collaborator

amunra commented Mar 11, 2025

We will soon be introducing an nd-array type in QuestDB and in the ILP protocol.

  • The new array fields will be sent as binary.
  • The existing Buffer is internally based on a mutable String type in Rust: This needs to be modified to internally use a Vec<u8> instead.
  • The existing methods that expose the buffer's accumulated string also need to be removed and replaced with equivalent methods that expose bytes. C, C++ and Python APIs need to be adjusted accordingly.
  • We also need to update our ureq dependency to 3.x. This will mean reworking some of the TLS logic since the new library more clearly decouples these details.

The resulting PR will thus effectively introduce a minor but nevertheless breaking change in the API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant