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

replace switch with simplified if statement

Browse source
This commit is contained in:
Travis Turner 2018-09-26 14:19:51 -05:00
parent 3d33cdbb74
commit 338f69b71d
No known key found for this signature in database
GPG key ID: 7F08008DFD9314C9
1 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
executor.go
View file

@ -1226,11 +1226,7 @@ func (e *executor) executeSetRow(ctx context.Context, index string, c *pql.Call,
if field == nil {
return false, ErrFieldNotFound
}
switch field.Type() {
case FieldTypeSet:
// These field types support SetRow().
default:
if field.Type() != FieldTypeSet {
return false, fmt.Errorf("SetRow() is not supported on %s field types", field.Type())
}
@ -1298,11 +1294,11 @@ func (e *executor) executeSetRowShard(ctx context.Context, index string, c *pql.
return false, errors.Wrapf(err, "creating fragment: %d", shard)
}
}
cleared, err := fragment.setRow(src, rowID)
set, err := fragment.setRow(src, rowID)
if err != nil {
return false, errors.Wrapf(err, "setting row %d on view %s shard %d", rowID, viewStandard, shard)
}
changed = changed || cleared
changed = changed || set
return changed, nil
}