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

Use iterator_refresh backend option for some folds #852

Merged
merged 1 commit into from
Feb 21, 2014

Conversation

andrewjstone
Copy link
Contributor

  • Add iterator_refresh capability to riak_kv_elveldb_backend and
    riak_kv_multi_backend
  • Handle iterator_refresh option/capability for ?FOLD_REQs in
    riak_kv_vnode
  • support iterator_refresh option for AAE (riak_kv_index_hashtree)
  • add riak_kv_vnode:fold/4
  • return handoff fold options from handoff_started
    • handoff_started has been modified to require this
  • Get value of iterator_refresh from env. Default to true.

 * Add iterator_refresh capability to riak_kv_elveldb_backend and
riak_kv_multi_backend
 * Handle iterator_refresh option/capability for ?FOLD_REQs in
   riak_kv_vnode
 * support iterator_refresh option for AAE (riak_kv_index_hashtree)
 * add riak_kv_vnode:fold/4
 * return handoff fold options from handoff_started
   * handoff_started has been modified to require this
 * Get value of iterator_refresh from env. Default to true.
@andrewjstone
Copy link
Contributor Author

Replaces #830 . See that for discussion.

@jrwest
Copy link
Contributor

jrwest commented Feb 21, 2014

re-iterating my +1. any eunit failures @andrewjstone and I came across (riak_kv_schema_tests and keys_fsm_eqc seem to be unrelated)


%% @private
maybe_enable_iterator_refresh(Capabilities, Opts) ->
case lists:member(iterator_refresh, Capabilities) of
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fwiw, we could have dumped the config check in here instead of in two places but meh

andrewjstone added a commit that referenced this pull request Feb 21, 2014
Use iterator_refresh backend option for some folds
@andrewjstone andrewjstone merged commit 10f21a1 into develop Feb 21, 2014
@andrewjstone
Copy link
Contributor Author

#853 addresses @jrwest's comment above

@seancribbs seancribbs deleted the ajs-iterator-refresh-squashed-rebased branch April 1, 2015 23:36
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.

2 participants