Remove legacy listkeys config option #239

Merged
merged 1 commit into from Nov 15, 2012

Conversation

Projects
None yet
2 participants
Contributor

kellymclaughlin commented Nov 14, 2012

Legacy keylisting is being removed from riak_kv so the legacy_keylisting configuration option is now obsolete.

@ghost ghost assigned kellymclaughlin Nov 14, 2012

Remove legacy_keylisting config option from app.config template
Fixes: RIAK239

Legacy keylisting is being removed from riak_kv so the
legacy_keylisting configuration option is now obsolete.
Contributor

kellymclaughlin commented Nov 14, 2012

Related riak_kv pr is here.

@ghost ghost assigned slfritchie Nov 14, 2012

Contributor

slfritchie commented Nov 14, 2012

Hrm, I thought that I saw this same app config change with the Luke app removal review that I did recently? If I'm thinking of another config item that @beerriot removed, then great.

This same packaging change should also be applied to the riak_ee repo?

I don't see anything in basho/riak_kv#436 that looks objectionable, +1 to merge.

@ghost ghost assigned kellymclaughlin Nov 14, 2012

Contributor

kellymclaughlin commented Nov 14, 2012

I think it was a similar, but different item that was in the luke removal. I've added a pr here to mirror this change in riak_ee.

@kellymclaughlin kellymclaughlin merged commit a53237f into master Nov 15, 2012

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