scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-09-07 00:55:55 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

safe cast of bool in executeClearRow

Browse source
This commit is contained in:
Travis 2020-08-03 18:00:52 -05:00
parent 7f6c6956d8
commit ddee2cb0a8
No known key found for this signature in database
GPG key ID: 37080CC2042BA34E
1 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
executor.go
View file

@ -3301,11 +3301,18 @@ func (e *executor) executeClearRow(ctx context.Context, tx Tx, index string, c *
// Merge returned results at coordinating node.
reduceFn := func(ctx context.Context, prev, v interface{}) interface{} {
val := v.(bool)
if prev == nil {
val, ok := v.(bool)
if !ok {
return errors.Errorf("executeClearRow.reduceFn: val is non-bool (%+v)", v)
}
if prev == nil || val {
return val
}
return val || prev.(bool)
pval, ok := prev.(bool)
if !ok {
return errors.Errorf("executeClearRow.reduceFn: prev is non-bool (%+v)", prev)
}
return pval
}
result, err := e.mapReduce(ctx, index, shards, c, opt, mapFn, reduceFn)
@ -3393,7 +3400,7 @@ func (e *executor) executeSetRow(ctx context.Context, tx Tx, indexName string, c
if !ok {
return errors.Errorf("executeSetRow.reduceFn: val is non-bool (%+v)", v)
}
if val {
if prev == nil || val {
return val
}