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

Support for Riak 2.0. Ability to use special indices $bucket, $key. Ability to provide max_results and continuation in db.query and get continuation back. #187

Closed
wants to merge 32 commits into from

Conversation

mogadanez
Copy link
Contributor

#180
basic tests included

Alexander Popov and others added 30 commits September 4, 2013 18:03
add the indices to meta.index on get as they are defined when
setting them via ```meta.index = { foo: 1, bar: "baz" }``` on save.

FIXES mostlyserious#183
… of Object[Object], because buffer is often looks like:

{
   phase: 0
   error: "[{<<"lineno">>,466},{<<"message">>,<<"SyntaxError: syntax error">>},{<<"source">... (length: 92)"
   input: "{ok,{r_object,<<"users">>,<<"b1f0e0b64f8143a6a45977db8e29ff89">>,[{r_content,{di... (length: 515)"
}
either we use asi or we don't mixing is bad
Since just dumping data in Riak without ever cleaning up is not nice I
added a cleanup for all writen keys in the tests. This should also make
sure that we don't have dependend tests

Also commeted out search tests relying in pre solar functionaliy
this should only return the keys added (4) since we clean the bucket
between tests
only expect keys this tests explicitly saves
otherwise riak.conf will be ignored due to a present app.config
locally 1 sec is more than enough wait for indexes to propagate but on
travis this seems to not be the case let's see...
@mogadanez mogadanez changed the title Ability to use special indices $bucket, $key. Ability to provide max_results and continuation in db.query and get continuation back. Support for Riak 2.0. Ability to use special indices $bucket, $key. Ability to provide max_results and continuation in db.query and get continuation back. Nov 20, 2014
@mogadanez mogadanez closed this Nov 20, 2014
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.

3 participants