Add comment clarifying current codepath

This commit is contained in:
Peter Rabbitson 2020-10-10 15:36:32 +02:00
parent 85abca4b16
commit 23f5a99117

View File

@ -454,7 +454,17 @@ func (cs *ChainStore) takeHeaviestTipSet(ctx context.Context, ts *types.TipSet)
func (cs *ChainStore) FlushValidationCache() error { func (cs *ChainStore) FlushValidationCache() error {
log.Infof("clearing block validation cache...") log.Infof("clearing block validation cache...")
dsWalk, err := cs.ds.Query(query.Query{KeysOnly: true}) dsWalk, err := cs.ds.Query(query.Query{
// Potential TODO: the validation cache is not a namespace on its own
// but is rather constructed as prefixed-key `foo:bar` via .Instance(), which
// in turn does not work with the filter, which can match only on `foo/bar`
//
// If this is addressed (blockcache goes into its own sub-namespace) then
// strings.HasPrefix(...) below can be skipped
//
//Prefix: blockValidationCacheKeyPrefix.String()
KeysOnly: true,
})
if err != nil { if err != nil {
return xerrors.Errorf("failed to initialize key listing query: %w", err) return xerrors.Errorf("failed to initialize key listing query: %w", err)
} }