scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-08-28 10:54:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

don't reduce errors with non-errors

Browse source 
if we got an error, we don't have to merge it. so either ctx.Err or
resp.err being non-nil means we shouldn't be reducing, but we still need
to grab the responses to make sure we waited for them all.
This commit is contained in:
Seebs 2021-02-26 14:26:02 -06:00
parent 29fddd40f6
commit 7d15fc2f26
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
executor.go
View file

@ -5786,7 +5786,7 @@ func (e *executor) mapperLocal(ctx context.Context, shards []uint64, mapFn mapFu
if resp.err != nil && err == nil {
err = resp.err
}
if ctx.Err() == nil {
if resp.err == nil && ctx.Err() == nil {
// Only useful to do a possibly-expensive
// reduce if we don't already know we don't
// need it.