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

Accept box_tuple_t in box C APIs #10007

Open
e-ivkov opened this issue May 14, 2024 · 0 comments
Open

Accept box_tuple_t in box C APIs #10007

e-ivkov opened this issue May 14, 2024 · 0 comments
Labels
feature A new functionality

Comments

@e-ivkov
Copy link

e-ivkov commented May 14, 2024

Is your feature request related to a problem? Please describe.

When I already have a tuple of box_tuple_t type that I got from tarantool interface I want to be able to supply it as an argument to box_insert, box_replace. So that it will not be needed to first offload tuple contents to buffer and only then call box_*. Offloading to buffer causes performance and memory overhead.

Describe the solution you'd like

To add function:

int box_replace_tuple(uint32_t space_id, box_tuple_t *tuple, box_tuple_t **result)

In addition to already existing:

int box_replace(uint32_t space_id, const char *tuple, const char *tuple_end, box_tuple_t **result)

Similarly do it for box_insert

Describe alternatives you've considered

Change the interface of box_replace. But I don't see the need to break compatibility over this.

Additional context

box_replace docs

Definition of Done

box_insert_tuple and box_replace_tuple functions added to C API

@e-ivkov e-ivkov added the feature A new functionality label May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new functionality
Projects
None yet
Development

No branches or pull requests

1 participant