Use 'logger' instead of 'error_logger'

[martinsumner]

wday-contrib 2325

Passes all eqc tests

[martinsumner]

OpenRiak/riak_kv#111

[WarpEngineer]

I didn't look through riak_core_util.erl since I'm guessing that was just formatting.

[martinsumner]

@WarpEngineer

With regards to riak_core_util, it is a merge issue. There was a commit, adding the Workday attribution to the Apache license, and changing the pmap_test unit test (formatting only) ... but there's also some functions that had been added prior to this in the workday branch related to calculating timestamps (for your larger timeout monitoring change), which aren't in the openriak branch. This seems to confuse the diff process, and it creates a diff for the whole module.

Not sure how to avoid this., but there isn't any changes that I'm aware of other than those highlighted here. It might have been a mistake to take the wday commits out of order.

[WarpEngineer]

@martinsumner I skimmed the file and compared to my local and it looks like the timeout functions are the major difference. I see no issues otherwise. I think it's good to go.