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

Implement index-agnostic Response #3

Closed
igorcoding opened this issue Oct 27, 2017 · 0 comments
Closed

Implement index-agnostic Response #3

igorcoding opened this issue Oct 27, 2017 · 0 comments

Comments

@igorcoding
Copy link
Owner

igorcoding commented Oct 27, 2017

asynctnt.Response's entries should be agnostic to indexing (either by an index or by a key).
Almost everything is there for implementing it (fields are already provided for each tuple in response), so it should be fairly easy to implement lazy build of dictionaries. body property should be deprecated.
Response should in some way contain tuples that can be accessed either by a key or a numeric index.

This produced some questions with non-space requests (such as call and eval), because there are no actual "tuples" (in sense of space) there.

By implementing this feature parameter tuple_as_dict to all requests would be also declared as deprecated and removed in the future.

Probably should wait until SQL protocol in Tarantool is documented to implement more general solution.

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

No branches or pull requests

1 participant