Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add {legacy_vnode_routing, false} to default app.config. #129

Merged
merged 1 commit into from Apr 4, 2012

Conversation

Projects
None yet
2 participants
Contributor

jonmeredith commented Apr 3, 2012

Set default correctly for new installs.

Contributor

jtuple commented Apr 4, 2012

Change is straightforward. Went ahead and built a 64-bit Ubuntu package off this branch and tested installing it clean versus installing over existing Riak install (and keeping existing config).

On fresh install:

(riak@127.0.0.1)2> application:get_env(riak_core, legacy_vnode_routing).
{ok,false}

On install-over-existing:

(riak@127.0.0.1)2> application:get_env(riak_core, legacy_vnode_routing).
undefined

When undefined, Riak defaults to legacy mode. All looks good to me, short of running the basho_expect rolling upgrade test, which seems like overkill here.

+1 Merge

jonmeredith added a commit that referenced this pull request Apr 4, 2012

Merge pull request #129 from basho/jdm-default-legacy-vnode-routing-f…
…alse

Add {legacy_vnode_routing, false} to default app.config.

@jonmeredith jonmeredith merged commit f301615 into 1.1 Apr 4, 2012

@javajolt javajolt deleted the jdm-default-legacy-vnode-routing-false branch Feb 26, 2016

hmmr pushed a commit that referenced this pull request Nov 8, 2016

Merge pull request #129 from basho/dip_add_131
added EE 1.3.1 to riak_test build script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment