Perfomance improvement. #303

Merged
merged 1 commit into from May 23, 2012

Projects

None yet

2 participants

@ddosia
Contributor
ddosia commented Mar 14, 2012

Function length/1 was used to check for list emptiness. Every time
when chunk of keys had arrived into riak_kv_map_phase:handle_input/3 this
function call was executed on it.

I checked riak_kv_map_phase:handle_input/3 and it is called when i do mapred vi erlang client.
As @seancribbs noticed, this function is called only in legacy modue for {mapred_system, legacy}, but in my app.config i got {mapred_system, pipe}, so could anyone review this behaviour and this commit?

Daniil Churikov Perfomance improvement.
Function length/1 was used to check for list emptiness. Every time
when chunk of keys had arrived into riak_kv_map_phase:handle_input/3 this
function call was executed on it.
4e8d31d
Contributor

Thanks! Sorry it took so long to review this.

@seancribbs seancribbs merged commit 0ce472d into basho:master May 23, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment