(#6018) - simplify processNextBatch in mapreduce #6018

Merged
merged 1 commit into from Dec 18, 2016

Projects

None yet

2 participants

@nolanlawson
Member

I noticed some extra profiling time being spent in EventEmitter during secondary indexing, which is not strictly necessary if we use normal Promises instead of the 'complete' event. As a side effect this also makes the code more idiomatic and easy to understand.

@nolanlawson nolanlawson (#6018) - simplify processNextBatch in mapreduce
1a80315
@nolanlawson nolanlawson changed the title from (#6017) - simplify processNextBatch in mapreduce to (#6018) - simplify processNextBatch in mapreduce Dec 16, 2016
@daleharvey

Nice cleanup

@daleharvey daleharvey merged commit 03f93b8 into master Dec 18, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls First build on master at 100.0%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment