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

temporary fix for int field replica sync bug

Browse source
This commit is contained in:
Travis 2020-03-05 20:52:51 -06:00
parent b99bdb8169
commit 86da5c7adf
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
holder.go
View file

@ -911,6 +911,16 @@ func (s *holderSyncer) syncFragment(index, field, view string, shard uint64) err
return ErrFieldNotFound
}
// TODO: this is a temporary fix put in place to prevent
// the syncer from trying to sync replicas of fields
// other than `set` or `time` (i.e. `int`, `mutex`, `bool`,
// `decimal`) using ImportRoaring, because ImportRoaring
// only supports `set` and `time` fields.
if f.Type() != FieldTypeSet && f.Type() != FieldTypeTime {
s.Holder.Logger.Printf("temporarily skipping fragment sync: %s/%d", field, shard)
return nil
}
// Ensure view exists locally.
v, err := f.createViewIfNotExists(view)
if err != nil {