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 new Query API #53

Closed
wants to merge 6 commits into from

Conversation

martinsumner
Copy link

@martinsumner martinsumner commented Dec 15, 2024

Add support for new query API.

The riak_kv_query module allows a query to be constructed with validation checks.

The riak_kv_query_server is a gen_server for managing a query. The coverage_fsm behaviour is not used, to aid future migration from the deprecated fsm.

There is a riak_kv_query_buffer added to replace the use of riak_kv_fold_buffer - which assumes the accumulator is a list.

Add support for new query API.

The riak_kv_query module allows a query to be constructed with validation checks.

The riak_kv_query_server is a gen_Server for managing a query.  The coverage_fsm behaviour is not used, to aid future migration from the deprecated fsm.

There is a riak_kv_query_buffer added to replace the use of riak_kv_fold_buffer - which assumes the accumulator is a list.
Resolves OTP 24 dialyzer complaints after clarifying specs in backend behaviour
upstream meck has updated master, and no longer supports OTP < 25
Have both vnode and node stats (as with other operations)
Add HTTP API for Query - allowing for PSOT of JSON with JSON response.
@martinsumner martinsumner changed the title Initial query commit Add support for new Query API Dec 17, 2024
The keys may be read in every oder - the keys are not ordered with the term.

Likewise with term_with_keys as the term may now be extracted from within the term (i.e. a projected attribute)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant